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

Update edmLumisInFiles.py to python3 #37186

Closed
sanuvarghese opened this issue Mar 9, 2022 · 7 comments
Closed

Update edmLumisInFiles.py to python3 #37186

sanuvarghese opened this issue Mar 9, 2022 · 7 comments

Comments

@sanuvarghese
Copy link

The issue is regarding an error when running edmLumisInFiles.py file.root command in CMSSW_12_3_0_pre5 (or any 12_X) , I am getting the error:

Traceback (most recent call last):
  File "/cvmfs/cms.cern.ch/slc7_amd64_gcc10/cms/cmssw/CMSSW_12_3_0_pre5/bin/slc7_amd64_gcc10/edmLumisInFiles.py", line 4, in <module>
    from FWCore.PythonUtilities.LumiList   import LumiList
ImportError: No module named FWCore.PythonUtilities.LumiList

The error seems to be a python3 vs python2 problem. I was able to run the same command in 10_6_X without error.

@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 9, 2022

A new Issue was created by @sanuvarghese Sanu Varghese.

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

cms-bot commands are listed here

@makortel
Copy link
Contributor

makortel commented Mar 9, 2022

assign core

@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 9, 2022

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

makortel commented Mar 9, 2022

This has been fixed already in #37107 (that was merged in CMSSW_12_3_X_2022-03-02-1100). @sanuvarghese Are you in need of a backport, or is this sufficient (it will be in 12_3_0_pre6 that should get built soon).

@sanuvarghese
Copy link
Author

No this is sufficient.

@makortel
Copy link
Contributor

makortel commented Mar 9, 2022

+core

@cmsbuild
Copy link
Contributor

cmsbuild commented Mar 9, 2022

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

4 participants