Initial integration with AutoDist#72
Conversation
| WORKDIR /root | ||
| COPY bert_config.json bert_config.json | ||
| COPY tf_examples.tfrecord tf_examples.tfrecord | ||
| COPY autodist autodist |
There was a problem hiding this comment.
Some of these COPY commands cannot work in a fresh clone of the AdaptDL repo? Can you make sure this example can work in that setting? Maybe git clone autodist instead of assuming it exists locally?
Codecov Report
@@ Coverage Diff @@
## master #72 +/- ##
==========================================
+ Coverage 61.16% 61.18% +0.01%
==========================================
Files 30 30
Lines 2243 2334 +91
Branches 338 357 +19
==========================================
+ Hits 1372 1428 +56
- Misses 806 834 +28
- Partials 65 72 +7
Continue to review full report at Codecov.
|
| return_list = [(pod_ip_list[i], pod_gpu_list[i]) | ||
| for i in range(len(pod_ip_list))] | ||
| LOG.info(return_list) | ||
| return web.json_response(return_list) |
There was a problem hiding this comment.
shall we unify the return values with L87?
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #72 +/- ##
==========================================
+ Coverage 61.16% 61.18% +0.01%
==========================================
Files 30 30
Lines 2243 2334 +91
Branches 338 357 +19
==========================================
+ Hits 1372 1428 +56
- Misses 806 834 +28
- Partials 65 72 +7 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
5 participants
No description provided.