USEPA / ElectricityLCI

Creative Commons Zero v1.0 Universal
24 stars 10 forks source link

Reaching the point of diminishing returns #206

Open dt-woods opened 9 months ago

dt-woods commented 9 months ago

There is a propensity for amazingly long names of methods and variables in this package. Unfortunately, these long names become quite cumbersome to use and difficult to read. Fortunately, good documentation can ameliorate the issue. While these isn't an exact rule for name length (e.g., in PEP8),

That being said, is there any opposition to renaming methods that are over ~70 characters long? This will at least give room for two 4-space indents + a trailing parenthesis.

So far, I found these culprits:

m-jamieson commented 9 months ago

I'm fine with making this change.