search

rs-station / reciprocalspaceship

Tools for exploring reciprocal space
https://rs-station.github.io/reciprocalspaceship/
MIT License
28 stars 12 forks source link

Standardize code style using black #86

Closed JBGreisman closed 3 years ago

JBGreisman commented 3 years ago

This is an initial look at using black to standardize the code style in rs. This initial set of commits just runs black on the existing code base.

To do:

codecov-commenter commented 3 years ago

Codecov Report

Merging #86 (ad2ff9a) into main (1555efa) will decrease coverage by 0.05%. The diff coverage is 98.80%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #86      +/-   ##
==========================================
- Coverage   98.92%   98.87%   -0.06%     
==========================================
  Files          40       40              
  Lines        1495     1514      +19     
==========================================
+ Hits         1479     1497      +18     
- Misses         16       17       +1
Flag Coverage Δ
unittests 98.87% <98.80%> (-0.06%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
reciprocalspaceship/dtypes/summarize.py 100.00% <ø> (ø)
reciprocalspaceship/io/pickle.py 100.00% <ø> (ø)
reciprocalspaceship/utils/units.py 80.00% <ø> (ø)
reciprocalspaceship/dataset.py 98.84% <95.94%> (-0.24%) :arrow_down:
reciprocalspaceship/io/crystfel.py 94.77% <96.29%> (ø)
reciprocalspaceship/__init__.py 100.00% <100.00%> (ø)
reciprocalspaceship/algorithms/__init__.py 100.00% <100.00%> (ø)
reciprocalspaceship/algorithms/merge.py 100.00% <100.00%> (ø)
...alspaceship/algorithms/scale_merged_intensities.py 100.00% <100.00%> (ø)
reciprocalspaceship/concat.py 100.00% <100.00%> (ø)
... and 30 more

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update 1555efa...ad2ff9a. Read the comment docs.

kmdalton commented 3 years ago

I haven't got any issues with this. As we discussed offline, please double check that there is a snippet in the dev docs with a good default "recipe" for installing by pip -e with pre-commit hooks. Looks good.