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.

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

Disable edmWriteConfigs #2021

Merged
merged 1 commit into from
Jan 14, 2014
Merged

Conversation

wmtan
Copy link
Contributor

@wmtan wmtan commented Jan 13, 2014

edmwriteConfigs takes an enormous amount of time to run in ROOT6, and will continue to do so until precompiled modules are available. This pull request comments out the code in edmWriteConfigs so that it just returns success immediately. This will greatly speed up ROOT6 IB's which now take 14 to 16 hours.
The configuration files written by edmWriteConfigs are needed to run the relvals successfully, but the config files
generated by ROOT5 can be used. for running the relvals for the time being.

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @wmtan for CMSSW_7_0_ROOT6_X.

Disable edmWriteConfigs

It involves the following packages:

FWCore/ParameterSet

@cmsbuild, @Degano, @Dr15Jones, @ktf, @nclopezo can you please review it and eventually sign? Thanks.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.

@wmtan
Copy link
Contributor Author

wmtan commented Jan 13, 2014

This needs further study. This is closed for now.

@wmtan wmtan closed this Jan 13, 2014
@ktf ktf reopened this Jan 14, 2014
@ktf
Copy link
Contributor

ktf commented Jan 14, 2014

Reopening this since I convinced myself this is the best option, given that we will not be able to run any relval job in any case in the IBs.

ktf added a commit that referenced this pull request Jan 14, 2014
ROOT6 hack -- Disable edmWriteConfigs
@ktf ktf merged commit 3d753f3 into cms-sw:CMSSW_7_0_ROOT6_X Jan 14, 2014
@wmtan wmtan deleted the edmWriteConfigsDisable branch January 15, 2014 00:18
slava77 added a commit to slava77/cmssw that referenced this pull request Nov 27, 2020
cmsbuild added a commit that referenced this pull request Nov 28, 2020
remove 11624.911, #2021 DD4hep from short matrix (until it runs reliably)
smuzaffar added a commit that referenced this pull request Nov 28, 2020
cmsbuild added a commit that referenced this pull request Nov 30, 2020
[11.2.X]Remove 11624.911, #2021 DD4hep from short matrix (until it runs reliably)
silviodonato added a commit that referenced this pull request Dec 4, 2020
cmsbuild added a commit that referenced this pull request Dec 4, 2020
Readd 1624.911, #2021 DD4hep from short matrix
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants