-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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
Heppy 7 4 4 patch4 #9544
Heppy 7 4 4 patch4 #9544
Conversation
Thanks Andrea, I'm merging only now cause I wanted to wait for the branch to be merged in CMSSW.
…e with 72X non-pf b-tagging
…ghts to add this information
…e previous commit (synchro issue)
…s called, what seems needed for the moment for the isValid() method to work properluy
Thanks guys!
…s.path and not scripts/
I think it may be this one, but @Degano can confirm: What I see is that ipython 3.1.0 is in the 75X IB. |
Perfect - thanks
|
As said in the cmsdist PR, Alessandro provided a testing recipe, and I could test that in the most recent IB:
|
+tested |
hold |
Thanks for this improvement Joosep!
Hello,
Here is a new version of heppy for CMSSW_7_4_X. This version contains important bug fixes, and is compatible with ipython 3.1. I am going to request ipython 3.1 for 74X in cmsdist now and link the two pull requests together.
It would be great if these two requests could enter CMSSW at the same time.
@arizzi @gpetruc : we do a PR later with new changes from you guys anytime you want after the migration to ipython 3.1. Here, I'm only fixing critical problems.
@Degano : same stuff as what we discussed for 75X.
Colin