You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
I can fix the conflicts with the master branch. Let me know if you prefer me to do this with a merge commit or rebasing this branch onto current master.
Let's count current bugs I created and left 🥲
(For rendering box testing, I turned on the red filler at line 47 in star_frag.glsl)
There are two stages of rendering a star, depending on its individual visible brightness: "9 px" mode (3x3 box size) and "glow" mode (variable box size). For simplicity, the separating value is chosen to be 1 Vega brightness at exposure 1. The separation is in star_vert.glsl at line 46.
You can notice that in the glow mode, the glow doesn't completely fill the box. Actually fills, but only in internal Float values. sRGB gamma correction is assumed in the glow design, which will “pull” it to the box edges.
Bug 1
Star field have never been seen using both modes at once, depending on the brightness of a particular star. If you don't change the code (if (all(lessThan(scaled_color, vec3(1.0))))), the glow mode is used. If you change to if (true), the 9 px mode is used.
That's how 4 px mode looked like. Stars feel square, that's why I moved it to 9 px instead.
Bug 2
The 9 px mode was broken since my last testing in the summer 2024. When rotating the camera, the 1 pixel stars rarely appear and then disappear again:
Bug 3
The wrong brightness values are obviously being supplied to the input. The glow mode worked correctly before the exposure transition was made:
Bug 4
After ruthless code reduction, barycenters began to emit. I'm guessing it was an old optimization whose limits were accidentally removed. I think it's best to find and remove the barycenter glow all the way through.
Bug 5
Limb darkening was added, but the star surfaces have stopped rendering. It could be a typo or caused by deleting something extra.
Bug 6
There is a small “bug” with exposure setting in the interface: I failed to make integer output when updating a variable, i.e. now Exposure: 16.00 is printed instead of Exposure: 16. I rolled back my experiments
Notes
Based on the Discord discussions, it would be both more realistic and easier to program the overall process of rendering both stars, planets, and spacecraft if they are in point mode. Without special brightness enhancement of planets, for example. For the sake of simplifying visual search, you can set different thresholds for displaying names. Now there are no star styles, which may allow simplifying the logic globally. I also removed the glow and atmosphere of the stars, while adding changes from the limb-darkening branch to optimize the logic of everything planned at once. To keep the Sun looking bright, I suggest increasing the maximum size of the glow to the size of the screen. At typical exposures, only the Sun will be bright, and it will switch to a starry background with no jumps when removed.
Also, maximum glow box size is discussed to be min(min(screen_hight, screen_width), max_point_size), depending on the max point size that openGL supports.
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.