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

Verify we have implemented missing converters for tuples in modelinfo #131

Open
EvanMachusak opened this issue Dec 18, 2023 · 0 comments
Open
Assignees
Labels
Enhancement New feature or request

Comments

@EvanMachusak
Copy link
Collaborator

EvanMachusak commented Dec 18, 2023

See #129

We should create some test code that:

  • Dynamically generates Copy T functions for all T of type ClassInfo present in a ModelInfo class which creates a new named tuple and assigns all of its properties to obj's value, e.g.:
define function "Copy Patient"(obj FHIR.Patient):
  FHIR.Patient {
     id: obj.id,
     birthDate: obj.birthDate,
     ... all other elements of Patient follow ...
}
  • Fix the cryptic ArgumentException that gets thrown when a conversion required for the member binding is missing, with information about which conversion is being attempted and not available
  • Add all missing converters for the FHIR model

This will allow us to be sure that we can dynamically create FHIR model tuples in CQL.

@EvanMachusak EvanMachusak added the Enhancement New feature or request label Dec 18, 2023
@EvanMachusak EvanMachusak self-assigned this Dec 18, 2023
@ewoutkramer ewoutkramer changed the title Implement missing converters Verify we have implemented missing converters for tuples in modelinfo Jul 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant