pisa

pisa copied to clipboard

Safer and more expressive expression evaluation

2

E.g. in `DataProcParams` (fields specified in `data_proc_params.json` and interpreted by `evaluateExpr`) and `Events` (`applyCut` method) classes could benefit from more generic and safe evaluation of the contents. Note that it...

jllanfranchi

Stage/service to define different binning for analysis

3

Additional stage that allows the user to define analysis binning that is possibly different than reco binning and possibly different binnings for each PID category. Discussion from PISA call on...

jllanfranchi

pisa name exists in PyPI

3

Can cause confusion, installation issues; the description certainly isn't confusing, but how to proceed may be. Can we come up with a different package name that gets referenced in the...

jllanfranchi

Physics unit tests for all stages

1

pull request #51 introduced physics unit tests to verify basic / hist services for each stage implemented in PISA 2 as well matched output of PISA 2's stages. We need...

jllanfranchi

unit tests: test that each stage works with arbitary binning

1

Binning orders I can think of that the user might specify (some are ok if they don't work in certain stages, if there's a good reason why that binning choice...

jllanfranchi

add GCD file name and md5sum to mc_sim_run_settings.json

1

see also issue #87

jllanfranchi

pipeline.py: create dummy inputs if pipeline begins with an input stage?

3

Normally, one would run flux -> osc -> etc. However, it's possible one would want to test osc -> etc. As of now, "dummy" inputs are only generated in the...

jllanfranchi

Merge 2d histogram plotting codes

1

Plotting itself--at least for maps, if not other things--should be handled by `utils/plotter.py`. If that doesn't do something it should, then that functionality should be added to `plotter.py` and not...

jllanfranchi

PID stage should output maps that have PID categories as binning dims

2

Currently, PID stage outputs multiple maps, one for each PID category. This is probably a bug since `DistributionMaker.get_outputs(return_sum=True)` I expect will simply collapse all the maps into one, losing any...

jllanfranchi

make_asymmetry_plot needs generalising

2

List of things needed: - Currently only handles 5-stage PISA (`pid` separate from `reco`) and event-by-event with the `combine` stage properly. This should be fixed. - Currently assumes two PID...

steven-j-wren