fermi-lat / Likelihood

BSD 3-Clause "New" or "Revised" License
2 stars 1 forks source link

gtsrcmaps/gtmodel does not understand projections properly #71

Closed eacharles closed 4 years ago

eacharles commented 4 years ago

https://jira.slac.stanford.edu/projects/LK/issues/LK-105

eacharles commented 4 years ago

Ok, I think this was caused by the class astro/SkyProj not copying the crota keyword. I'm put in a PR that I think fixes this. The results fits files had the correct keywords set. Still need to verify that the rotation are in fact correct.

donhorner commented 4 years ago

Jean Ballet reports (using the Fermitools beta 1.3.0): "looks good (I tested it on gtmodel, not gtsrcmaps, but I am pretty sure they go together). The model map is now correctly aligned with the counts map even when CROTA is not 0."

Closing the issue.