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

Update merged timeline

Open NRauschmayr opened this issue 4 years ago • 2 comments

Description of changes:

I added the PyTorch dataloader events to the merged time line.

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.

NRauschmayr avatar Apr 08 '21 03:04 NRauschmayr

Codecov Report

Merging #480 (59b151a) into master (a7c697f) will decrease coverage by 0.67%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #480      +/-   ##
==========================================
- Coverage   66.88%   66.20%   -0.68%     
==========================================
  Files         173      163      -10     
  Lines       13280    12934     -346     
==========================================
- Hits         8882     8563     -319     
+ Misses       4398     4371      -27     
Impacted Files Coverage Δ
smdebug/profiler/analysis/utils/merge_timelines.py 97.51% <ø> (ø)
smdebug/core/modes.py 55.00% <0.00%> (-20.00%) :arrow_down:
smdebug/xgboost/singleton_utils.py 0.00% <0.00%> (-20.00%) :arrow_down:
smdebug/trials/profiler_trial.py 24.61% <0.00%> (-18.47%) :arrow_down:
smdebug/mxnet/collection.py 73.33% <0.00%> (-16.67%) :arrow_down:
smdebug/exceptions.py 65.47% <0.00%> (-15.48%) :arrow_down:
smdebug/core/reduction_config.py 88.31% <0.00%> (-7.80%) :arrow_down:
smdebug/core/reader.py 85.18% <0.00%> (-7.41%) :arrow_down:
smdebug/tensorflow/callable_cache.py 82.60% <0.00%> (-6.53%) :arrow_down:
smdebug/core/singleton_utils.py 85.29% <0.00%> (-5.89%) :arrow_down:
... and 20 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 a7c697f...59b151a. Read the comment docs.

codecov-io avatar Apr 08 '21 04:04 codecov-io

Codecov Report

Merging #480 (682a85e) into master (f841a8b) will decrease coverage by 1.12%. The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #480      +/-   ##
==========================================
- Coverage   75.74%   74.61%   -1.13%     
==========================================
  Files         126      116      -10     
  Lines       10872    10526     -346     
==========================================
- Hits         8235     7854     -381     
- Misses       2637     2672      +35     
Impacted Files Coverage Δ
smdebug/profiler/analysis/utils/merge_timelines.py 97.51% <ø> (ø)
smdebug/pytorch/utils.py 45.23% <0.00%> (-35.72%) :arrow_down:
smdebug/core/reduction_config.py 88.31% <0.00%> (-7.80%) :arrow_down:
smdebug/core/singleton_utils.py 79.41% <0.00%> (-5.89%) :arrow_down:
smdebug/profiler/hvd_trace_file_rotation.py 89.39% <0.00%> (-4.55%) :arrow_down:
smdebug/core/logger.py 76.38% <0.00%> (-2.78%) :arrow_down:
smdebug/core/utils.py 80.11% <0.00%> (-1.17%) :arrow_down:
smdebug/pytorch/hook.py 77.88% <0.00%> (-0.65%) :arrow_down:
smdebug/tensorflow/base_hook.py 74.81% <0.00%> (-0.37%) :arrow_down:
smdebug/core/tfevent/proto/types_pb2.py
... and 9 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 f841a8b...682a85e. Read the comment docs.

codecov-commenter avatar Apr 22 '21 05:04 codecov-commenter