search

AllenInstitute / em_stitch

EM microscopy lens correction
BSD 2-Clause "Simplified" License
4 stars 3 forks source link

first test for montage solve #26

Closed djkapner closed 5 years ago

pep8speaks commented 5 years ago

Hello @djkapner! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:

Comment last updated at 2019-04-18 23:26:00 UTC
codecov-io commented 5 years ago

Codecov Report

Merging #26 into develop will increase coverage by 4.02%. The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop      #26      +/-   ##
===========================================
+ Coverage    72.28%   76.31%   +4.02%     
===========================================
  Files           11       14       +3     
  Lines          837     1068     +231     
===========================================
+ Hits           605      815     +210     
- Misses         232      253      +21
Impacted Files Coverage Δ
on_scope/montage/montage_solver.py 88.77% <100%> (ø)
on_scope/montage/meta_to_collection.py 83.78% <0%> (ø)
on_scope/montage/schemas.py 95.45% <0%> (ø)
on_scope/utils/utils.py 95.74% <0%> (+19.14%) :arrow_up:

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 128ac16...a2f216f. Read the comment docs.