Skip to content

Conversation

@humanzz
Copy link
Contributor

@humanzz humanzz commented Aug 3, 2023

GitHub Issue #, if available: #2509

Note:

  • If merging this PR should also close the associated Issue, please also add that Issue # to the Linked Issues section on the right.

  • All PR's are checked weekly for staleness. This PR will be closed if not updated in 30 days.

Description

Tests

  • I wasn't able to follow through the contribution guide to identify/run relevant tests
  • I built/published the containers to my own repo using
python src/main.py --buildspec pytorch/inference/buildspec.yml \
                   --framework pytorch \
                   --image_types inference \
                   --device_types cpu \
                   --py_versions py3
python src/main.py --buildspec pytorch/inference/buildspec-graviton.yml \
                   --framework pytorch \
                   --image_types inference \
                   --device_types cpu \
                   --py_versions py3

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license. I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@humanzz humanzz requested review from a team as code owners August 3, 2023 08:35
@aws-deep-learning-containers-ci aws-deep-learning-containers-ci bot added build Reflects file change in build folder pytorch Reflects file change in pytorch folder Size:XS Determines the size of the PR labels Aug 3, 2023
@aws-deep-learning-containers-ci aws-deep-learning-containers-ci bot added the Size:S Determines the size of the PR label Aug 12, 2023
@arjkesh
Copy link
Contributor

arjkesh commented Aug 12, 2023

/rerun

@arjkesh
Copy link
Contributor

arjkesh commented Aug 13, 2023

/rerun

arjkesh
arjkesh previously approved these changes Aug 13, 2023
Copy link
Contributor

@arjkesh arjkesh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SM remote/local tests pass on PT 2.0.1 inference images on 607320a

@arjkesh
Copy link
Contributor

arjkesh commented Aug 14, 2023

/rerun

@arjkesh arjkesh merged commit 63321b8 into aws:master Aug 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build Reflects file change in build folder pytorch Reflects file change in pytorch folder Size:S Determines the size of the PR Size:XS Determines the size of the PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants