dkpro-jwktl icon indicating copy to clipboard operation
dkpro-jwktl copied to clipboard

#66 Add inflection group to word form

Open highsource opened this issue 6 years ago • 2 comments

See #66.

This is a WIP PR, please do not merge yet.

highsource avatar Aug 12 '18 16:08 highsource

Codecov Report

Merging #67 into master will increase coverage by 0.05%. The diff coverage is 100%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #67      +/-   ##
============================================
+ Coverage     76.52%   76.57%   +0.05%     
+ Complexity     1352     1350       -2     
============================================
  Files           117      117              
  Lines          4749     4760      +11     
  Branches        798      799       +1     
============================================
+ Hits           3634     3645      +11     
  Misses          902      902              
  Partials        213      213
Impacted Files Coverage Δ Complexity Δ
...mstadt/ukp/jwktl/api/entry/WiktionaryWordForm.java 94.73% <100%> (+0.61%) 22 <3> (+1) :arrow_up:
...parser/de/components/nountable/EinzahlHandler.java 100% <100%> (ø) 3 <0> (ø) :arrow_down:
...arser/de/components/nountable/SingularHandler.java 100% <100%> (ø) 4 <0> (ø) :arrow_down:
.../parser/de/components/nountable/PluralHandler.java 100% <100%> (ø) 2 <0> (ø) :arrow_down:
...nountable/PatternBasedIndexedParameterHandler.java 94.11% <100%> (+4.11%) 4 <0> (+1) :arrow_up:
...mponents/nountable/DEWordFormNounTableHandler.java 92.85% <100%> (-1.59%) 7 <3> (-4)
...arser/de/components/nountable/MehrzahlHandler.java 100% <100%> (ø) 2 <0> (ø) :arrow_down:

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 0e09454...523dca4. Read the comment docs.

codecov-io avatar Aug 12 '18 16:08 codecov-io

This is done and could (theoretically) be merged. But it depends on #65, so we have to review/fix/merge that one first - and probably this PR will be influenced by that as well.

highsource avatar Aug 12 '18 20:08 highsource