foxsi / foxsi-smex

Tools for scientists to investigate the capabilities of FOXSI
MIT License
2 stars 8 forks source link

Updates to rate warning and more #51

Closed LinErinG closed 8 years ago

LinErinG commented 8 years ago

Four changes, all fairly innocuous:

-- Added more information to rate warnings. The per-pixel and the whole detector alarms are now separate, and the warning message states the threshold levels and the actual levels.

-- Fixed the bug Albert found for running the imaging/spectral response on a one-element input map.

-- Added a "DT" keyword to foxsi_get_output_image_cube. This way you can integrate a source over an interval DT of time in order to get better statistics, but the rate warning is still useful.

-- Changed the default source spectral normalization to something more useful. Now running foxsi_get_output_image_cube with the default will invoke a rate warning, but one can get rid of that warning by inserting a thick attenuator.

Note that the rate warnings are still treating all the counts on one detector, so we're probably a factor of 2 off. I can change this, but I need a bit more info on how we're treating the two telescopes differently (i.e. thermal/nonthermal).

LinErinG commented 8 years ago

This PR addresses Issues #50 and #49

ayshih commented 8 years ago

Does what it says on the tin :+1: