Closed WeichenXu123 closed 4 years ago
Merging #506 into master will increase coverage by
0.03%
. The diff coverage is86.00%
.
@@ Coverage Diff @@
## master #506 +/- ##
==========================================
+ Coverage 85.98% 86.02% +0.03%
==========================================
Files 81 81
Lines 4311 4402 +91
Branches 674 704 +30
==========================================
+ Hits 3707 3787 +80
- Misses 499 504 +5
- Partials 105 111 +6
Impacted Files | Coverage Δ | |
---|---|---|
petastorm/spark/spark_dataset_converter.py | 90.62% <86.00%> (-2.66%) |
:arrow_down: |
petastorm/codecs.py | 77.77% <0.00%> (-1.17%) |
:arrow_down: |
petastorm/arrow_reader_worker.py | 92.00% <0.00%> (ø) |
|
petastorm/etl/dataset_metadata.py | 88.88% <0.00%> (ø) |
|
petastorm/reader.py | 90.99% <0.00%> (+0.17%) |
:arrow_up: |
petastorm/fs_utils.py | 91.01% <0.00%> (+2.27%) |
: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 e3acecf...0d704af. Read the comment docs.
@selitvin Ready.
Note: the unittest.mock
has bug on python3 (some version) which make CI raise error. So I import built-in mock first and if failed fallback to unittest mock.
try:
from mock import mock
except:
from unitest import mock
What changes are proposed in this PR?
Add
converter.make_tf_dataset()
with advanced params.API
Example code