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
In the fermi module of sunkit-instruments, the code get_detector_sun_angles_for_time returns the angle with respect to the Sun of each Fermi/GBM detector. This is done by reading in a Fermi spacecraft weekly file.
However, these files contain gaps due to the South Atlantic Anomaly. If the time requested falls in one of these gaps, the code will return the detector angles for the next available time. This can be several minutes different from the time requested. Currently, the code does not provide any indication that this has happened.
#128 addresses this in the short term by adding a warning. However, there are two other potential fixes for this issue:
interpolate the detector angles over the time gap, and warn the user that this has happened.
switch to reading the spacecraft location information from GBM poshist files, instead of LAT spacecraft weekly files. GBM poshist files do not contain the data gaps. This would require a more substantial rewrite of the code.
Describe the bug
In the fermi module of sunkit-instruments, the code get_detector_sun_angles_for_time returns the angle with respect to the Sun of each Fermi/GBM detector. This is done by reading in a Fermi spacecraft weekly file.
However, these files contain gaps due to the South Atlantic Anomaly. If the time requested falls in one of these gaps, the code will return the detector angles for the next available time. This can be several minutes different from the time requested. Currently, the code does not provide any indication that this has happened.
#128 addresses this in the short term by adding a warning. However, there are two other potential fixes for this issue:
To Reproduce
Note that detector angles are returned for 06:07 UT, when the requested time was 05:58 UT.
Screenshots
No response
System Details
No response
Installation method
conda
The text was updated successfully, but these errors were encountered: