sagemaker-mxnet-training-toolkit icon indicating copy to clipboard operation
sagemaker-mxnet-training-toolkit copied to clipboard

Recycling old, unused files

Open ChaiBapchya opened this issue 5 years ago • 4 comments

Since the dockerfiles, tests have been migrated to https://github.com/aws/deep-learning-containers

We should clean up the redundant [and hence confusing] files such as

  • Dockerfiles
  • test/

ChaiBapchya avatar Jun 11 '20 00:06 ChaiBapchya

@saimidu FYI

ChaiBapchya avatar Jun 11 '20 00:06 ChaiBapchya

Thanks @ChaiBapchya. All the redundant files will be deleted as soon as the migration of the dockerfiles for older framework versions is complete. Thank you for raising this issue.

saimidu avatar Jun 11 '20 00:06 saimidu

Hi @ChaiBapchya, we will remove the DLC docker images once all existing container versions are migrated.

chuyang-deng avatar Jun 11 '20 07:06 chuyang-deng

Hi,

We are still in the process of migration/separation plan of DLC and sagemaker toolkits that has been happening in the last few months.

We already cleaned up tests 10 days ago: https://github.com/aws/sagemaker-mxnet-training-toolkit/commit/4a647a964551f4ff3f3b920811b49f375c4c2e51

And, as part of the plan, will be keeping dockerfiles until all of them are fully migrated to the dlc repo. They have been put in read only mode and changes to the dockerfiles are no longer accepted on this repo.

nadiaya avatar Jun 11 '20 16:06 nadiaya