We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
I expected this to work:
./panda-shortname data16_13TeV:data16_13TeV.periodI.physics_Main.PhysCont.DAOD_SUSY10.grp16_v03_p3372
but it doesn't since the dataset ID is periodI instead of a number (somewhat unexpected but...)
periodI
The text was updated successfully, but these errors were encountered:
I've done the somewhat useless thing here and added a help wanted label.
Sorry, something went wrong.
To maybe pick up this Issue again soon:
$ docker run --rm -it -e USER="${USER}" python:3.8 /bin/bash # python -m pip install --upgrade -q pip setuptools wheel # pip install -q pandamonium # pip list Package Version ------------ ------- panda-client 1.4.36 pandamonium 0.1.0 pip 20.2.3 setuptools 50.3.0 wheel 0.35.1 # panda-shortname data16_13TeV:data16_13TeV.periodI.physics_Main.PhysCont.DAOD_SUSY10.grp16_v03_p3372 Traceback (most recent call last): File "/usr/local/bin/panda-shortname", line 8, in <module> sys.exit(main()) File "/usr/local/lib/python3.8/site-packages/pandamonium/panda_shortname.py", line 49, in main oname = process_name(ds_name, args.remove, args.prefix, args.suffix) File "/usr/local/lib/python3.8/site-packages/pandamonium/panda_shortname.py", line 74, in process_name assert dsid.isdigit(), "dsid ({}) not a number".format(dsid) AssertionError: dsid (periodI) not a number
So currently looking at the code
pandamonium/src/pandamonium/panda_shortname.py
Lines 73 to 74 in 2c99516
it seems this could be addressed with an or.
or
This example is also probably good as a unit test.
No branches or pull requests
I expected this to work:
but it doesn't since the dataset ID is
periodI
instead of a number (somewhat unexpected but...)The text was updated successfully, but these errors were encountered: