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

Hotline skim AlCaReco workflows #9627

Merged
merged 15 commits into from Jul 6, 2015
Merged

Hotline skim AlCaReco workflows #9627

merged 15 commits into from Jul 6, 2015

Conversation

duanders
Copy link
Contributor

-Adding new package Calibration/Hotline containing hotline skim code
-Adding two new AlCaRecos implementing the hotline skims
-Including the hotline skims in the AlCaRecoMatrix under StreamExpress
-Including the hotline skims in the RelVal RECOD step

See presentation here: https://indico.cern.ch/event/368342/contribution/5/material/slides/0.pdf

@cmsbuild
Copy link
Contributor

A new Pull Request was created by @duanders (Dustin Anderson) for CMSSW_7_5_X.

Hotline skim AlCaReco workflows

It involves the following packages:

Calibration/Hotline
Configuration/AlCa
Configuration/EventContent
Configuration/PyReleaseValidation
Configuration/StandardSequences

The following packages do not have a category, yet:

Calibration/Hotline

@diguida, @boudoul, @franzoni, @cerminar, @cmsbuild, @srimanob, @mmusich, @davidlange6 can you please review it and eventually sign? Thanks.
@ghellwig, @Martin-Grunewald, @makortel, @GiacomoSguazzoni, @rovere, @VinInn, @appeltel, @cerati, @dgulhan this is something you requested to watch as well.
You can sign-off by replying to this message having '+1' in the first line of your reply.
You can reject by replying to this message having '-1' in the first line of your reply.
If you are a L2 or a release manager you can ask for tests by saying 'please test' in the first line of a comment.
@Degano you are the release manager for this.
You can merge this pull request by typing 'merge' in the first line of your comment.

@davidlange6
Copy link
Contributor

@duanders -did you mean to commit a plugin? (if not, why the BuildFile.xml?)

@cmsbuild
Copy link
Contributor

Pull request #9627 was updated. @diguida, @boudoul, @franzoni, @cerminar, @cmsbuild, @srimanob, @mmusich, @davidlange6 can you please check and sign again.

@duanders
Copy link
Contributor Author

@davidlange6 Sorry, the BuildFile was included by mistake. (a previous version of the skim code contained a new plugin.) Fixed this.

@franzoni
Copy link

please test

@diguida
Copy link
Contributor

diguida commented Jun 30, 2015

+1

  • new package Calibration/Hotline containing hotline skim code
  • new AlCaReco implementing the hotline skim: sequences are in the client code, and paths are implemented in Configuration/StandardSequences/python/AlCaRecoStreams_cff.py;
  • hotline skim is included in the AlCaRecoMatrix under StreamExpress;
  • hotline skim is included in dataReco definition for data RelVals;
  • re-shaping of the definition of AlCaReco EventContent, in order to avoid python circular dependencies.

To be implemented: AlCa prompt implementation for AlCa harvesting step.

@diguida
Copy link
Contributor

diguida commented Jun 30, 2015

to be implemented: AlCa prompt implementation for AlCa harvesting step.

@mmusich
Copy link
Contributor

mmusich commented Jun 30, 2015

@diguida we will take care when merging the other changes (e.g. muon alignment producers)

@mmusich
Copy link
Contributor

mmusich commented Jul 8, 2015

Reporting here analysis from Salvatore:

1. Test setup:
a. job run on Run1 HighMet2011A sample in RelVal (wf 4.28)
b. Input data tier: RAW-RECO
2. Output events and collections:
a. Output events: 10 [1]
b. Output collections: see [2] for the full list; they are in agreement with the FEVT event content definition as in [3]
3. Event size: see [4]. Biggest ones:
Branch Name | Average Uncompressed Size (Bytes/Event) | Average Compressed Size (Bytes/Event)
FEDRawDataCollection_rawDataCollector__reHLT. 401125 158183
SiStripClusteredmNewDetSetVector_siStripClusters__reRECO. 118813 65220.1
recoTrackExtras_generalTracks__reRECO. 44114.8 26372.6
CaloTowersSorted_towerMaker__reRECO. 65780 24862.2
recoHcalNoiseRBXs_hcalnoise__reRECO. 57250.4 24046.1
EcalRecHitsSorted_ecalRecHit_EcalRecHitsEB_reRECO. 31920.9 20155
recoPFClusters_particleFlowClusterPS__reRECO. 64398.3 19341.7
recoPFCandidates_particleFlow__reRECO. 91831.5 18728.4
SiPixelClusteredmNewDetSetVector_siPixelClusters__reRECO. 31339 16795.3
TrackingRecHitsOwned_generalTracks__reRECO. 98426.1 16293.2
recoTracks_generalTracks__reRECO. 35294.3 15940.6
recoGsfElectrons_uncleanedOnlyGsfElectrons__reRECO. 15930.3 15617.6
recoGsfElectrons_gedGsfElectrons__reRECO. 15255.5 14939.7
HBHERecHitsSorted_hbheprereco__reRECO. 29119.7 14430.7
HBHERecHitsSorted_hbhereco__reRECO. 29086.7 14398
4. Event rate. Looking at WBM, I see that these two runs correspond to about 10 hours of data taking:
https://cmswbm.web.cern.ch/cmswbm/cmsdb/servlet/RunSummary?RUN=165121&SUBMIT=Submit
https://cmswbm.web.cern.ch/cmswbm/cmsdb/servlet/RunSummary?RUN=172802&SUBMIT=Submit

[1] edmFileUtil 4.28_HighMet2011A+HighMet2011A+HLTDSKIM+RECODreHLT+HARVESTDreHLT+MINIAODDreHLT/Hotline.root -e
[2] edmDumpEventContent 4.28_HighMet2011A+HighMet2011A+HLTDSKIM+RECODreHLT+HARVESTDreHLT+MINIAODDreHLT/Hotline.root --forceColumns
[3] https://github.com/cms-sw/cmssw/blob/CMSSW_7_6_X/Calibration/Hotline/python/hotlineSkims_Output_cff.py
[4] edmEventSize 4.28_HighMet2011A+HighMet2011A+HLTDSKIM+RECODreHLT+HARVESTDreHLT+MINIAODDreHLT/Hotline.root -v

Findings on 75X IB relvals are in line with expectations.

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.

None yet

6 participants