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

Need to fix obs:gotolonglat error #3

Closed
CelestiaProject opened this issue Feb 2, 2017 · 2 comments
Closed

Need to fix obs:gotolonglat error #3

CelestiaProject opened this issue Feb 2, 2017 · 2 comments
Assignees
Labels
bug Something isn't working

Comments

@CelestiaProject
Copy link
Collaborator

CelestiaProject commented Feb 2, 2017

In Celestia 1.7.0 source code (even in the 5229 revision from SF), the error occurred: In CELX scripts where used obs:gotolonglat command, not processed distance parameter. The reason for this is unknown to me, but I think that error associated with porting the source code on a new mathematical engine Eigen.

@CelestiaProject CelestiaProject added the bug Something isn't working label Feb 2, 2017
@Alexell
Copy link
Contributor

Alexell commented Feb 4, 2017

Write in celx: obs:gotolonglat(Earth, math.rad(0), math.rad(30), 3.5004702E14, 5)
Celestia must go to Earth at a distance 3.5004702E14, but Celestia go to Earth center!
File: celx_observer.cpp
String 363: distance = distance / KM_PER_LY;
If comment this string, the Celestia go to Earth the correct distance.

@Alexell
Copy link
Contributor

Alexell commented Feb 4, 2017

c81bd5f

@Alexell Alexell closed this as completed Feb 4, 2017
SevenSpheres pushed a commit to SevenSpheres/Celestia that referenced this issue Apr 9, 2020
To be more up-to-date with the official code
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant