Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DataRequirementProcessor does not account for overloaded CQL functions while gathering the DataRequirements #1366

Open
adongare opened this issue Jun 3, 2024 · 0 comments
Labels
md impact tracking The issue is impacting measure development

Comments

@adongare
Copy link

adongare commented Jun 3, 2024

DataRequirementProcessor provides a config option named collapseDataRequirements to remove duplicates. However, it appears that while removing the duplicates, it doesn't consider the overloaded functions. Here's the piece of code that does the filtering

This makes effectiveDataRequirements incomplete. Also, effectiveDataRequirements are used to generate the Human readable in liquid templates, it misses out a bunch of overloaded functions. Could someone please take a look at this or provide any suggestions?

cc: @JSRankins

@jenseeman jenseeman added the md impact tracking The issue is impacting measure development label Jun 6, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
md impact tracking The issue is impacting measure development
Projects
None yet
Development

No branches or pull requests

2 participants