Skip to content

Fixes and improvements to Moon's orbital modeling#276

Merged
attipaci merged 1 commit into
mainfrom
moon-orbit
Nov 18, 2025
Merged

Fixes and improvements to Moon's orbital modeling#276
attipaci merged 1 commit into
mainfrom
moon-orbit

Conversation

@attipaci

@attipaci attipaci commented Nov 18, 2025

Copy link
Copy Markdown
Collaborator

Fix and improve Moon's orbital modeling:

  • novas_make_moon_orbit() had wrong eccentricity, causing errors at the few degrees level.
  • Updated mean orbital elements to ELP2000-85 (Chapront-Touze & Chapron 1988).
  • Now including the principal Solar perturbations for ~10 arcmin accuracy +/- a day or so from orbital reference epoch.
  • Apsis period expressed w.r.t. the ascending node.

@attipaci attipaci added this to the 1.5.1 milestone Nov 18, 2025
@attipaci attipaci self-assigned this Nov 18, 2025
@attipaci attipaci added the enhancement New feature or request label Nov 18, 2025
@codecov

codecov Bot commented Nov 18, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.84%. Comparing base (77e3504) to head (9c0180c).
⚠️ Report is 5 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #276   +/-   ##
=======================================
  Coverage   99.84%   99.84%           
=======================================
  Files          24       24           
  Lines        4611     4622   +11     
  Branches      912      912           
=======================================
+ Hits         4604     4615   +11     
  Partials        7        7           
Flag Coverage Δ
unittests 99.84% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
src/orbital.c 100.00% <100.00%> (ø)
src/planets.c 99.31% <100.00%> (+0.05%) ⬆️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8bf5252...9c0180c. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@attipaci attipaci force-pushed the moon-orbit branch 3 times, most recently from 462e1ad to 2653d94 Compare November 18, 2025 12:48
@attipaci attipaci marked this pull request as ready for review November 18, 2025 12:58
@attipaci attipaci merged commit 671d3b3 into main Nov 18, 2025
27 checks passed
@attipaci attipaci deleted the moon-orbit branch November 18, 2025 16:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant