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

Pluto got listed as a planet #10

Open
apetrov opened this issue Aug 4, 2021 · 1 comment
Open

Pluto got listed as a planet #10

apetrov opened this issue Aug 4, 2021 · 1 comment

Comments

@apetrov
Copy link
Contributor

apetrov commented Aug 4, 2021

Describe the bug
There is Pluto in the planet list

Expected behaviour
Pluto should not be in the list

@smirik
Copy link
Owner

smirik commented Aug 7, 2021

Actually, I have mentioned somewhere in the documentation (or the code) that while Pluto is not a planet anymore, many authors (including Chambers) make their calculations including Pluto. Thus, if one wants to reproduce or compare one's results with their results, it is important to keep the same model. Also, the idea of having a separate list of planets is not to define these planets but to define the list of bodies that have masses and thus affecting the particles.

However, I agree that the name "planets" is ambigous. Hence, it would be a good idea to rename it in something more straightforward.

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

No branches or pull requests

2 participants