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

Rings as Eclipse Shadow Receiver #281

Closed
JonasGilg opened this issue Jun 1, 2022 · 4 comments
Closed

Rings as Eclipse Shadow Receiver #281

JonasGilg opened this issue Jun 1, 2022 · 4 comments
Labels
csp-rings good first issue Good for newcomers new feature A feature request or a general improvement idea

Comments

@JonasGilg
Copy link
Member

It would be nice, if rings also receive eclipse shadows.

@JonasGilg JonasGilg added new feature A feature request or a general improvement idea good first issue Good for newcomers csp-rings labels Jun 1, 2022
@JonasGilg
Copy link
Member Author

I tried implementing this feature, but sadly it is not as trivial as I thought, since there is a check for self shadowing, that just compares the center names here: https://github.com/cosmoscout/cosmoscout-vr/blob/develop/src/cs-core/SolarSystem.cpp#L101

Since the Ring and the planet share the same center the shadow isn't casted onto the ring.

For fun I disabled this check and this came out of it:
image

@Schneegans
Copy link
Member

This doesn't look too bad after all :)

I think I have an idea how this could be fixed. Did you push your changes already?

@JonasGilg
Copy link
Member Author

I think I have an idea how this could be fixed. Did you push your changes already?

Yes, they are on this branch: https://github.com/cosmoscout/cosmoscout-vr/tree/feature/ring-eclipses

Sadly I accidentally branched from the more-simple-bodies branch, so we shouldn't merge this before the other branch is merged.

@Schneegans
Copy link
Member

I think we can close this now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
csp-rings good first issue Good for newcomers new feature A feature request or a general improvement idea
Projects
None yet
Development

No branches or pull requests

2 participants