Nihal Harish comments

Results 7 comments of


                                            Nihal Harish

[Do Not Merge] Temporarily adding logic to create sagemaker client with endpoint url

How is this logic being tested?

Enable SMDDP ZCC tests

Remove tests if they cannot run on codebuild

Updated distributed_training.md

> @NihalHarish Please make the changes that we discussed last week offline. Changes discussed offline: Remove ZCC examples that need additional configuration,

TypeError: get() takes no keyword arguments - breaks training jobs

We have filed an issue here: https://github.com/huggingface/datasets/issues/2115

Version 0.9.1 makes saving a tf model fail with KeyError: "callable_inputs"

This issue has been addressed in the PR: https://github.com/awslabs/sagemaker-debugger/pull/333

Reduce ZCC Runtimes

Leaving this PR open. The performance concern was addressed in https://github.com/awslabs/sagemaker-debugger/pull/316

[DO Not Merge] Avoid using 'default' keyword while invoking 'get' method

Let's not merge this. If this is the correct way to handle env variables then be it.