Repository navigation
fix: Transformer tags pydantic validation and shape compatibility - #6103
wasim-builds wants to merge 3 commits into
Conversation
|
By the way, I've really been enjoying contributing to this project! My goal is to get more involved in open source to help maintain projects and learn from great engineering teams. Is there a process or pathway for becoming a member of the aws GitHub organization? I'd love to continue helping out where I can! |
|
Hey 👋 Just a friendly ping on this one — fixes Transformer tags pydantic validation and shape compatibility. Ready for review when you get a chance! 🙏 |
08549a1 to
2c6b973
Compare
|
Hey 👋 Just a friendly bump — this fixes the Transformer tags pydantic validation error and shape compatibility issue. Ready for review whenever you have a moment! 🙏 |
|
Greetings maintainers, Following up on this proposed change (fix: Transformer tags pydantic validation and shape compatibility) in aws/sagemaker-python-sdk. Working on Amazon SageMaker Python SDK machine learning and model deployment pipelines has been a great experience, and I am very keen to continue contributing to aws/sagemaker-python-sdk. I am also open to collaborating on a contract/freelance basis, discussing full-time opportunities, or joining the organization/team. You can view my background and other open-source contributions at https://github.com/wasim-builds. |
|
Issue has been closed. Please reopen if you have any concerns. Thanks |
Fixes #6090
Problem
Transformer.transform()has two tag-handling issues insagemaker-core:pydantic.ValidationError(extra_forbidden) becausetags/Tagsare passed through toTransformJob, which does not accept extra fields.[{"key": "foo", "value": "bar"}]instead of the canonical[{"Key": "foo", "Value": "bar"}].Fix
tags/Tagsfrom the transformed dict before constructingTransformJob.Key/Valuebefore callingCreateTransformJob.Scope
sagemaker-core/src/sagemaker/core/transformer.pyandsagemaker-core/src/sagemaker/core/clarify/__init__.py.Verification
TransformJobcreation now succeeds with tags in either casing, and pydantic validation no longer raisesextra_forbidden.