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

Nazfib/check maths #8

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Nazfib/check maths #8

wants to merge 3 commits into from

Conversation

Nazfib
Copy link

@Nazfib Nazfib commented Dec 11, 2023

Fixes a few small issues with the calculations. In particular, this makes it work from the southern hemisphere.

Fixes #7
Maybe fixes #6 ?

Use the msbuild copy target instead of xcopy; change file encoding to
utf-8.
- Use double precision quaternion
- Make sure the orbit normal always points roughly northwards
- The 'East' vector used to point to the west;
- Replace latitude with abs(latitude) for checking against inclinations

Fixes #7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
1 participant