-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Py312 upgrade step 2: Update dependencies, integ tests and unit tests #5123
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Merged
zhaoqizqwang
merged 6 commits into
aws:master
from
mollyheamazon:origin/master-ci-py312-squash
Apr 19, 2025
Merged
Py312 upgrade step 2: Update dependencies, integ tests and unit tests #5123
zhaoqizqwang
merged 6 commits into
aws:master
from
mollyheamazon:origin/master-ci-py312-squash
Apr 19, 2025
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Can we update branch to rebase with master |
f2c0c41
to
53b7f72
Compare
benieric
reviewed
Apr 18, 2025
benieric
reviewed
Apr 18, 2025
0356caf
to
00741ed
Compare
benieric
reviewed
Apr 18, 2025
benieric
previously approved these changes
Apr 18, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
benieric
approved these changes
Apr 18, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
author Molly He [email protected] 1740529460 -0800 committer Molly He [email protected] 1744156034 -0700
parent fb22b91
author Molly He [email protected] 1740529460 -0800 committer Molly He [email protected] 1744155953 -0700
parent fb22b91
author Molly He [email protected] 1740529460 -0800 committer Molly He [email protected] 1744155703 -0700
parent fb22b91
author Molly He [email protected] 1740529460 -0800 committer Molly He [email protected] 1744155683 -0700
Test py312 ci support
Test py312 ci support
first commit
test to point to personal stack
changed tox.ini
Revert "changed tox.ini"
This reverts commit 696cb47.
Revert "Revert "changed tox.ini""
This reverts commit 5d35bd0.
Revert "Revert "Revert "changed tox.ini"""
This reverts commit 2ab95fb.
Revert "Revert "changed tox.ini""
This reverts commit 5d35bd0.
Revert "changed tox.ini"
This reverts commit 696cb47.
Revert "test to point to personal stack"
This reverts commit 49de844.
Revert "first commit"
This reverts commit 1374184.
add pyproject.toml
add py312 to ci
bump numpy version
numpy version change
add py312
upgrade pip version
add setuptools wheel to tox.ini
add pyyaml version constraint, remove py312 from docstring because there is no py312 image yet
update pyyaml version constraint
update pyyaml version constraint
deprecate py38
bump scipy
bump tensorflow and tensorboard
bump dill
bump apache-airflow to ensure dill and greenlet version
remove constraint for apache-airflow
remove constraint for apache-airflow
bump torch version
bump torchvision version
new tests
try changing some tests
update model trainer test
fix test_pipeline
add constraint to apache in tox
Fix key error in _send_metrics() (#5068)
Co-authored-by: pintaoz [email protected]
Use sagemaker session's s3_resource in download_folder (#5064)
Co-authored-by: pintaoz [email protected]
Fix error when there is no session to call _create_model_request() (#5062)
Fix error when there is no session to call _create_model_request()
Fix codestyle
Co-authored-by: pintaoz [email protected]
Ensure Model.is_repack() returns a boolean (#5060)
Ensure Model.is_repack() returns a boolean
update test
Co-authored-by: pintaoz [email protected]
feat: Make DistributedConfig Extensible (#5039)
feat: Make DistributedConfig Extensible
pylint
Include none types when creating config jsons for safer reference
fix: update test to account for changes
format
Add integ test
pylint
prepare release v2.240.0
update development version to v2.240.1.dev0
Fix key error in _send_metrics() (Fix key error in _send_metrics() #5068)
Co-authored-by: pintaoz [email protected]
Co-authored-by: Keshav Chandak [email protected]
Co-authored-by: pintaoz [email protected]
remove union
fix merge artifact
Change dir path to distributed_drivers
update paths
Co-authored-by: ci
Co-authored-by: pintaoz-aws [email protected]
Co-authored-by: pintaoz [email protected]
Co-authored-by: Keshav Chandak [email protected]
Co-authored-by: Keshav Chandak [email protected]
Skip tests with deprecated instance type (#5077)
Co-authored-by: pintaoz [email protected]
pipeline definition function doc update (#5074)
Co-authored-by: Rohan Gujarathi [email protected]
feat: add integ tests for training JumpStart models in private hub (#5076)
feat: add integ tests for training JumpStart models in private hub
fixed formatting
remove unused imports
fix unused imports
fix unit test failure and fix bug around versioning
fix formatting
fix unit tests
fix model_uri usage issue
fix some formatting
separate private hub setup code
add try catch block
fix flake8 issue so except clause is not bare
black formatting
fix: resolve infinite loop in _find_config on Windows systems (#4970)
fix: resolve Windows path handling in _find_config
Replace Path.match("/") with Path.anchor comparison
Fix infinite loop in _studio.py path traversal
test: Add tests for the new root path exploration
Fix formatting style
Fixed line to long
Fix docstyle by running black manually
Fix testcase with \ when running on non-windows machines
Fix formatting style
cleanup unused import
change: update image_uri_configs 03-11-2025 07:18:09 PST
Fixing Pytorch training python version in tests (#5084)
Fixing Pytorch training python version in tests
Updating Inference test handling
remove s3 output location requirement from hub class init (#5081)
remove s3 output location requirement from hub class init
fix integ test hub
lint
fix test
Co-authored-by: Gokul Anantha Narayanan [email protected]
fix: Prevent RunContext overlap between test_run tests (#5083)
Co-authored-by: Gokul Anantha Narayanan [email protected]
remove py38 from unit testing
fix integ test by bumping py38 to py39 for PyTorch
change framework_version that supports py39 in integ tests
remove py38 from unit testing
Update estimator.py
Issue #, if available:
Description of changes:
Testing done:
Merge Checklist
Put an
x
in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your pull request.General
Tests
unique_name_from_base
to create resource names in integ tests (if appropriate)By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.