-
Notifications
You must be signed in to change notification settings - Fork 4.4k
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
Provide a way in configuration validation to specify containers of a set length #13180
Comments
A new Issue was created by @Dr15Jones (Chris Jones). @davidlange6, @smuzaffar, @Degano, @davidlt, @Dr15Jones can you please review it and eventually sign/assign? Thanks. cms-bot commands are listed here |
assign core |
New categories assigned: core @Dr15Jones,@smuzaffar you have been requested to review this Pull request/Issue and eventually sign? Thanks |
The problem with using This does bring up the possibility that we abandon the need for standard types to be specified in the python. When converting from the python values to C++ we just enforce that the types are convertible (e.g. python |
@Dr15Jones , @makortel was this implemented? |
Thanks Matti, I had missed the second part... I've made #35316 to use the new functionality. |
There are times where a container of parameters is required to be of a set length. We'd like to have the configuration validation system be able to enforce such a requirement. One suggestion was to possibly allow the use of
std::array<>
as a C++ parameter type.The text was updated successfully, but these errors were encountered: