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

Look into enabling edm::ESTransientHandle memory optimization again #31085

Open
makortel opened this issue Aug 6, 2020 · 4 comments
Open

Look into enabling edm::ESTransientHandle memory optimization again #31085

makortel opened this issue Aug 6, 2020 · 4 comments

Comments

@makortel
Copy link
Contributor

makortel commented Aug 6, 2020

After all EDModules have been migrated to EventSetup consumes (#31061), framework knows exactly which ESProducts are consumed at which stages of the application. With this information we could look into enabling the memory optimization with edm::ESTransientHandle again.
https://twiki.cern.ch/twiki/bin/view/CMSPublic/SWGuideHowToGetDataFromES#Optimizing_memory_usage

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 6, 2020

A new Issue was created by @makortel Matti Kortelainen.

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

cms-bot commands are listed here

@makortel
Copy link
Contributor Author

makortel commented Aug 6, 2020

assign core

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 6, 2020

New categories assigned: core

@Dr15Jones,@smuzaffar,@makortel you have been requested to review this Pull request/Issue and eventually sign? Thanks

@makortel
Copy link
Contributor Author

All EDModules run in IBs have been migrated, and esConsumes is now enforced at run time (#35588). Therefore this optimization could be thought more seriously now.

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

2 participants