awslabs
[Do Not Merge] Temporarily adding logic to create sagemaker client with endpoint url
Description of changes:
Style and formatting:
I have run pre-commit install to ensure that auto-formatting happens with every commit.
Issue number, if available
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Codecov Report
Merging #389 into master will decrease coverage by
20.76%. The diff coverage is100.00%.
@@ Coverage Diff @@
## master #389 +/- ##
===========================================
- Coverage 85.49% 64.72% -20.77%
===========================================
Files 86 86
Lines 6514 6517 +3
===========================================
- Hits 5569 4218 -1351
- Misses 945 2299 +1354
| Impacted Files | Coverage Δ | |
|---|---|---|
| smdebug/rules/action/stop_training_action.py | 78.46% <100.00%> (+1.04%) |
:arrow_up: |
| smdebug/tensorflow/singleton_utils.py | 0.00% <0.00%> (-100.00%) |
:arrow_down: |
| smdebug/tensorflow/collection.py | 0.00% <0.00%> (-95.88%) |
:arrow_down: |
| smdebug/tensorflow/keras.py | 0.00% <0.00%> (-92.33%) |
:arrow_down: |
| smdebug/tensorflow/tensor_ref.py | 0.00% <0.00%> (-88.71%) |
:arrow_down: |
| smdebug/tensorflow/utils.py | 0.00% <0.00%> (-88.33%) |
:arrow_down: |
| smdebug/tensorflow/session.py | 6.25% <0.00%> (-85.58%) |
:arrow_down: |
| smdebug/tensorflow/base_hook.py | 2.38% <0.00%> (-78.18%) |
:arrow_down: |
| smdebug/tensorflow/version_check.py | 0.00% <0.00%> (-75.00%) |
:arrow_down: |
| smdebug/tensorflow/__init__.py | 37.50% <0.00%> (-62.50%) |
:arrow_down: |
| ... and 19 more |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing dataPowered by Codecov. Last update 311a6f4...1ea2597. Read the comment docs.
How is this logic being tested?
This change we may not want to merge. It made to enable the spot training testing.