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

Default VarParsing options in runGEMUnpacker_cfg.py do not work #46815

Open
makortel opened this issue Nov 27, 2024 · 5 comments
Open

Default VarParsing options in runGEMUnpacker_cfg.py do not work #46815

makortel opened this issue Nov 27, 2024 · 5 comments

Comments

@makortel
Copy link
Contributor

I noticed in #45640 (comment) the VarParsing defaults in

options.register('feds',
[1467,1468],
VarParsing.VarParsing.multiplicity.list,
VarParsing.VarParsing.varType.int,
"List of FEDs")

do not work, because (given how VarParsing, unfortunately, works) they result in a list of lists [[1467, 1468]] that then results an error

TypeError: wrong types (<class 'list'>) added to <class 'FWCore.ParameterSet.Types.vint32'>

Presently there is no intention to improve VarParsing behavior in this regards (see rationale in #45961 (comment) and #45640 (comment)). Our recommendation would be switch to standard python argparse. If that is not feasible, and if the aforementioned behavior is important, the configuration file should handle this case by itself.

@makortel
Copy link
Contributor Author

assign EventFilter/GEMRawToDigi

@cmsbuild
Copy link
Contributor

New categories assigned: reconstruction

@jfernan2,@mandrenguyen you have been requested to review this Pull request/Issue and eventually sign? Thanks

@cmsbuild
Copy link
Contributor

cms-bot internal usage

@cmsbuild
Copy link
Contributor

A new Issue was created by @makortel.

@Dr15Jones, @antoniovilela, @makortel, @mandrenguyen, @rappoccio, @sextonkennedy, @smuzaffar can you please review it and eventually sign/assign? Thanks.

cms-bot commands are listed here

@jfernan2
Copy link
Contributor

FYI @jshlee @dildick

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

3 participants