StingraySoftware / stingray

Anything can happen in the next half hour (including spectral timing made easy)!
https://stingray.science/stingray
MIT License
174 stars 144 forks source link

Fix header #853

Closed matteobachetti closed 1 month ago

matteobachetti commented 1 month ago

When defining the new FITSTimeseriesREader, I mistakenly changed the format of the header attribute of the output time series (from string to dictionary). This broke upstream code (HENDRICS, at least). I'm now fixing the issue.

Also, RXTE files sometimes do not contain the mission information in HDU0: we need to be robust to that (hence the cycling of hdus until MISSION or TELESCOP are found).

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 95.98%. Comparing base (28f2479) to head (4802257). Report is 6 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #853 +/- ## ======================================= Coverage 95.98% 95.98% ======================================= Files 48 48 Lines 9680 9686 +6 ======================================= + Hits 9291 9297 +6 Misses 389 389 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.