cositools / cosipy

The COSI high-level data analysis tools
Apache License 2.0
3 stars 16 forks source link

Add method to generate pre-computed all-sky point source response #129

Open israelmcmc opened 5 months ago

israelmcmc commented 5 months ago

Related comment from @ckarwin on slack https://ucbssl.slack.com/archives/C02TWC7U4NL/p1706213731961159?thread_ts=1700492800.163339&cid=C02TWC7U4NL

Hi Israel, I'm converting the Ti44 response to the point source response using the same code we used for 511 (and Al26). I was running into errors until I remembered that I needed to add "return psr" before converting it to a tuple, as per your comment above. This was lost when I updated my cosipy version. Can you make this an option eventually? Maybe something like: if make_tuple==False: return psr? Or maybe you have in mind a better way to implement this? It's not urgent. I modified the code for now. But I wanted to mention this while it's fresh in my mind. I can also open an issue for the record.