Skip to content

Add Remove Unused Measure Function#1054

Open
DAXNoobJustin wants to merge 3 commits intomicrosoft:mainfrom
DAXNoobJustin:Unused-Report-Measure-PBIR-Function
Open

Add Remove Unused Measure Function#1054
DAXNoobJustin wants to merge 3 commits intomicrosoft:mainfrom
DAXNoobJustin:Unused-Report-Measure-PBIR-Function

Conversation

@DAXNoobJustin
Copy link

Adds a function that removes all unused measures from a pbir report

@m-kovalsky
Copy link
Collaborator

Thanks Justin. I will pause on this as the unused objects currently has some gaps.

@DAXNoobJustin
Copy link
Author

Hey Michael, thanks for the response. Can you share the gaps so I can address?

@m-kovalsky
Copy link
Collaborator

#759. I have a separate repo with a fix for this. However it’s a moving target as more functionality is added to pbi reporting.

@DAXNoobJustin
Copy link
Author

This PR specifically targets report level measures, not semantic model objects. The function goes through each report measure and checks to see if it is referenced.

Is that scenario in scope for your PR? It looks like yours is targeting unused semantic model objects, not unused report defined measures.

@DAXNoobJustin
Copy link
Author

Hey @m-kovalsky,

Just to clarify, I am not referencing the list_semantic_model_objects function. I am just targeting measures defined in the extensions file.

@m-kovalsky
Copy link
Collaborator

Yep I understand. I’m just holding on stuff related to the reportwrapper as I have a zillion changes in another repo on this and am still trying to determine the best way for it all to land.

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