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
158 stars 82 forks source link

Moved framework hook checks to zero_change #536

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 #536 (afca683) into master (6903873) will decrease coverage by 16.02%. The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           master     #536       +/-   ##
===========================================
- Coverage   75.53%   59.51%   -16.03%     
===========================================
  Files         127      117       -10     
  Lines       10917    10591      -326     
===========================================
- Hits         8246     6303     -1943     
- Misses       2671     4288     +1617     
Impacted Files Coverage Δ
smdebug/pytorch/__init__.py 0.00% <0.00%> (-100.00%) :arrow_down:
smdebug/pytorch/singleton_utils.py 0.00% <0.00%> (-100.00%) :arrow_down:
...ug/profiler/analysis/utils/pandas_data_analysis.py 0.00% <0.00%> (-98.43%) :arrow_down:
smdebug/profiler/analysis/utils/merge_timelines.py 0.00% <0.00%> (-97.52%) :arrow_down:
smdebug/profiler/analysis/python_stats_reader.py 0.00% <0.00%> (-95.72%) :arrow_down:
...debug/profiler/analysis/python_profile_analysis.py 0.00% <0.00%> (-90.91%) :arrow_down:
smdebug/pytorch/collection.py 0.00% <0.00%> (-90.00%) :arrow_down:
...er/analysis/utils/python_profile_analysis_utils.py 0.00% <0.00%> (-88.41%) :arrow_down:
smdebug/pytorch/utils.py 0.00% <0.00%> (-83.73%) :arrow_down:
smdebug/pytorch/hook.py 0.00% <0.00%> (-78.53%) :arrow_down:
... and 43 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 6903873...afca683. Read the comment docs.