-
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
"Class versions" check failed in CMSSW_15_0_X_2025-01-23-1100 #47177
Comments
cms-bot internal usage |
A new Issue was created by @iarspider. @Dr15Jones, @antoniovilela, @makortel, @mandrenguyen, @rappoccio, @sextonkennedy, @smuzaffar can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
assign core |
New categories assigned: core @Dr15Jones,@makortel,@smuzaffar you have been requested to review this Pull request/Issue and eventually sign? Thanks |
@smuzaffar what are the commands used to run the class versions check? |
If I dump the actual source of page, then the actual line is
the problem is the page is being interpreted as html by the browser so it 'hides' the stuff in |
The whole cmssw/CondFormats/L1TObjects/src/classes.h Lines 74 to 135 in 75451d5
could be removed as unnecessary since many years, but I suspect that has nothing to do with the failure itself. |
Done in #47181 |
For future reference (since logs will disappear in ~2 weeks) here are (hopefully) relevant parts of the log
|
Given the stack trace above, the call to cmssw/FWCore/Reflection/python/ClassesDefXmlUtils.py Lines 121 to 122 in 9952e64
called from
|
@Dr15Jones , the command we run is
e.g. for
|
this is fixed by cms-sw/cms-bot@b5dd77d |
When I run the I get the same error messages with the same dump, but I also get additional error messages
|
correction, edmDumpClassVersion returns just fine (exit code 0) but does print the errors above. The program does write a out.json file containing information, including information about the classes it complained about (although those classes are missing the class version information). |
Yeah, they would need a careful look (e.g. in the first case the
should be just removed as redundant because
and
already exists ) |
The check was marked as failed in CMSSW_15_0_X_2025-01-23-1100, but not in -2300 (but it's been failing for a long time). Logs: 1100, 2300
I think the class dictionary isn't generated properly:
std::pair > dummy13;
@makortel FYI
The text was updated successfully, but these errors were encountered: