wellcometrust / deep_reference_parser

A deep learning model for extracting references from text
MIT License
24 stars 1 forks source link

Reorganise sub-module structure #19

Closed ivyleavedtoadflax closed 4 years ago

ivyleavedtoadflax commented 4 years ago

This is a general tidy up:

There are no substantive changes to the codebase, so if they tests pass, this should be good to go.

codecov-io commented 4 years ago

Codecov Report

Merging #19 into master will increase coverage by 3.24%. The diff coverage is 92.77%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #19      +/-   ##
==========================================
+ Coverage   81.31%   84.56%   +3.24%     
==========================================
  Files          14       16       +2     
  Lines        1247     1192      -55     
==========================================
- Hits         1014     1008       -6     
+ Misses        233      184      -49
Impacted Files Coverage Δ
deep_reference_parser/reference_utils.py 100% <ø> (+35.94%) :arrow_up:
deep_reference_parser/deep_reference_parser.py 87.87% <100%> (-0.05%) :arrow_down:
deep_reference_parser/prodigy/labels_to_prodigy.py 100% <100%> (ø)
deep_reference_parser/prodigy/prodigy_to_tsv.py 97.47% <100%> (ø) :arrow_up:
deep_reference_parser/prodigy/misc.py 68.75% <68.75%> (ø)
deep_reference_parser/io/io.py 96.87% <97.82%> (+2.43%) :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 085f0fb...2e7da2e. Read the comment docs.

nsorros commented 4 years ago

On a separate note from this PR, make test should depend on virtualenv/.installed