awslabs / sagemaker-debugger

Amazon SageMaker Debugger provides functionality to save tensors during training of machine learning jobs and analyze those tensors
Apache License 2.0
161 stars 83 forks source link

Fix publik key string #530

Closed mariumof closed 2 years ago

mariumof commented 2 years ago

Description of changes:

Style and formatting:

I have run pre-commit install && pre-commit run --all-files 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-commenter commented 2 years ago

Codecov Report

Merging #530 (593ab5a) into master (efa0041) will decrease coverage by 4.92%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #530      +/-   ##
==========================================
- Coverage   72.95%   68.02%   -4.93%     
==========================================
  Files         127      117      -10     
  Lines       10917    10591     -326     
==========================================
- Hits         7964     7205     -759     
- Misses       2953     3386     +433     
Impacted Files Coverage Δ
smdebug/mxnet/__init__.py 0.00% <0.00%> (-100.00%) :arrow_down:
smdebug/mxnet/singleton_utils.py 0.00% <0.00%> (-100.00%) :arrow_down:
smdebug/mxnet/hook.py 0.00% <0.00%> (-83.34%) :arrow_down:
smdebug/mxnet/collection.py 0.00% <0.00%> (-76.67%) :arrow_down:
smdebug/rules/action/message_action.py 13.25% <0.00%> (-68.68%) :arrow_down:
smdebug/rules/action/action.py 30.61% <0.00%> (-61.23%) :arrow_down:
smdebug/tensorflow/reductions.py 0.00% <0.00%> (-47.06%) :arrow_down:
...iler/analysis/utils/pytorch_dataloader_analysis.py 0.00% <0.00%> (-44.95%) :arrow_down:
smdebug/rules/action/stop_training_action.py 12.50% <0.00%> (-42.19%) :arrow_down:
smdebug/core/modes.py 55.00% <0.00%> (-30.00%) :arrow_down:
... and 33 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 efa0041...593ab5a. Read the comment docs.

mariumof commented 2 years ago

Better run the tests locally