Closed gueguenster closed 3 years ago
Merging #627 (dedb6b0) into master (f8c427c) will increase coverage by
0.01%
. The diff coverage is100.00%
.
@@ Coverage Diff @@
## master #627 +/- ##
==========================================
+ Coverage 85.31% 85.32% +0.01%
==========================================
Files 85 85
Lines 4929 4933 +4
Branches 783 783
==========================================
+ Hits 4205 4209 +4
Misses 584 584
Partials 140 140
Impacted Files | Coverage Δ | |
---|---|---|
petastorm/pytorch.py | 90.72% <100.00%> (ø) |
|
petastorm/weighted_sampling_reader.py | 84.78% <100.00%> (+1.44%) |
: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 f8c427c...dedb6b0. Read the comment docs.
petastorm.pytorch.DataLoader expects the field is_batched_reader in the reader. WeightedSamplingReader does not have this field. It seems to be replaced by the property: batched_output.
petastorm.pytorch.DataLoader expects the functions stop() and join() from the reader. WeightedSamplingReader does not have these functions. This PR adds these functions.