-
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
Assertion failure in edm::service::MessageLogger::unEstablishModule() #34520
Comments
assign core |
New categories assigned: core @Dr15Jones,@smuzaffar,@makortel you have been requested to review this Pull request/Issue and eventually sign? Thanks |
A new Issue was created by @makortel Matti Kortelainen. @Dr15Jones, @perrotta, @dpiparo, @silviodonato, @smuzaffar, @makortel, @qliphy can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
So the thread 9 is running |
seems so. |
@makortel I had thought that now ROOT had put an |
Maybe @pcanal could comment? |
There is another one in workflow 4.28 step 3 in CMSSW_12_0_X_2021-07-16-1100
|
assign dqm |
New categories assigned: dqm @jfernan2,@kmaeshima,@rvenditti,@andrius-k,@ErnestaP,@ahmad3213 you have been requested to review this Pull request/Issue and eventually sign? Thanks |
Another one also in DQMServices/Demo unit test in CMSSW_12_0_X_2021-07-16-1100 |
#34532 protects all |
ROOT has isolation calls around the TThreadExecutor parallel for/reduce/map/etc. However, the TTree (and RNTuple) IMT compression uses TTaskGroup, and the way that is structured it isn't possible to do the arena isolation internally to the TTaskGroup, and doing it externally would mean breaking the TTaskGroup abstraction--so it ends up being left to the caller to decide what to do. During the RNTuple work there was some discussion with @jblomer about this but I don't think we reached any conclusion on whether it would be possible to use one of the TThreadExecutor operations instead of TTaskGroup. |
+1 |
I am looking at this now carefully trying to understand if I should remove those new asserts. |
See pull request #34557. Hopefully this handles this better. |
In PR test #34517 (comment)
TestDQMServicesDemo
test, more specificallycmsRun DQMServices/Demo/test/run_analyzers_cfg.py outfile=huge.root numberEventsInRun=300 numberEventsInLuminosityBlock=100 nEvents=600 nThreads=10
failed withThe text was updated successfully, but these errors were encountered: