Skip to content

[pbi-fixer] Add fix_isavailable_in_mdx: set IsAvailableInMDX=False on hidden columns - #1206

Open
KornAlexander wants to merge 3 commits into
microsoft:mainfrom
KornAlexander:feature/fix-isavailable-in-mdx
Open

[pbi-fixer] Add fix_isavailable_in_mdx: set IsAvailableInMDX=False on hidden columns#1206
KornAlexander wants to merge 3 commits into
microsoft:mainfrom
KornAlexander:feature/fix-isavailable-in-mdx

Conversation

@KornAlexander

Copy link
Copy Markdown

Adds a sempy_labs fixer fix_isavailable_in_mdx() that set IsAvailableInMDX=False on hidden columns.

Adds one source file (src/sempy_labs/semantic_model/_Fix_IsAvailableInMdx.py) plus one re-export line in the matching __init__.py. No other files touched, no shared helpers, no dependency on any other PR.

Part of the PBI Fixer contribution. Tracking issue: #1185

Copilot AI review requested due to automatic review settings May 5, 2026 12:38

Copilot AI 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.

Pull request overview

Adds a new semantic model “fixer” function intended to enforce the BPA rule “Set IsAvailableInMdx to false on non-attribute columns” by updating IsAvailableInMDX on columns, and re-exports it from sempy_labs.semantic_model.

Changes:

  • Introduces fix_isavailable_in_mdx() to scan/update column IsAvailableInMDX flags via TOM.
  • Re-exports fix_isavailable_in_mdx from sempy_labs.semantic_model.__init__.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 6 comments.

File Description
src/sempy_labs/semantic_model/_Fix_IsAvailableInMdx.py New fixer implementation to update IsAvailableInMDX on columns.
src/sempy_labs/semantic_model/__init__.py Re-export of the new fixer function.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/sempy_labs/semantic_model/_Fix_IsAvailableInMdx.py
Comment thread src/sempy_labs/semantic_model/_Fix_IsAvailableInMdx.py
Comment thread src/sempy_labs/semantic_model/_Fix_IsAvailableInMdx.py Outdated
Comment thread src/sempy_labs/semantic_model/_Fix_IsAvailableInMdx.py Outdated
Comment thread src/sempy_labs/semantic_model/_Fix_IsAvailableInMdx.py
Comment thread src/sempy_labs/semantic_model/__init__.py Outdated
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.

2 participants