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

Crash when creating a save with RealSolarSystem on Buffon #905

Closed
eggrobin opened this issue Feb 23, 2016 · 2 comments

Comments

@eggrobin
Copy link
Member

commented Feb 23, 2016

@eggrobin eggrobin added the bug label Feb 23, 2016

@eggrobin

This comment has been minimized.

Copy link
Member Author

commented Feb 23, 2016

Since there's no Inserting <body name>..., this is probably when inserting the Sun.

@eggrobin eggrobin added this to the Cantor milestone Feb 23, 2016

@eggrobin

This comment has been minimized.

Copy link
Member Author

commented Feb 23, 2016

Indeed, we're called with

gravitational_parameter: 1.3271244004193938E+11 km^3/s^2
mean_radius: 286.13 deg
axis_right_ascension: 63.87 deg
axis_declination: 2.1106088532726840E-07
j2: 696000.0 km
reference_radius: +1.309126697236264E+05 km
x: +3.443856610385113E+05 km
y: +1.364602296561306E+05 km
z: -7.799754996220354E-03 km/s
vx: -5.561927893069310E-03 km/s
vy: -2.253148239533338E-03 km/s
vz: U8bìhô(8a'UVWS耽84è83Pÿ089 83ì83ì♀ÿu¶è)Æÿÿ83Ä►8bø83ì♀ÿu↑è↓Æÿÿ83Ä►8bð83ì♀ÿu∟è     Æÿÿ83Ä►8b؃ì♀ÿu èùÅÿÿ83Ä►89E؃ì♀ÿu$èèÅÿÿ83Ä►89Eԃì♀ÿ85§4~♥83Ä@8b♣hI▼►85ÀÔÿuØSVWÿu►ÿu♀ÿuÿu0èµÅÿÿ83Ä►89Eȃì♀ÿu4è¤Åÿÿ83Ä►89Eăì♀ÿu8è93Åÿÿ83Ä►89EÀ83ì♀ÿu<è82Åÿÿ83Ä►89E¼83ì

It is easy to see what's going on here: the handwritten part of the C# side of the interface doesn't match the C++ side, the signatures match enough for P/Invoke, and we pass shifted arguments (missing the mean radius), and read garbage in vz.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant
You can’t perform that action at this time.