Any chance this wil hit SVN soon?
![Wink ;)](./images/smilies/icon_wink.gif)
CAP-Team wrote:I really like the new way of drawing stars. Is this going to be an extra option for CTRL+S or will it replace the older methods?
Any chance this wil hit SVN soon?
chris wrote:CAP-Team wrote:I really like the new way of drawing stars. Is this going to be an extra option for CTRL+S or will it replace the older methods?
I'd like it to replace the older methods on capable hardware. After using this new star rendering technique, none of the old methods seem adequate.
--Chris
CAP-Team wrote:Chris, in the pictures above, do you use the standard star database or some other database?
Guckytos wrote:chris wrote:CAP-Team wrote:I really like the new way of drawing stars. Is this going to be an extra option for CTRL+S or will it replace the older methods?
I'd like it to replace the older methods on capable hardware. After using this new star rendering technique, none of the old methods seem adequate.
--Chris
Chris,
what is "capable hardware" in this case? Can you give a minimum graphics chip generation? And how would it impact on the FPS of this minimum?
Boux wrote:I have been playing with Chris's code (big thanks for sharing!).
The new star rendering is very nice and can be gorgeous with some tweaking.
First, I have got rid of the 1 Ly solar system size limit so that the transition between shader and mesh rendering is smooth.
Then I have been playing with the variables and I found that brightnessBias is the most controllable.
The screenshots below - /!\ big pics! - are made with brightnessBias set to 0.5, 0.7 and 0.8
Code: Select all
// Saturation magnitude is the magnitude at which a star centered on a pixel
// will have the maximum pixel value.
float saturationMag = faintestMag - 4.5f;
// Minimum pixel value (wihout gamma correction) that is visible; this will vary
// based on display device and viewing conditions
float justVisiblePixelValue = 1.0f / 300.0f;
Boux wrote:For what it's worth, here is what I am finding here:
- Qt4 build 5043, EXT_framebuffer_sRGB extension supported
Without patch:
- Start-up screen, automag on --> 149 fps
- Center on alpha Crux, automag on --> 193 fps
- Center on alpha Crux, automag off, limitig magnitude 12 --> 150 fps
With patch:
- Start-up screen, automag on --> 150 fps
- Center on alpha Crux, automag on --> 195 fps
- Center on alpha Crux, automag off, limitig magnitude 12 --> 153 fps
No significant loss or gain of performance.
I did the test several times and the results are pretty steady.
Looks like the EXT_framebuffer_sRGB extension is doing its job.
t00fri wrote:Chris,
in the context of our Qt discussion, I compiled SVN 5043 with your new star patch, which worked fine.
However, with my laptop (the graphics card of which does not support the EXT_framebuffer_sRGB extension as you know), I observe a rather dramatic decrease in performance (star style = Point):
SVN 5043, Qt 4.6.3: ..............................55 fps for startup screen
SVN 5043 + your star patch, Qt 4.6.3: 32 fps for same startup screen
or worse, centering on Alfa Crux with, automag=OFF and lim.mag ( ] ) = 12
SVN 5043, Qt 4.6.3:................................44.2 fps
SVN 5043 + your star patch, Qt 4.6.3: 22.0 fps! i.e. 50%
How is the analogous situation with a card supporting EXT_framebuffer_sRGB ?
Anyway, it seems some further speed-tuning might be necessary.
Code: Select all
Limiting mag New stars Fuzzy points Scaled discs
10.0 370 342 335
12.0 284 342 304
Boux wrote:No significant loss or gain of performance.
I did the test several times and the results are pretty steady.
Looks like the EXT_framebuffer_sRGB extension is doing its job.
Code: Select all
Limiting mag. linToSRGB on linToSRGB off
10 370 370
12 238 284
t00fri wrote:Chris,
my own interpretation of the lim.mag 12 drop matches yours perfectly. I also had tried lim.mag=10, 8
but displayed the one with the dramatic drop.
This was the alfa Crux - centered test image with your new stars that above gave the strong fps-drop by a factor of two. The large number of star pixels to be drawn here is evident.