search

foxsi / foxsi-smex

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

`foxsi_get_instrument_background.pro` to obtain the estimate of instrument background #56

Closed ayshih closed 6 years ago

ayshih commented 8 years ago

Added a function (only on the IDL side) to obtain the estimate of instrument background (in units of counts/s/keV) for the entire instrument (i.e., both telescopes). The optional keyword in_hpd provides a convenient scaling of the background to only the HPD (assuming uniform background) for the purposes of point sources.

LinErinG commented 7 years ago

@ayshih, the background function has been working great for me; sorry I didn't realize this PR hadn't been merged yet. Just so I can use the function in the laziest possible way, can you adjust it so that it returns the energy array as well if one is not input? Thanks.

ayshih commented 7 years ago

@LinErinG I think I made the change as you requested

LinErinG commented 7 years ago

@ayshih This works great for my purposes but I suggest you edit the header to make it clear that energy_arr can be either an input or an output.

ayshih commented 6 years ago

I've finally updated this PR for the CSR, which removes the extra factor of 2 that I threw in at the last minute for the Step-1 proposal. I also addressed @LinErinG's suggestion. Review and merge!

LinErinG commented 6 years ago

Ok by me