grimme-lab / CENSO

CENSO - Commandline ENergetic SOrting of Conformer Rotamer Ensembles
https://xtb-docs.readthedocs.io/en/latest/CENSO_docs/censo.html
GNU Lesser General Public License v3.0
22 stars 9 forks source link

Default argument mutation #7

Closed awvwgk closed 3 years ago

awvwgk commented 3 years ago

Checking the CENSO code base on LGTM flagged a recurring pattern of default argument mutation (see https://lgtm.com/projects/g/grimme-lab/CENSO/?mode=list&severity=error). With 44 instances flagged this could become a potentially hard to debug error at some point. It doesn't seem to case any issues yet, but it is probably worth looking out for.

awvwgk commented 3 years ago

Fixed with #8