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

Update of the LumiPCC module veto list and afterglow factors for 2024 #44568

Merged
merged 1 commit into from
Apr 3, 2024

Conversation

radla118
Copy link
Contributor

PR description:

Update of the LumiPCC module veto list and afterglow parameters implemented in the corresponding configuration files. The parameters and the veto are defined by studying the reprocessed 2023 data. The workflow remains unchanged. Similar changes have been requested for 2023 #41242 #41256

PR validation:

Validated by running runTheMatrix.py -l 1020.0 configuration.

Backport will be requested later

@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 28, 2024

cms-bot internal usage

@cmsbuild
Copy link
Contributor

+code-checks

Logs: https://cmssdt.cern.ch/SDT/code-checks/cms-sw-PR-44568/39716

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @radla118 for master.

It involves the following packages:

  • Calibration/LumiAlCaRecoProducers (alca)

@saumyaphor4252, @cmsbuild, @perrotta, @consuegs can you please review it and eventually sign? Thanks.
@yuanchao, @mmusich, @rsreds, @tocheng this is something you requested to watch as well.
@antoniovilela, @rappoccio, @sextonkennedy you are the release manager for this.

cms-bot commands are listed here

@perrotta
Copy link
Contributor

please test

@perrotta
Copy link
Contributor

@radla118 in a comment posted for the similar PR prepared last year it was written:
"privately also agreed with the developers that this list should be provided dynamically through the DB"
After exactly one year, which is the status of that migration of these parameters into the DB?

And, besides the "technical" validation that you are mentioning in the PR description (i.e. the PR doesn't crash cmssw), how did you verify that the parameters were typed correctly in the updated configuration?

@perrotta
Copy link
Contributor

please abort

@perrotta
Copy link
Contributor

perrotta commented Mar 28, 2024

please test workflow 1020.0

@cmsbuild
Copy link
Contributor

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-879e89/38478/summary.html
COMMIT: 7734bc2
CMSSW: CMSSW_14_1_X_2024-03-27-2300/el8_amd64_gcc12
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-sw/cmssw/44568/38478/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

  • You potentially added 29 lines to the logs
  • Reco comparison results: 47 differences found in the comparisons
  • DQMHistoTests: Total files compared: 49
  • DQMHistoTests: Total histograms compared: 3297437
  • DQMHistoTests: Total failures: 6
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 3297411
  • DQMHistoTests: Total skipped: 20
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 48 files compared)
  • Checked 207 log files, 167 edm output root files, 49 DQM output files
  • TriggerResults: no differences found

@radla118
Copy link
Contributor Author

radla118 commented Apr 3, 2024

@radla118 in a comment posted for the similar PR prepared last year it was written: "privately also agreed with the developers that this list should be provided dynamically through the DB" After exactly one year, which is the status of that migration of these parameters into the DB?

@perrotta we are working on a more flexible dynamic module veto (i e define unique veto list for ~100 LS periods). Some preliminary studies are already done, but the final strategy and the implementation is not finalized yet. It is clear that the current configuration is rather inconvenient, but we did not want to change it until everything is settled.

And, besides the "technical" validation that you are mentioning in the PR description (i.e. the PR doesn't crash cmssw), how did you verify that the parameters were typed correctly in the updated configuration?

We store our module veto in the dedicated CMS-LUMI-POG/PCCTools git repo. I just compared the central list and the copied one using a simple script.

@perrotta
Copy link
Contributor

perrotta commented Apr 3, 2024

+alca

  • Same procedure to determine the misbehaving modules with past year data as was done for similar previous PRs
  • The migration towards the storage of these parameters in the db rather than in python config is still ongoing...

@cmsbuild
Copy link
Contributor

cmsbuild commented Apr 3, 2024

This pull request is fully signed and it will be integrated in one of the next master IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @sextonkennedy, @antoniovilela, @rappoccio (and backports should be raised in the release meeting by the corresponding L2)

@antoniovilela
Copy link
Contributor

+1

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.

4 participants