madminer-tool / madminer-workflow

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

Split workflow into different repos #26

Closed Sinclert closed 4 years ago

Sinclert commented 4 years ago

This repository is currently serving 2 different purposes, and may soon serve a different one:

Proposal

We could create 3 new repositories:

And leave this repository as a convergence point where madminer-workflow-ml and madminer-workflow-ph are defined as GIT sub-modules (a way of referencing a GIT repository from within a GIT repository).

Example .gitmodules files:

[submodule "madminer-workflow-ml"]
    path = madminer-workflow-ml
    url = git@github.com:scailfin/madminer-workflow-ml.git
    branch = master
[submodule "madminer-workflow-ph"]
    path = madminer-workflow-ph
    url = git@github.com:scailfin/madminer-workflow-ph.git
    branch = master

It would also containing the reana folder with the joint workflow specification.


If Scailfin is not the place to create those repositories, then maybe DIANA-HEP (where madminer is hosted), or a brand new organization are better options.

irinaespejo commented 4 years ago

Hi @Sinclert, I think it's a great suggestion. The project is growing and the current repo will be too crowded. I tried using git sub-modules to host a separate repo with the docker images but I didn't get it to work properly. With respect to Scailfin vs DIANA-HEP maybe on the #madminer-on-reana channel they have a better picture on where the repos should go. Thank you for this idea!