-
Notifications
You must be signed in to change notification settings - Fork 265
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes to simulation containers #2335
Comments
No, please not again anything with 'height' as that still implies the wrong units. |
good point @kbernloehr : let's at least pick a standard name for atmospheric depth units. What is the best term for a unit compatible with g/cm^2? Is it "depth" (also confusing as can be in distance units), "slant depth", "grammage", just "X", "column density"? |
The CORSIKA User's Guide calls it 'starting grammage' and 'mass overburden' (of the starting altitude). The terms 'slant depth' and 'column density' seem to imply something along the direction of the primary particle. Same for 'X0', given that 'Xmax' is counted along the incoming primaries path. 'Grammage' may be ambiguous as well and the fact that Wikipedia knows it as a term from the paper industry does not help much either. 'Overburden' sounds about right, making it clearer that it counts what is (vertically) above. (For Wikipedia it is mainly a term from the mining industry but it also mentions that 'the overburden of an underground laboratory may be important to shield the facility from cosmic radiation'. Oh well, those underground guys count it in meters of water equivalent but at least it is in the right direction.) |
I'd choose |
Please describe the use case that requires this feature.
In a discussion with @kbernloehr in the gitlab, it came up that the
injection_height
in themcrunheader
object, which we store in theSimulationConfigContainer
is not used by simtel array and exists only for historical reasons, it is always -1.The real injection height, is stored for each shower in the
MCShower
object, but currently we don't store this.Describe the solution you'd like
Remove
injection_height
from theSimulationConfigContainer
(or at least set it to nan or None inSimtelEventSource
) and addinjection_height
to theSimulatedShowerContainer
.The text was updated successfully, but these errors were encountered: