Adding streamerDQMOnlineScouting streamer files from Run392642 #11
Conversation
|
Pull request #11 was updated. |
|
cms-bot internal usage |
README.md
Outdated
| @@ -26,7 +26,11 @@ Currently the unitTest that make use of input streamer files are: | |||
| run362321/run362321_ls0231_streamHIDQM_pid276864.dat | |||
| run362321/run362321_ls0231_streamHIDQM_pid276864.jsn | |||
| ``` | |||
|
|
|||
| * the `scouting_dqm_sourceclient-live_cfg.py` reads the `streamDQMOnlineScouting` streamer files generated from run 392642 (from Run2025C pp run, [OMS link](https://cmsoms.cern.ch/cms/runs/report?cms_run=392642&cms_run_sequence=GLOBAL-RUN): | |||
There was a problem hiding this comment.
picky a paranthesis is missing here :)
|
A new Pull Request was created by @jprendi for branch main. @akritkbehera, @antoniovagnerini, @cmsbuild, @ctarricone, @iarspider, @rseidita, @smuzaffar can you please review it and eventually sign? Thanks. |
README.md
Outdated
| @@ -124,6 +128,69 @@ cmsRun hlt.py &> hlt.log | |||
| rm -f run362321/run362321_ls0*_index*.* | |||
| ``` | |||
|
|
|||
| The streamer files for the streamDQMOnlineScouting were prepared using the scouting specific menu: | |||
| ``` | |||
There was a problem hiding this comment.
| ``` | |
| ```bash |
will render it nicer!
README.md
Outdated
| @@ -124,6 +128,69 @@ cmsRun hlt.py &> hlt.log | |||
| rm -f run362321/run362321_ls0*_index*.* | |||
| ``` | |||
|
|
|||
| The streamer files for the streamDQMOnlineScouting were prepared using the scouting specific menu: | |||
There was a problem hiding this comment.
| The streamer files for the streamDQMOnlineScouting were prepared using the scouting specific menu: | |
| The streamer files for the `streamDQMOnlineScouting` were prepared using the scouting specific menu: |
README.md
Outdated
| mkdir -p prepared | ||
| cat run392642/run392642_ls0000_streamDQMOnlineScouting_pid${job_pid}.ini run392642/run392642_ls0174_streamDQMOnlineScouting_pid${job_pid}.dat > prepared/run392642_ls0174_streamDQMOnlineScouting_pid${job_pid}.dat | ||
| ``` | ||
| In the end, one needs to `cat`the resulting `.dat` and `.ini` file into a combined `.dat` file! |
There was a problem hiding this comment.
is this line needed? (I'ts literally what the script does in the last line)
|
Pull request #11 was updated. |
|
test parameters:
|
|
Pull request #11 was updated. |
|
@cmsbuild, please test |
|
+1 Summary: https://cmssdt.cern.ch/SDT/jenkins-artifacts/pull-request-integration/PR-6714c2/46952/summary.html 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: Comparison SummarySummary:
|
|
REMINDER @sextonkennedy, @antoniovilela, @rappoccio, @mandrenguyen: This PR was tested with cms-sw/cmssw#48398, please check if they should be merged together |
|
+dqm |
|
@akritkbehera,@iarspider, @rseidita, @smuzaffar any objections to merge this addition? |
|
+externals new data files which looks like only used by new unit test added via https://github.com/cms-sw/cmssw/pull/48398/files#diff-b31b8d983593343c10bd8e9fbef9520af282683368c21c4c520f19c3ad83574f @jprendi @antoniovagnerini , I think this change can go in to IB even before cms-sw/cmssw#48398 ... right? |
cmsbuild
added
externals-approved
fully-signed
and removed
externals-pending
pending-signatures
labels
|
This pull request is fully signed and it will be integrated in one of the next main IBs (tests are also fine). This pull request will now be reviewed by the release team before it's merged. @rappoccio, @sextonkennedy, @mandrenguyen, @antoniovilela (and backports should be raised in the release meeting by the corresponding L2) |
right. |
|
this should be available in 11h00 IB today |
|
+1 |
Dear DQM-Integration community,
the addition of a new Scouting Online DQM Stream (as discussed here and here) lead to the preparation of a new HLT scouting DQM online client. The new so-called
DQMOnlineScoutingrequires new streamer files to run the unit tests on. In this PR, I added the streamer files specifically for that stream, updated the documentation to include this new streamer files and also added a recipe on how the streamer files were produced.Thanks in advance.