cosmicrays / hermes

HERMES is a publicly available computational framework for the line of sight integration over galactic radiative processes which creates sky maps in the HEALPix-compatibile format.
GNU General Public License v3.0
22 stars 9 forks source link

How to implement a radial dependent XCO? #4

Closed carmeloevoli closed 2 years ago

carmeloevoli commented 3 years ago

https://github.com/cosmicrays/hermes/blob/1587c84bf0f4620852f9c3bc5b3165360202c003/src/neutralgas/RingModel.cpp#L97

adundovi commented 3 years ago

It should dependent on r, not dir, right?

carmeloevoli commented 3 years ago

yes, but I think it would be better if implemented in post-processing. I'm still thinking which should be the most efficient way to do that.

carmeloevoli commented 3 years ago

I would say that to implement this efficiently we should provide the user with two functions:

That should be easy to implement. What do you think?

carmeloevoli commented 2 years ago

Implemented in commit https://github.com/cosmicrays/hermes/commit/b40dae2b749e7d9cb8c32ccd91ef6738c91f97fa