sagemaker-debugger icon indicating copy to clipboard operation
sagemaker-debugger copied to clipboard

[Do-Not-Merge] Ignore sagemaker-uploading

Open NihalHarish opened this issue 4 years ago • 1 comments

Description of changes:

  • We should ignore special status files that can be added by Sagemaker before determining if we should update the checkpointing directory.

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.

NihalHarish avatar Mar 23 '21 05:03 NihalHarish

Codecov Report

Merging #470 (2897643) into master (1aac94b) will decrease coverage by 15.52%. The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #470       +/-   ##
===========================================
- Coverage   65.53%   50.00%   -15.53%     
===========================================
  Files         173      163       -10     
  Lines       13280    12934      -346     
===========================================
- Hits         8703     6468     -2235     
- Misses       4577     6466     +1889     
Impacted Files Coverage Δ
smdebug/core/state_store.py 77.58% <ø> (-3.45%) :arrow_down:
smdebug/_version.py 100.00% <100.00%> (ø)
smdebug/tensorflow/__init__.py 0.00% <0.00%> (-100.00%) :arrow_down:
smdebug/tensorflow/constants.py 0.00% <0.00%> (-100.00%) :arrow_down:
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/session.py 0.00% <0.00%> (-91.83%) :arrow_down:
smdebug/tensorflow/keras.py 0.00% <0.00%> (-89.30%) :arrow_down:
smdebug/tensorflow/tensor_ref.py 0.00% <0.00%> (-88.71%) :arrow_down:
smdebug/tensorflow/utils.py 0.00% <0.00%> (-86.26%) :arrow_down:
... and 57 more

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 1aac94b...2897643. Read the comment docs.

codecov-io avatar Mar 23 '21 05:03 codecov-io