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

Adding ZDC LUT for use with L1TZDC Emulator; to be pushed to cms-data… #23

Merged
merged 1 commit into from
Sep 1, 2023

Conversation

cfmcginn
Copy link
Contributor

LUT for L1TZDCProducer, emulator of ZDC l1 trigger for 2023 HI datataking - PR in progress to cmssw here:

cms-sw/cmssw#42634

Simultaneous PR to cms-l1t-offline/L1Trigger-L1TCalorimeter here:

cms-l1t-offline#20

Tagging a few people to track - @ginnocen @bundocka @aloeliger

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @cfmcginn (Chris) for branch master.

@smuzaffar, @epalencia, @aandvalenzuela, @iarspider, @cmsbuild, @aloeliger can you please review it and eventually sign? Thanks.
@perrotta, @dpiparo, @antoniovilela, @rappoccio you are the release manager for this.
cms-bot commands are listed here

@aloeliger
Copy link

please test

@cmsbuild
Copy link
Contributor

-1

Failed Tests: RelVals AddOn
Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c59d2f/34554/summary.html
COMMIT: 6a8af5a
CMSSW: CMSSW_13_3_X_2023-08-30-2300/el8_amd64_gcc11
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-data/L1Trigger-L1TCalorimeter/23/34554/install.sh to create a dev area with all the needed externals and cmssw changes.

RelVals

AddOn Tests

----- Begin Fatal Exception 31-Aug-2023 11:04:29 CEST-----------------------
An exception of category 'ConfigFileReadError' occurred while
   [0] Processing the python configuration file named /cvmfs/cms-ib.cern.ch/sw/x86_64/week0/el8_amd64_gcc11/cms/cmssw/CMSSW_13_3_X_2023-08-30-2300/src/Utilities/ReleaseScripts/scripts/read312RV_cfg.py
Exception Message:
 unknown python problem occurred.
ModuleNotFoundError: No module named 'past'

At:
  /cvmfs/cms-ib.cern.ch/sw/x86_64/week0/el8_amd64_gcc11/cms/cmssw/CMSSW_13_3_X_2023-08-30-2300/src/FWCore/ParameterSet/python/Types.py(6): <module>
  <frozen importlib._bootstrap>(228): _call_with_frames_removed
  <frozen importlib._bootstrap_external>(850): exec_module
  <frozen importlib._bootstrap>(695): _load_unlocked
  <frozen importlib._bootstrap>(986): _find_and_load_unlocked
  <frozen importlib._bootstrap>(1007): _find_and_load
  /cvmfs/cms-ib.cern.ch/sw/x86_64/week0/el8_amd64_gcc11/cms/cmssw/CMSSW_13_3_X_2023-08-30-2300/src/FWCore/ParameterSet/python/Config.py(15): <module>
  <frozen importlib._bootstrap>(228): _call_with_frames_removed
  <frozen importlib._bootstrap_external>(850): exec_module
  <frozen importlib._bootstrap>(695): _load_unlocked
  <frozen importlib._bootstrap>(986): _find_and_load_unlocked
  <frozen importlib._bootstrap>(1007): _find_and_load
  /cvmfs/cms-ib.cern.ch/sw/x86_64/week0/el8_amd64_gcc11/cms/cmssw/CMSSW_13_3_X_2023-08-30-2300/src/Utilities/ReleaseScripts/scripts/read312RV_cfg.py(2): <module>

----- End Fatal Exception -------------------------------------------------
[fastsim:1] cmsDriver.py TTbar_8TeV_TuneCUETP8M1_cfi  --conditions auto:run1_mc --fast  -n 100 --eventcontent AODSIM,DQM --relval 100000,1000 -s GEN,SIM,RECOBEFMIX,DIGI:pdigi_valid,L1,DIGI2RAW,L1Reco,RECO,VALIDATION  --customise=HLTrigger/Configuration/CustomConfigs.L1THLT --datatier GEN-SIM-DIGI-RECO,DQMIO --beamspot Realistic8TeVCollision : FAILED - elapsed time: 0 sec (ended on Thu Aug 31 11:04:34 2023) - exit: 256
[fastsim1:1] cmsDriver.py TTbar_13TeV_TuneCUETP8M1_cfi --conditions auto:run2_mc_l1stage1 --fast  -n 100 --eventcontent AODSIM,DQM --relval 100000,1000 -s GEN,SIM,RECOBEFMIX,DIGI:pdigi_valid,L1,DIGI2RAW,L1Reco,RECO,VALIDATION  --customise=HLTrigger/Configuration/CustomConfigs.L1THLT --datatier GEN-SIM-DIGI-RECO,DQMIO --beamspot NominalCollision2015 --era Run2_25ns : FAILED - elapsed time: 0 sec (ended on Thu Aug 31 11:04:39 2023) - exit: 256
Expand to see more addon errors ...

@aloeliger
Copy link

@cfmcginn According to your email today, it seems like externals really aren't actually the place for this. We need to instead discuss with the o2o/db experts.

@aloeliger
Copy link

@cfmcginn According to your email today, it seems like externals really aren't actually the place for this. We need to instead discuss with the o2o/db experts.

After further discussion, perhaps this is the correct location for this for the time being, the relative rate of change of this LUT may be such that having it in the externals for testing while a db solution is created is desirable.

@aloeliger
Copy link

@smuzaffar The test failures are complaining largely about past, which is core python stuff. Is this just a transient python error?

@cfmcginn
Copy link
Contributor Author

@cfmcginn According to your email today, it seems like externals really aren't actually the place for this. We need to instead discuss with the o2o/db experts.

After further discussion, perhaps this is the correct location for this for the time being, the relative rate of change of this LUT may be such that having it in the externals for testing while a db solution is created is desirable.

Understood - please let me know if these test failures require changes by me, will iterate.

@smuzaffar
Copy link
Contributor

test parameters:

@smuzaffar
Copy link
Contributor

please test

@smuzaffar The test failures are complaining largely about past, which is core python stuff. Is this just a transient python error?

I guess it needs cmssw PR to run properly ?

@cmsbuild
Copy link
Contributor

cmsbuild commented Sep 1, 2023

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c59d2f/34568/summary.html
COMMIT: 6a8af5a
CMSSW: CMSSW_13_3_X_2023-08-31-1100/el8_amd64_gcc11
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-data/L1Trigger-L1TCalorimeter/23/34568/install.sh to create a dev area with all the needed externals and cmssw changes.

The following merge commits were also included on top of IB + this PR after doing git cms-merge-topic:

You can see more details here:
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c59d2f/34568/git-recent-commits.json
https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-c59d2f/34568/git-merge-result

Comparison Summary

Summary:

  • You potentially added 28 lines to the logs
  • Reco comparison results: 331 differences found in the comparisons
  • DQMHistoTests: Total files compared: 48
  • DQMHistoTests: Total histograms compared: 3153350
  • DQMHistoTests: Total failures: 119
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 3153209
  • DQMHistoTests: Total skipped: 22
  • DQMHistoTests: Total Missing objects: 0
  • DQMHistoSizes: Histogram memory added: 0.0 KiB( 47 files compared)
  • Checked 207 log files, 159 edm output root files, 48 DQM output files
  • TriggerResults: no differences found

@aloeliger
Copy link

+l1

@rappoccio
Copy link

+1

@rappoccio
Copy link

merge

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.

5 participants