Skip to content

fix(coreai_utils): re-export QScheme at package level (#115) - #116

Merged
dengqiaoyu merged 1 commit into
apple:mainfrom
rohith500:fix/coreai-utils-export-qscheme
Sep 30, 2026
Merged

dengqiaoyu merged 1 commit into
apple:mainfrom
rohith500:fix/coreai-utils-export-qscheme

Conversation

@rohith500

@rohith500 rohith500 commented Sep 22, 2026 •

Copy link
Copy Markdown
Contributor

Re-export QScheme from coreai_opt.coreai_utils alongside CompressionGranularity and DType.

This resolves #115 by fixing an ImportError when executing the documented example in docs/src/utils/coreai_compression.md#L92-L97.

@dengqiaoyu dengqiaoyu left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks for the changes! Let's keep the fix simple by just re-export it.

Comment thread changelog.d/115.fixed Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No need to add change for such as simple fix.

Comment thread tests/coreai_utils/test_exports.py Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No need to add test for this, let's keep it simple.

Re-export QScheme from coreai_opt.coreai_utils alongside CompressionGranularity
and DType. This resolves an ImportError when executing the documented example
in docs/src/utils/coreai_compression.md#L92-L97 and eliminates the orphaned
coreai_opt.coreai_utils.common section in auto-generated API documentation.

Closes apple#115
@rohith500
rohith500 force-pushed the fix/coreai-utils-export-qscheme branch from 16dc00b to 71d0d43 Compare September 30, 2026 17:01
@rohith500

Copy link
Copy Markdown
Contributor Author

Updated to remove the test and changelog fragment, leaving only the re-export in src/coreai_opt/coreai_utils/__init__.py. Thanks for the review!

@dengqiaoyu
dengqiaoyu enabled auto-merge (squash) September 30, 2026 17:28
@dengqiaoyu
dengqiaoyu merged commit 8f55f30 into apple:main Sep 30, 2026
14 checks passed
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.

[Bug]: Cannot import QScheme from coreai_opt.coreai_utils as documented in coreai_compression.md

2 participants