Closed WeichenXu123 closed 4 years ago
Merging #496 into master will increase coverage by
0.21%
. The diff coverage is93.38%
.
@@ Coverage Diff @@
## master #496 +/- ##
==========================================
+ Coverage 85.77% 85.98% +0.21%
==========================================
Files 79 81 +2
Lines 4190 4311 +121
Branches 665 674 +9
==========================================
+ Hits 3594 3707 +113
- Misses 494 499 +5
- Partials 102 105 +3
Impacted Files | Coverage Δ | |
---|---|---|
petastorm/spark/__init__.py | 100% <100%> (ø) |
|
petastorm/spark/spark_dataset_converter.py | 93.27% <93.27%> (ø) |
|
petastorm/unischema.py | 94.58% <0%> (ø) |
: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 b3120b1...fa8ebb6. Read the comment docs.
@tgaddair @selitvin Address all comments. Ready for review. :)
@selitvin Ready for review.
@selitvin Ready.
What does this PR do?
This is the first PR for project: Simplify data conversion from Spark to TensorFlow. This PR include basic implementation, including basic API for spark dataset converter.
API
Example code:
Follow-up PRs
make_tf_dataset
, including:SparkDatasetConverter.make_torch_data_loader