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

Possible issue with tracking objects in local horizon mode #268

Closed
Carifio24 opened this issue Sep 8, 2023 · 2 comments · Fixed by #270
Closed

Possible issue with tracking objects in local horizon mode #268

Carifio24 opened this issue Sep 8, 2023 · 2 comments · Fixed by #270

Comments

@Carifio24
Copy link
Member

I'm trying to make a sky mode view that follows the Sun while using local horizon mode. I've got the Sun set as the tracking object, but it seems that the tracking only works when I turn local horizon mode off.

After digging around in the code, I think that the issue is that, in local horizon mode, the actual Alt/Az values for the render context aren't updated (only the target values are). It seems like the two assignments here should look more like the ones in the preceding block here. With this change, my local-horizon-mode view does indeed stay centered on the Sun as time passes.

@pkgw does this make sense to you? Or is there something that I'm missing here?

@pkgw
Copy link
Contributor

pkgw commented Sep 14, 2023

New code references, after the ESM/JS port:

I agree that it seems like the local horizon mode should do the same thing as the galactic mode ... and/or, I see no reason why it shouldn't! I wouldn't be surprised if an oversight was discovered in the galactic mode, but the fix wasn't propagated to local horizon mode.

@Carifio24
Copy link
Member Author

I wouldn't be surprised if an oversight was discovered in the galactic mode, but the fix wasn't propagated to local horizon mode.

Yeah, I agree that's the most likely reason for the discrepancy. Since the JS port has been merged, I can open a PR to make the suggested change.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants