text icon indicating copy to clipboard operation
text copied to clipboard

AG NEWS iterable dataset offsets

Open salilsdesai opened this issue 4 years ago • 2 comments

Added offsets for AG NEWS iterable

salilsdesai avatar Jun 17 '21 16:06 salilsdesai

Codecov Report

Merging #1338 (0e7966b) into master (0435df1) will decrease coverage by 0.27%. The diff coverage is 46.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1338      +/-   ##
==========================================
- Coverage   76.85%   76.58%   -0.28%     
==========================================
  Files          63       63              
  Lines        3275     3301      +26     
==========================================
+ Hits         2517     2528      +11     
- Misses        758      773      +15     
Impacted Files Coverage Δ
torchtext/data/datasets_utils.py 83.63% <42.30%> (-5.60%) :arrow_down:
torchtext/datasets/ag_news.py 100.00% <100.00%> (ø)

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 0435df1...0e7966b. Read the comment docs.

codecov[bot] avatar Jun 28 '21 17:06 codecov[bot]

Codecov Report

Merging #1338 (0e7966b) into master (0435df1) will decrease coverage by 0.27%. The diff coverage is 46.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1338      +/-   ##
==========================================
- Coverage   76.85%   76.58%   -0.28%     
==========================================
  Files          63       63              
  Lines        3275     3301      +26     
==========================================
+ Hits         2517     2528      +11     
- Misses        758      773      +15     

Impacted Files Coverage Δ torchtext/data/datasets_utils.py 83.63% <42.30%> (-5.60%) ⬇️ torchtext/datasets/ag_news.py 100.00% <100.00%> (ø) 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 0435df1...0e7966b. Read the comment docs.

This is because I used this function to generate the offsets and hardcoded them into the AG NEWS dataset file, but the function isn't actually run otherwise. Should I remove it from the code, or add a test for it, or do something else so it is used?

salilsdesai avatar Jun 28 '21 18:06 salilsdesai

Stale.

joecummings avatar Mar 09 '23 16:03 joecummings