madminer-tool / madminer-workflow

Madminer complete cloud-based analysis
MIT License
4 stars 4 forks source link

Fix Physics workflow steps #15

Closed Sinclert closed 4 years ago

Sinclert commented 4 years ago

This PR makes the whole Physics workflow locally testable.

Last PR (https://github.com/scailfin/workflow-madminer/pull/14) stablished a solid foundation to move the shell commands from the steps.yml file to proper .sh files (more easily runnable), but the whole Physics workflow hadn't been completely run due to a Pythia8 never ending step.

Turns out the Pythia8 step is not that computationally expensive. The long delays were due to a misconfiguration on the pythia8_card.dat, were the option useWeightIDs was activated, making Pythia to loop infinitely. This option was, at some point, disabled on the real MadMiner repository (check this commit), but never on the workflow version (this repo).


Additionally, this PR performs includes: