april-tools / cirkit

a python framework to build, learn and reason about probabilistic circuits and tensor networks
https://cirkit-docs.readthedocs.io/en/latest/
GNU General Public License v3.0
71 stars 1 forks source link

adjust `snake_case` to allow short names for vars #25

Closed lkct closed 1 year ago

lkct commented 1 year ago

affected: argument, attr, variable


By default, pylint only allows a name with at least length 3 for the snake_case style. However, based on context, short names can be reasonable. Also, long names can still be bad and cannot be detected by the default rules.

Therefore, it's decided to remove the constraint on the length of var names, and the authors should guarantee good naming in their code.