search

gwnrtools / nr-catalog-tools

A unified interface to various catalogs of Numerical Relativity simulations of compact binary mergers.
https://github.com/gwnrtools/nr-catalog-tools
GNU General Public License v3.0
1 stars 5 forks source link

Uniform Formatting and PEP8 consistency #9

Closed adivijaykumar closed 1 year ago

adivijaykumar commented 1 year ago

I saw a lot of flake8 failures and also noted that there is no uniform formatting across the package. For the latter, I prefer black, but I know people have major disagreements there. In any case, thoughts on these?

I can volunteer to put in a PR implementing these (possibly also adding flake8 checks to CI).

Pinging @prayush on this.

prayush commented 1 year ago

Yes, please go ahead @adivijaykumar . I've been using yapf, but probably not uniformly across the package.