-
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
Validation/CTPPS/python/simu_config/year_*_cff.py #32448
Comments
A new Issue was created by @silviodonato Silvio Donato. @Dr15Jones, @dpiparo, @silviodonato, @smuzaffar, @makortel, @qliphy can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
assign dqm |
New categories assigned: dqm @jfernan2,@andrius-k,@fioriNTU,@kmaeshima,@ErnestaP you have been requested to review this Pull request/Issue and eventually sign? Thanks |
@jan-kaspar has this been solved? Thanks |
We have added some cloning, e.g.:
But please double check that this is sufficient. |
In
Validation/CTPPS/python/simu_config/year_2016_cff.py
Validation/CTPPS/python/simu_config/year_2017_cff.py
Validation/CTPPS/python/simu_config/year_2018_cff.py
some configuration files are loaded and used without cloning them.
Please clone them to avoid bad surprise (see #31484 (comment))
The text was updated successfully, but these errors were encountered: