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

Consolidate CQL datatypes #2926

Open
ewoutkramer opened this issue Oct 18, 2024 · 0 comments
Open

Consolidate CQL datatypes #2926

ewoutkramer opened this issue Oct 18, 2024 · 0 comments

Comments

@ewoutkramer
Copy link
Member

Both our FHIR and CQL SDK have a set of CQL datatypes, that are covering exactly the same ground. They were created for the same purpose, but, back then, by separate teams. It's been our goal for a while to pick a moment when we are going to consolidate those types in a shared assembly.

Is 6.0 the time? There is certainly a good reason to converge, not only do we have duplicate code, but we also do unnecessary interconversions from FhirDate <-> our CqlDate <-> Cql SDK's CqlDate. Also, we could probably improve caching or avoid parsing if we make a single set of types.

I don't know whether we have time for this, as it also requires coordination with the CQL SDK team, but I will put this action here as a reminder in case we have time left, or have shifting priorities.

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

No branches or pull requests

1 participant