uclnlp / jack

Jack the Reader
MIT License
257 stars 82 forks source link

fix to instantiate memory map embeddings with the right format string #317

Closed undwie closed 6 years ago

codecov-io commented 6 years ago

Codecov Report

Merging #317 into master will increase coverage by 0.13%. The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #317      +/-   ##
==========================================
+ Coverage   63.48%   63.62%   +0.13%     
==========================================
  Files          72       72              
  Lines        4204     4231      +27     
==========================================
+ Hits         2669     2692      +23     
- Misses       1535     1539       +4
Impacted Files Coverage Δ
jack/io/embeddings/memory_map.py 68.29% <100%> (-31.71%) :arrow_down:
...tural_language_inference/decomposable_attention.py 82.82% <0%> (-13.67%) :arrow_down:
jack/readers/extractive_qa/shared.py 96.79% <0%> (-0.65%) :arrow_down:
jack/core/tensorport.py 94.52% <0%> (-0.15%) :arrow_down:
jack/util/hooks.py 53.9% <0%> (-0.12%) :arrow_down:
jack/readers/natural_language_inference/bilstm.py 100% <0%> (ø) :arrow_up:
jack/tfutil/xqa.py
jack/tfutil/activations.py
jack/tfutil/rnn.py
... and 22 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 e2a5d31...bf16476. Read the comment docs.