-
Notifications
You must be signed in to change notification settings - Fork 146
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CMS: SW records for ML sample production (Run2 QCD for jettuples) #2584
Comments
* (closes cernopendata#2584) Signed-off-by: Artemis Lavasa <artemis.lavasa@cern.ch>
@katilp The new github repository in cms-opendata-analyses/JetNtupleProducerTool is up now and I licensed it under GPL 3.0. If I provide the example Python scripts for loading the ntuples, where should I host them? Or should they be written directly in the record? |
@kimmokal @caredg I see different options:
in case it goes to a separate repo, I would be in favour of a new organization in a similar way as for cms-opendata-education. That would follow the logic:
|
I see no problem in creating another organization if needed. If it is a test of the JeNtupleProducerTool, then maybe it could go into another package within the same repository as well. In any case, let me know and I can create the additional organization if needed. BTW, Once everything is in place in the cms-opendata-analysis organization, the JeNtupleProducerTool should be deleted from cms-legacydata-analysis in order not to duplicate efforts. |
@katilp I reckon the examples are rather short, so I don't think it makes sense in this case to create a separate repository. I would provide the following two very similar code snippets. Loading a ROOT file to a Pandas dataframe:
Loading an HDF5 file to a Pandas dataframe:
I guess they could just go directly in the record. @caredg I verified that the cms-opendata-analysis/JeNtupleProducerTool repository works as intended, so should I now just delete the repository in cms-legacydata-analysis? |
@kimmokal, yes I think that would be good. |
@caredg Done! |
* (closes cernopendata#2584) Signed-off-by: Artemis Lavasa <artemis.lavasa@cern.ch> Signed-off-by: Tibor Simko <tibor.simko@cern.ch>
* (closes cernopendata#2584) Signed-off-by: Artemis Lavasa <artemis.lavasa@cern.ch> Signed-off-by: Tibor Simko <tibor.simko@cern.ch>
From #2447
The structure can be as in http://opendata-dev.cern.ch/record/5104
@caredg @laramaktub This is what we would need to extract from a github repo or CAP in order to build a SW record on the portal
Needed:
https://github.com/cms-legacydata-analyses/JetNtupleProducerTool/tree/2016
but it will be a placholder as it will be moved to https://github.com/cms-opendata-analyses/JetNtupleProducerTool/tree/2016 and have a release tag there
@ArtemisLav @tiborsimko :
json could be:
The text was updated successfully, but these errors were encountered: