Removed the lines where pointing variables was trying to access ra and dec members.
Closes issue #2 (closed).
Merge request reports
Activity
ah sorry, it was not clear from the merge request that it is related to this issue: #2 (closed)
There I explain the reason why those lines should be removed.
Hi Alessio, indeed, ra & dec are not present in default ctapipe, but the containers should exist in the ctapipe-mpp master. They don't & it still doesn't work for you? Anyhow, can still discuss whether we want at all to incorporate into ctapipe reading ra,dec from the MARS files (i.e., the MARS internal alt,az->ra,dec calculation result). I did it once only to check against the Astropy ra,dec calculation. In fact, I could quite well reproduce the MARS ra,dec output, but only after considering the atmospheric conditions in the Astropy calculation (see https://github.com/cta-observatory/ctapipe/pull/1210; still, MARS is using a slightly different standard atmospheric model, iirc).
mentioned in issue #2 (closed)