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

Is GenXSecAnalyzer still useful? #29098

Closed
makortel opened this issue Mar 4, 2020 · 11 comments
Closed

Is GenXSecAnalyzer still useful? #29098

makortel opened this issue Mar 4, 2020 · 11 comments

Comments

@makortel
Copy link
Contributor

makortel commented Mar 4, 2020

The GenXSecAnalyzer is an edm::one::EDAnalyzer<edm::one::WatchRuns, edm::one::WatchLuminosityBlocks> that prevents efficient use of concurrent luminosity blocks (#25090 (comment)). By quick look it aggregates various information in endLuminosityBlock() and endRun(), and prints out the result in endJob().

Is this behavior needed in MC production (or RelVals)? Are these printouts looked at systematically?

@makortel
Copy link
Contributor Author

makortel commented Mar 4, 2020

assign generators

@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 4, 2020

New categories assigned: generators

@alberto-sanchez,@SiewYan,@qliphy,@efeyazgan,@mkirsano,@agrohsje you have been requested to review this Pull request/Issue and eventually sign? Thanks

@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 4, 2020

A new Issue was created by @makortel Matti Kortelainen.

@Dr15Jones, @smuzaffar, @silviodonato, @makortel, @davidlange6, @fabiocos can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@qliphy
Copy link
Contributor

qliphy commented Mar 5, 2020

@makortel

GenXSecAnalyzer is used to gather cross section and filter informations from each luminosityBlock, and then combine them to reach high accuracy with larger statistics.
https://twiki.cern.ch/twiki/bin/viewauth/CMS/HowToGenXSecAnalyzer

This is needed in MC production, especially for MC Contacts who make McM requests and need to input these information (especially on Filter/Matching efficiency which affect the job submission ) based on the print out.

@makortel
Copy link
Contributor Author

makortel commented Mar 5, 2020

@qliphy Thanks. Is it needed in every GEN job (notably in RelVal and production)? Or would it be sufficient if it could be easily included in a GEN job when needed, but it would not be run by default?

@qliphy
Copy link
Contributor

qliphy commented Mar 5, 2020

@makortel Yes, I think for RelVal and Production, this is not needed to be run .

@makortel
Copy link
Contributor Author

makortel commented Mar 9, 2020

How are the configurations for the MC contacts created? Is there perhaps a template, a recipe for cmsDriver command, or generation by a web service?

I'm just thinking if a modifier (e.g. --processModifier genXsecAnalyzer for cmsDriver) would work, and where exactly that would need to be propagated (recipe on webpage/twiki, script, web service)

@qliphy
Copy link
Contributor

qliphy commented Mar 10, 2020

The cmsDriver commands for MC contacts are got from McM automatically:
B2G-RunIISummer15GS-00007
and
https://cms-pdmv.cern.ch/mcm/public/restapi/requests/get_test/B2G-RunIISummer15GS-00007

@makortel
Copy link
Contributor Author

Thanks. So in principle it could be sufficient to modify McM to add an extra parameter to cmsDriver for the MC contacts, right? Does/can McM give a (slightly) different cmsDriver for production? Can that depend on CMSSW version?

Who should be contacted to get such change in McM?

@qliphy
Copy link
Contributor

qliphy commented Mar 12, 2020

adding @justinasr and @pgunnell

@makortel
Copy link
Contributor Author

Made obsolete by GenXSecAnalyzer getting migrated to be thread efficient in #29328.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants