Borda / BIRL

BIRL: Benchmark on Image Registration methods with Landmark validations

http://borda.github.io/BIRL

BSD 3-Clause "New" or "Revised" License

90 stars 26 forks source link

Refactor: more deepsource suggestions #62

Closed Borda closed 3 years ago

Borda commented 3 years ago

Before submitting

[x] Was this discussed/approved via a Github issue? (no need for typos, doc improvements)
[x] My code follows the style guidelines of this project, see the contributor guideline
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes

What does this PR do?

Fixes # (issue).

Type of change

Bug fix (non-breaking change which fixes an issue)
New feature (non-breaking change which adds functionality)
Breaking change (fix or feature that would cause existing functionality to not work as expected)
This change requires a documentation update

PR review

Anyone in the community is free to review the PR once the tests have passed. If we didn't discuss your PR in Github issues there's a high chance it will not be merged.

Did you have fun?

Make sure you had fun coding 🙃

codecov[bot] commented 3 years ago

Codecov Report

Merging #62 (049dac3) into master (4013b29) will decrease coverage by 0%. The diff coverage is 83%.

@@          Coverage Diff          @@
##           master    #62   +/-   ##
=====================================
- Coverage      93%    93%   -0%     
=====================================
  Files          10     10           
  Lines        1746   1746           
=====================================
- Hits         1626   1625    -1     
- Misses        120    121    +1