Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Issues with wfs 25040X in IB tests #38229

Closed
qliphy opened this issue Jun 3, 2022 · 5 comments · Fixed by #38236
Closed

Issues with wfs 25040X in IB tests #38229

qliphy opened this issue Jun 3, 2022 · 5 comments · Fixed by #38236
Labels
pending-signatures simulation-pending

Comments

@qliphy
Copy link
Contributor

qliphy commented Jun 3, 2022

https://cmssdt.cern.ch/SDT/cgi-bin/logreader/el8_amd64_gcc11/CMSSW_12_5_X_2022-06-03-1100/pyRelValMatrixLogs/run/250400.18_ZEE_13+FS_ZEE_13_PRMXUP18_PU50+HARVESTUP18FS+MINIAODMCUP18FS/step1_ZEE_13+FS_ZEE_13_PRMXUP18_PU50+HARVESTUP18FS+MINIAODMCUP18FS.log#/

File "/cvmfs/cms-ib.cern.ch/nweek-02735/el8_amd64_gcc11/cms/cmssw-patch/CMSSW_12_5_X_2022-06-03-1100/src/SimGeneral/PreMixingModule/python/mixOne_premix_on_sim_cfi.py", line 324, in
(runDependentForPixel & premix_stage2).toModify(mixData.workers.pixel,
AttributeError: 'PSet' object has no attribute 'pixel'
@qliphy
Copy link
Contributor Author

qliphy commented Jun 3, 2022 •
edited

Seem to be Related to #37823 @carolinecollard

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 3, 2022

A new Issue was created by @qliphy Qiang Li.

@Dr15Jones, @perrotta, @dpiparo, @makortel, @smuzaffar, @qliphy can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@qliphy qliphy changed the title Issues with 25040X wfs in IB tests Issues with wfs 25040X in IB tests Jun 3, 2022
@qliphy
Copy link
Contributor Author

qliphy commented Jun 3, 2022 •
edited

assign simulation

@cmsbuild
Copy link
Contributor

cmsbuild commented Jun 3, 2022

New categories assigned: simulation

@mdhildreth,@civanch you have been requested to review this Pull request/Issue and eventually sign? Thanks

@mmusich mmusich mentioned this issue Jun 3, 2022
Merged
@mmusich
Copy link
Contributor

mmusich commented Jun 3, 2022

I propose a fix here: #38236

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
pending-signatures simulation-pending
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

apply the runDependentForPixel modifier before the fastSim modifications mmusich/cmssw
3 participants
@cmsbuild @mmusich @qliphy