shirtsgroup / physical_validation

Physical validation of molecular simulations
https://physical-validation.readthedocs.io
MIT License
55 stars 19 forks source link

Reconsider default arguments for low-level functions #116

Closed ptmerz closed 3 years ago

ptmerz commented 3 years ago

It's worth considering removing most of the default arguments for low-level functions. Having defaults increases the chances to miss a usage when changing defaults or adding / removing arguments.

We should keep reasonable defaults for high-level (user-facing) functions.