Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Atmosphere background shader to the scene #567

Merged
merged 15 commits into from
Apr 21, 2024
Merged

Add Atmosphere background shader to the scene #567

merged 15 commits into from
Apr 21, 2024

Conversation

smlpt
Copy link
Contributor

@smlpt smlpt commented Mar 12, 2024

This PR adds the option to add an atmosphere background from the "Add" menu to the scene.
The atmosphere exposes three parameters in the inspector:

  • Latitude: The current latitude of the user to display the correct sun direction based on the local time
  • Enable keybindings and manual control: a checkbox to attach/detach keybindings. This allows the user to move the sun around with Ctrl + arrow keys and Ctrl + Shift + arrow keys for fast and slow movement respectively
  • Emission Strength: change the brightness of the background with a default strength of 1.

This will currently require the functionality of the scenery atmosphere-shader branch that has not been merged yet.

Edit: the scenery branch has changed to this one

@smlpt smlpt linked an issue Mar 12, 2024 that may be closed by this pull request
2 tasks
@smlpt smlpt marked this pull request as draft March 12, 2024 13:37
@kephale
Copy link
Member

kephale commented Mar 13, 2024

What scenery version should we try (jitpack or whatever)? I tried 9062d7f but that doesn't seem to build on jitpack.

@kephale
Copy link
Member

kephale commented Mar 13, 2024

I got this working with scenery = c4114b0

The inspector UI was working, key controls, etc. It all looks good!

@smlpt
Copy link
Contributor Author

smlpt commented Mar 14, 2024

I got this working with scenery = c4114b0

The inspector UI was working, key controls, etc. It all looks good!

Glad to hear that, thanks!

@smlpt smlpt marked this pull request as ready for review April 18, 2024 14:30
@smlpt smlpt requested review from kephale and skalarproduktraum and removed request for kephale and skalarproduktraum April 19, 2024 09:43
Copy link
Member

@kephale kephale left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is super fun. I made a video that I'll post somewhere using this :D

@smlpt smlpt merged commit c4f9319 into main Apr 21, 2024
4 checks passed
@smlpt smlpt deleted the atmosphere branch April 21, 2024 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add atmosphere background shader from scenery
3 participants