Skip to content

Port of dnwrnr fixes - #42

Merged
parzivail merged 7 commits into
parzivail:masterfrom
jnyrup:dnwrnr_pr_fixes
Aug 7, 2026
Merged

Port of dnwrnr fixes#42
parzivail merged 7 commits into
parzivail:masterfrom
jnyrup:dnwrnr_pr_fixes

Conversation

@jnyrup

@jnyrup jnyrup commented Aug 7, 2026

Copy link
Copy Markdown
Contributor

Port of dnwrnr/sgp4#7

Disclosure: I had an AI help me do the porting.

jnyrup added 7 commits August 7, 2026 18:30
Ports dnwrnr/sgp4@2462be1 "Add a new constant".

`A3Ovk2` is derived purely from compile-time constants, so there is no
reason to recompute it per instance and store it in `CommonConstants`.
Ports dnwrnr/sgp4@adc1c36 "Create function to compute perturbed values".
Ports dnwrnr/sgp4@836b67e "Use WrapTwoPI. Formatting and wrapping.".
Ports the periodics half of dnwrnr/sgp4@6982421 "Improve DeepSpacePeriodics.".
Ports the `theta` part of dnwrnr/sgp4@6982421 "Improve DeepSpacePeriodics.".
Ports the structural half of dnwrnr/sgp4@6982421 "Improve DeepSpacePeriodics.".
Ports dnwrnr/sgp4@a7d72d7 "Rename xincl to xinc. xincl is used to
represent the original inclination.".
@parzivail

Copy link
Copy Markdown
Owner

No problem. Ultimately, it's a tool like any other, and the user is as responsible for their PR as ever.

@parzivail

Copy link
Copy Markdown
Owner

And, thanks for porting these from the "upstream"!

@parzivail
parzivail merged commit 01c5b86 into parzivail:master Aug 7, 2026
1 check passed
@jnyrup
jnyrup deleted the dnwrnr_pr_fixes branch August 7, 2026 17:58
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 this pull request may close these issues.

2 participants