You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Currently CLI options use underscores to separate words, which differs from the default for click/typer option naming (see Parametershere) (and so also from janus currently), and more generally from GNU conventions.
We certainly want to be consistent with janus, and I think sticking to the standard would be the best approach.
Note: Using hyphens was an issue for reading the config for janus, but that's now fixed. A similar fix may be needed for reading aiida inputs, but anything passed to janus should be fine.
While changing option names, we may want to consider if we want to unify some of the shared options e.g. arch/architecture
Currently CLI options use underscores to separate words, which differs from the default for click/typer option naming (see
Parameters
here) (and so also fromjanus
currently), and more generally from GNU conventions.We certainly want to be consistent with janus, and I think sticking to the standard would be the best approach.
Note: Using hyphens was an issue for reading the config for janus, but that's now fixed. A similar fix may be needed for reading aiida inputs, but anything passed to janus should be fine.
While changing option names, we may want to consider if we want to unify some of the shared options e.g.
arch
/architecture
Originally posted by @ElliottKasoar in #104 (comment)
The text was updated successfully, but these errors were encountered: