madminer-tool / madminer-workflow

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

Define sub-modules interactions #31

Closed Sinclert closed 4 years ago

Sinclert commented 4 years ago

This PR updates the Makefile, README and reana/workflow.yml with the necessary changes after the introduction of GIT sub-modules to manage sub-workflow repositories: madminer-workflow-ph and madminer-workflow-ml.


~In addition, it addresses issue #30 by introducing template render Makefile rule for the ML sub-workflow, allowing it to both:~

~Update:~

~In addition, it addresses issue #30 by moving the last step of the Physics sub-workflow (combine) into this repository workflow.yml, so that it can be referenced from the ML sub-workflow. This change allows both:~

Update 2:

In addition, it addresses issue #30 by concatenating the Physics and ML sub-workflows directly on this repo workflow.yml. This change allows both:

irinaespejo commented 4 years ago

Thank you! Is it now ready for merge?

Sinclert commented 4 years ago

Yes, it is 🚀