Update MC scripts to allow for switching between XENONnT and XENON1T

[jpienaar13]

Added cases into the two macros to allow for switching between XENON1T and XENOnT geometries. At the moment all this should do is select the appropriate macro directory in the MC code, and exit the production chain after the G4 stage.

Will continue to propogate as other parts of the code get upgraded. Should be merged with https://github.com/XENON1T/mc/pull/103. (Which is ready to merge as soon as this is approved.)

[sthapa]

Hi Jacques,

Your changes look good overall. I did notice a few things:

• on line 423, I believe you should have get_configs(args_exp.experiment)
• after you run parse_args, you should be able to get parameters using args_exp.experiment or args.num_events without the index, did you find that you need the indexing?
  • (Optional but recommended) I'd recommend adding a :param explanation in the docstring when adding a new parameter (e.g. get_configs, check_macro). Otherwise it's easy for changes to build up and result in the code doing something totally different from what the documentation says.
  • (Optional) on line 263, using something like if experiment not in ["XENON1T", "XENONnT"]: would be more idiomatic python and is a bit more readable.
  • (Optional) Finally, this is super minor but on line 413, it should be Choose not Chose.

[jpienaar13]

Hi @sthapa

For points 1 and 2 in combination:

• I found the indexing is needed, as the function has been changed from parse_args, to parse_known_args for both parsers. Parse_known_args returns a two item tuple, the first of which is the namespace, and the second is the list of remaining unused argument strings.

3.) Implemented

4.) Changed

5.) Fixed

[jpienaar13]

Oh wait, didnt fix your first point correctly

[sthapa]

Ah, I didn't catch the change to use parse_known_args. The indexing now makes sense.