You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Describe the bug
The spacecraftLocation class incorrectly calculates the closest vector rClose making it smaller than the celestial body's radius and thus not calculating the rest of the parameters after the if statement.
Right now the if statement is evaluating the closest distance to the bodies center from the "infinite" (parameter not clipped) line that joins the two satellites.
From what I understood of how the function works, the solution should just be to move the calculation of the rClose vector to after the clipping of the parameter value has been done, around line 190 of the file spacecraftLocation.cpp
To reproduce
Accessing the hasAccess values from the recorder of the spacecraftLocation of two satellites with orbital parameters (a, ecc, inc, Omega, omega, f) and with infinite range and without specifying any antenna in either satellite:
Describe the bug
The spacecraftLocation class incorrectly calculates the closest vector rClose making it smaller than the celestial body's radius and thus not calculating the rest of the parameters after the if statement.
Right now the if statement is evaluating the closest distance to the bodies center from the "infinite" (parameter not clipped) line that joins the two satellites.
From what I understood of how the function works, the solution should just be to move the calculation of the rClose vector to after the clipping of the parameter value has been done, around line 190 of the file spacecraftLocation.cpp
To reproduce
Accessing the hasAccess values from the recorder of the spacecraftLocation of two satellites with orbital parameters (a, ecc, inc, Omega, omega, f) and with infinite range and without specifying any antenna in either satellite:
1, 8000, 0.0, 0.0, 0.0, 0.0, 0.0
2, 8020, 0.003, 0.0, 0.0, 0.0, 0.0
Expected behavior
There should be a clear line of sight between them
Screenshots
Desktop (please complete the following information):
Additional context
I have made the fix locally and the simulation now seems to work as expected
The text was updated successfully, but these errors were encountered: