cositools / cosipy

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

Spacecraft file tutorial #149

Closed israelmcmc closed 3 months ago

israelmcmc commented 4 months ago

Closed #147

I'll track my changes here to coordinate with @Yong2Sheng

israelmcmc commented 3 months ago

@Yong2Sheng Can you please check and merge this PR? In particular, that both tutorials work, that I didn't remove any information you wanted, and that SpacecraftFile.py still works well.

Yong2Sheng commented 3 months ago

Hi @israelmcmc,

I reviewed the PR, it looks great! I really like your algorithm to get the summed pixel durations.

I tested DetectorResponse.ipynb and SpacecraftFile.ipynb. They both ran very well.

I only have one minor comment on one line I posted above.

Thank you very much!

israelmcmc commented 3 months ago

Note: all commits from PR #150 were merged here.

israelmcmc commented 3 months ago

Thanks @Yong2Sheng, I'll merge it