Skip to content

Fix/skip none hyperparameters#5775

Open
Harsh270519 wants to merge 1 commit intoaws:masterfrom
Harsh270519:fix/skip-none-hyperparameters
Open

Fix/skip none hyperparameters#5775
Harsh270519 wants to merge 1 commit intoaws:masterfrom
Harsh270519:fix/skip-none-hyperparameters

Conversation

@Harsh270519
Copy link
Copy Markdown

Issue #, if available:
to_dict() converts Python None to the string "None" via str(), which gets sent as a hyperparameter value to the training container causing failures (e.g., resume_from_path="None" triggers invalid checkpoint resume). This fix omits None-valued hyperparameters instead of serializing them.

Description of changes:
Changes in common.py

###Testing
Launched the jobs via the notebooks : - https://github.com/aws/sagemaker-python-sdk/tree/master/v3-examples/model-customization-examples

https://tiny.amazon.com/dc7sqv9n/IsenLink
https://tiny.amazon.com/tpg255wa/IsenLink
https://tiny.amazon.com/1437xy0rb/IsenLink

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

@Harsh270519 Harsh270519 deployed to manual-approval April 17, 2026 22:36 — with GitHub Actions Active
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant