search

LSSTDESC / clmassmod

Cluster weak lensing mass modeling and verification code
3 stars 4 forks source link

Epic: Refactoring #1

Open aimalz opened 6 years ago

aimalz commented 6 years ago

Let's collect all issues related to the great refactoring of 2017-18 here!

matthewkirby commented 6 years ago

So since we are talking refactoring. Does it make sense to rename things to be consistent with major python packages (https://www.python.org/dev/peps/pep-0008/#naming-conventions)? The main takeaway being classes use camelcase and begin with a capital letter and functions use lowercase with underscores.

aimalz commented 6 years ago

@matthewkirby Thanks for bringing this up! Since we ultimately want to release a code that can be easily installed and used by those outside DESC, I think we should indeed try to stick to the standards. Let's keep the naming conventions in mind when reviewing pull requests.