-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Read errors in opening ROOT 6.30 files in earlier ROOT versions #43882
Comments
assign core |
type root |
New categories assigned: core @Dr15Jones,@makortel,@smuzaffar you have been requested to review this Pull request/Issue and eventually sign? Thanks |
cms-bot internal usage |
A new Issue was created by @makortel Matti Kortelainen. @antoniovilela, @Dr15Jones, @smuzaffar, @sextonkennedy, @makortel, @rappoccio can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
For the purpose of potentially running the HLT step in a future Run 3 Ultra Legacy MC production in a similar way as in Run 2 Ultra Legacy production, we'd need the ROOT fix backported down to 12_4_X, which used ROOT 6.24. If the "reading newer files with older ROOTs" would be needed for something else, we might have backport to even earlier releases. FYI @cms-sw/upgrade-l2 |
Thanks @makortel for following up on this. I see on the ROOT PR that you ask if there will be a backport down to release older than 6.28. |
@smuzaffar According to Philippe the fix has been backported down to 6.14, but not tagged yet in all release series. Do you want to wait for the tags, or could we update ROOT already now in all release cycles down to CMSSW_12_4_X? |
root-project/root@5858905...v6-26-00-patches are the root 6.26-patches branch (used in CMSSW 13 release cycle) changes |
On a quick look the list of commits looked like there were some fixes that sounded useful, and some that could warrant more thorough testing for a "closed" release cycle. I'd be fine with just cherry-picking the fix. One commit that caught my eye was from root-project/root#10641 that you had asked to be backported to 6.24 root-project/root#10576 (comment) :) |
+core |
This issue is fully signed and ready to be closed. |
It turned out that files written in ROOT 6.30 (or later) are not readable in earlier ROOT versions "out of the box". The problem has been fixed for ROOT 6.28 in root-project/root#12845 . The error is along
This issue is for us to track what ROOT fixes we need to backport to which CMSSW release cycles.
The problem was observed e.g. in #43827 (comment)
The text was updated successfully, but these errors were encountered: