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

answer to question about orbital parameter bounds checks in astronomy.F90 comment #1627

Open
japhir opened this issue Dec 13, 2024 · 0 comments
Labels
enhancement Issue/PR for a modification that increases performance, improves syntax, or adds functionality.

Comments

@japhir
Copy link

japhir commented Dec 13, 2024

to answer the question in the comment in astronomy.F90
QUESTION : ARE THESE THE RIGHT LIMITS ???

I am not sure if the limits should be based on the physically possible values, or on the values for Earth specifically.

Earth's orbital eccentricity has varied between 0 and 0.067 for the past 100 Myr, so maybe a test more like 0 to 0.1 would be more suited?

Earth's obliquity has varied from 22° to 24.5° in the same time period, so a better range to check would be something like 10° to 40°? A negative value would not make sense, I think. It should be between 0° and 180°.

Earth's longitude of perihelion with respect to the moving equinox rotates and librates, so it does vary between 0° and 360°.

I could implement these changes in this pull request if desired.

These are the equivalent lines in shr_orb_mod.F90:
https://github.com/ESCOMP/CDEPS/blob/8197f05ef2549d3c53e4050c9821a683e2728bab/share/shr_orb_mod.F90#L27-L33

Describe the solution you'd like
Change the testing limits. Either to the theoretically valid ranges (i.e. change obliquity to 0 to 180) or to narrower ranges for Earth's orbital history.

Describe alternatives you've considered
Don't change it because the current wider ranges encompass the true ranges.

Additional context
I created this unrelated pull request that describes the issue there, before noticing that that was a fork of this repository.

ESCOMP#4

@japhir japhir added the enhancement Issue/PR for a modification that increases performance, improves syntax, or adds functionality. label Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Issue/PR for a modification that increases performance, improves syntax, or adds functionality.
Projects
None yet
Development

No branches or pull requests

1 participant