-
Notifications
You must be signed in to change notification settings - Fork 329
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
haddnano.py should we moved to cmssw #13
Comments
why?
it's needed also standalone to merge skims, and it's not needed by
production
(or we tell the standalone "build" to wget it from github?)
Il 21 Set 2017 18:10, "arizzi" <[email protected]> ha scritto:
… haddnano.py should be better distributed with CMSSW
I suggest to move it to
PhysicsTools/NanoAOD/scripts
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#13>, or mute the
thread
<https://github.com/notifications/unsubscribe-auth/AEbbR51oMOgEdMI_59kDp2D5zIRb7nEKks5skop4gaJpZM4Pfj_J>
.
|
the main reason is for crab to work without having to ship the merger with
it... but more in general as we do not expect the tool to evolve with
features there is no reason to keep it out of the main distribution
Il 21 set 2017 7:10 PM, "Giovanni Petrucciani" <[email protected]>
ha scritto:
… why?
it's needed also standalone to merge skims, and it's not needed by
production
(or we tell the standalone "build" to wget it from github?)
Il 21 Set 2017 18:10, "arizzi" ***@***.***> ha scritto:
> haddnano.py should be better distributed with CMSSW
> I suggest to move it to
> PhysicsTools/NanoAOD/scripts
>
> —
> You are receiving this because you are subscribed to this thread.
> Reply to this email directly, view it on GitHub
> <#13>, or mute the
> thread
> <https://github.com/notifications/unsubscribe-auth/AEbbR51oMOgEdMI_
59kDp2D5zIRb7nEKks5skop4gaJpZM4Pfj_J>
> .
>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#13 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AEyilu2mayv9t859UKOzdowk6L7YaQJoks5skpgygaJpZM4Pfj_J>
.
|
Hi, is there any intention to do this, or was the decision made to keep haddnano.py in this repository only? |
there are pro and cons with both options... right now we decided to keep it only here, but if you have good arguments to go for central cmssw we can change that. |
I don't have any different arguments that what has already been said, I was just following the crab example and noticing the comment about how the script would in CMSSW at some point. I have not strong need for it to be there, just curious, and I suppose it has as much right to be there as hadd_heppy.py... |
Is there a ROOT JIRA issue about having native |
since this come over and over I made the PR in master and will backport in 10_6 |
Merge cms-nanoAOD tools master
haddnano.py should be better distributed with CMSSW
I suggest to move it to
PhysicsTools/NanoAOD/scripts
The text was updated successfully, but these errors were encountered: