TCDSolar / stixpy

STIX data analysis in python
https://stixpy.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
17 stars 20 forks source link

Compressed pixel data plotted in units of cts/cs/keV with plot_timeseries() #129

Closed hannahc243 closed 2 months ago

hannahc243 commented 3 months ago

Compressed pixel data is currently plotted in units of cts/cs/keV using the plot_timeseries() method, whereas quicklook data is in units of cts/s/keV. It might be more straightforward to have them in the same units or to have the option to plot in different units (e.g. cts/s/keV) using a kwarg. Is there a particular reason for plotting in cts/cs/keV currently?

samaloney commented 3 months ago

Thanks for raising the issue essentially cs is the native time unit in the file and currently don't do any conversion. It should be pretty straightforward to use .to(u.s) somewhere to change back to seconds.