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

Include studyGroup and componentResult attributes in the va-core model #268

Open
mbrush opened this issue Feb 12, 2025 · 0 comments
Open

Comments

@mbrush
Copy link
Contributor

mbrush commented Feb 12, 2025

Following on the discussion in #250, I would like to include in the va-core model the sepio studyGroup attribute (on which the CAF cohort attribute is based), and similarly include the sepio componentResult attribute (on which the CAF subCohortFrequency attribute is based).

These were previously removed va-core, based on the argument that only one implementation uses them (CAF) - but I think one implementer is sufficient to warrant inclusion in the core model. And think it is helpful for incoming adopters to be able to see these attributes them there. They can remain 'draft' in va core until a second adopter implements, but I see no reason to not include them in the core model.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant