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 slha files for EXO DisappTrk Run 3 analysis #14

Merged
merged 1 commit into from
Sep 14, 2023

Conversation

borzari
Copy link
Contributor

@borzari borzari commented Sep 11, 2023

Distinct mass/ctau slha files for the central production of LL charginos signal samples for the EXO disappearing tracks analysis in Run 3.

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @borzari for branch master.

@SiewYan, @smuzaffar, @mkirsano, @aandvalenzuela, @iarspider, @Saptaparna, @cmsbuild, @alberto-sanchez, @menglu21, @GurpreetSinghChahal can you please review it and eventually sign? Thanks.
@rappoccio, @antoniovilela, @sextonkennedy you are the release manager for this.
cms-bot commands are listed here

@tvami
Copy link
Contributor

tvami commented Sep 11, 2023

@cmsbuild , please test

@cmsbuild
Copy link
Contributor

+1

Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-18e078/34694/summary.html
COMMIT: adeb544
CMSSW: CMSSW_13_3_X_2023-09-11-1100/el8_amd64_gcc11
User test area: For local testing, you can use /cvmfs/cms-ci.cern.ch/week0/cms-data/Configuration-Generator/14/34694/install.sh to create a dev area with all the needed externals and cmssw changes.

Comparison Summary

Summary:

  • You potentially added 13 lines to the logs
  • Reco comparison results: 6 differences found in the comparisons
  • DQMHistoTests: Total files compared: 48
  • DQMHistoTests: Total histograms compared: 3153414
  • DQMHistoTests: Total failures: 3
  • DQMHistoTests: Total nulls: 0
  • DQMHistoTests: Total successes: 3153389
  • 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

@menglu21
Copy link

+1

@tvami
Copy link
Contributor

tvami commented Sep 13, 2023

hi @smuzaffar can you please merge this?

@smuzaffar
Copy link
Contributor

This PR only adds new files, so testing without cmssw PR (which actually make sure of these files) is only going to check if packaging works. Is there any cmssw PR to go with it?

@tvami
Copy link
Contributor

tvami commented Sep 14, 2023

Is there any cmssw PR to go with it?

hi @smuzaffar this is used in McM for signal generation, so there is no main cmssw PR to go with.

@smuzaffar
Copy link
Contributor

+externals

@smuzaffar
Copy link
Contributor

so by default itwill go to 13.3.X release cycle. let us know if this should be backported to any older release cycle

@cmsbuild
Copy link
Contributor

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. @rappoccio, @antoniovilela, @sextonkennedy (and backports should be raised in the release meeting by the corresponding L2)

@tvami
Copy link
Contributor

tvami commented Sep 14, 2023

let us know if this should be backported to any older release cycle

yes, it needs to go back to 12_4_X for the 2022 MC productions. I remember that there used to be a tag update in cmsdist to do this, but I dont see the master PR for that automatically triggered.

@tvami
Copy link
Contributor

tvami commented Sep 14, 2023

but I dont see the master PR for that automatically triggered.

nvm, it did come up 4 min after I wrote this :)

@smuzaffar
Copy link
Contributor

we are updating bot scripts to python3 and automatic master branch PR creation script had a bug. cms-sw/cms-bot@a4f7e3e has fixed the issue :-)

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