fix: include sagemaker.mlops subpackages in sagemaker-mlops wheel (Fixes #5612)#5647
fix: include sagemaker.mlops subpackages in sagemaker-mlops wheel (Fixes #5612)#5647junagent wants to merge 1 commit intoaws:masterfrom
Conversation
94d5465 to
c12dfd0
Compare
aws#5612) Set namespaces=false to ensure subpackages sagemaker.mlops, sagemaker.mlops.feature_store, and sagemaker.mlops.feature_store.feature_processor are included in the wheel distribution. Previously, namespaces=true caused setuptools to treat sagemaker as a PEP 420 namespace package, excluding all subdirectories from the wheel. Fixes aws#5612
c12dfd0 to
f537dd7
Compare
Bug Label RequestThis fix addresses a critical distribution regression (Issue #5612) where Impact: Users who install Root cause: Fix verification:
Requesting a |
Summary
Set
namespaces=falsein[tool.setuptools.packages.find]to ensure subpackagessagemaker.mlops,sagemaker.mlops.feature_store, andsagemaker.mlops.feature_store.feature_processorare included in the wheel distribution.Root Cause
The
sagemaker-mlopswheel was missing themlopssubpackage tree becausenamespaces=true(PEP 420 namespace package mode) caused setuptools to exclude subdirectories from the wheel.Fix
Changed in
sagemaker-mlops/pyproject.toml:With
namespaces=false, setuptools treatssagemakeras a regular package and correctly includes all discovered subpackages:mlops,mlops.feature_store,mlops.feature_store.feature_processor.Testing
After this fix, the wheel should include:
sagemaker_mlops-*/sagemaker/__init__.pysagemaker_mlops-*/sagemaker/mlops/__init__.pysagemaker_mlops-*/sagemaker/mlops/feature_store/__init__.pysagemaker_mlops-*/sagemaker/mlops/feature_store/feature_processor/__init__.py.pymodulesFixes #5612