search

martius-lab / cluster_utils

https://cluster-utils.readthedocs.io/stable/
Other
12 stars 0 forks source link

Allow more flexible parameter names #62

Open luator opened 9 months ago

luator commented 9 months ago

Currently, clusterutils complains when using any characters outside of `[0-9][a-z][A-Z]-.:for the parameter names. This restriction should not be there, because depending on the used frameworks, other characters might be needed, e.g. slashes/`.

This restriction might be there because cluster_utils stores parameters in the smart_settings dict, where they are accessible as attributes, and python attributes have a restricted set of allowed characters as well.

By @mseitzer (imported from GitLab)

luator commented 9 months ago

My first thought was that it is also a restriction imposed by the config file formats but I quickly checked and json, yaml and toml are all to be okay with quoted keys containing special characters. So the attribute-access is probably really the only restricting part. If it's not too urgent, I suggest that we address this as part of #41 (which will require a lot of config-related changes anyway).

By Felix Widmaier on 2024-02-19T10:39:40 (imported from GitLab)