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 getLumiListInValidFiles() in UserUtilities #5175

Closed
2 tasks
belforte opened this issue Oct 13, 2022 · 1 comment
Closed
2 tasks

update getLumiListInValidFiles() in UserUtilities #5175

belforte opened this issue Oct 13, 2022 · 1 comment
Assignees

Comments

@belforte
Copy link
Member

belforte commented Oct 13, 2022

Current code only works in python2, and even in that case fails with new go-based DBS server.
See
https://cms-talk.web.cern.ch/t/error-when-using-getlumilistinvalidfiles/16240/3

  • quick fix: make old client work with new server using the tip from Kifri [1]
  • add a warning in case it is called in python2 script.

long term fix: use dasgoclient - likely not worth !

[1]
I edited the following lines:

lumis = f['lumi_section_num']
for lumi in lumis:
runLumiPairs.append((run,lumi))

to

lumi = f['lumi_section_num']
runLumiPairs.append((run, lumi))

since f['lumi_section_num'] is just an int. It now works as expected.

@belforte
Copy link
Member Author

But in reality new python3 DBS Client is not part of CMSSW distribution. It needs to be pip-installed by users.
So I will go for dasgoclient

belforte added a commit to belforte/CRABClient that referenced this issue Sep 22, 2023
belforte added a commit to belforte/CRABClient that referenced this issue Sep 22, 2023
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

1 participant