xpsi-group / xpsi

X-PSI: X-ray Pulse Simulation and Inference
Other
34 stars 21 forks source link

Deprecate/remove `TwoHotRegions`? #385

Open DevarshiChoudhury opened 3 months ago

DevarshiChoudhury commented 3 months ago

Currently the Likelihood API has separate documentation for TwoHotRegions. I feel that no one ever uses this module, and I don't really see a point in using it either since HotRegions can take in multiple HotRegion objects. I am in favour of removing/deprecating this from the Likelihood API, as well as maybe the actual codebase.