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

Invalid filename specified in 2021 geometry XML file #26771

Closed
christopheralanwest opened this issue May 13, 2019 · 8 comments
Closed

Invalid filename specified in 2021 geometry XML file #26771

christopheralanwest opened this issue May 13, 2019 · 8 comments

Comments

@christopheralanwest
Copy link
Contributor

There is a minor problem with the rearranging of files in PR #26535. The line

'Geometry/MuonSimData/data/2021/muonProdCuts.xml',

specifies an invalid filename. When commit a87a020 renamed Geometry/MuonSimData/data/2021/muonProdCuts.xml
to Geometry/MuonSimData/data/2021/v1/muonProdCuts.xml, I suppose the corresponding change should have also been made to cmsExtendedGeometry2021XML_cfi.py.

I came across this problem as follows:

cmsrel CMSSW_10_6_X_2019-05-10-1100
cd CMSSW_10_6_X_2019-05-10-1100/src
cmsenv
cmsRun $CMSSW_RELEASE_BASE/src/CondTools/Hcal/test/runDumpHcalCond_cfg.py dumplist="LutMetadata" usehardcode=True run=999999 geometry=Extended2021

which results in the exception message:
edm::FileInPath unable to find file Geometry/MuonSimData/data/2021/muonProdCuts.xml anywhere in the search path.
though clearly the exact command isn't relevant.

@cmsbuild
Copy link
Contributor

A new Issue was created by @christopheralanwest .

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

cms-bot commands are listed here

@Dr15Jones
Copy link
Contributor

assign geometry

@cmsbuild
Copy link
Contributor

New categories assigned: geometry

@Dr15Jones,@cvuosalo,@mdhildreth,@ianna,@civanch you have been requested to review this Pull request/Issue and eventually sign? Thanks

@kpedro88
Copy link
Contributor

@boudoul @ianna FYI

@boudoul
Copy link
Contributor

boudoul commented May 21, 2019

It should be fixed with #26856 - Thanks for reporting.

@christopheralanwest
Copy link
Contributor Author

I'm closing this issue as it has been resolved. @cvuosalo @ianna, you may want to sign off for bookkeeping purposes.

@ianna
Copy link
Contributor

ianna commented Aug 3, 2019

+1

@cmsbuild
Copy link
Contributor

cmsbuild commented Aug 3, 2019

This issue is fully signed and ready to be closed.

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

6 participants