-
Notifications
You must be signed in to change notification settings - Fork 5
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
Additional options for hasEvidenceItems
#258
Comments
Responses from original post: From korikuzma:
From mrinal-thomas-epic:
|
@Mrinal-Thomas-Epic if I am understanding your example correctly, you want to report that you have a Pathogenicity Statement with an Evidence Line based on the BA1 ACMG criterion - and want to capture a gnomAD record as the evidence item for this Evidence Line (as opposed to representing the frequency data from that record explicitly using a CAFStudyResult object). I would simply use the
I don't think there is anything in the definition of the |
UPDATE (1-17-25): If Epic intends to stand up a service / system that stores and serves va-compliant data objects of the types that can be values of the If not, another solution to support the gnomAD scenario is to create and send a
In this way, the data provider has to do very little work and data consumers will be able to follow the provided url to navigate to the gnomAD page that shows the data items for this StudyResult. |
I think the second option could work for us as long as we don't add additional required properties to |
I've also hit a couple other questions while trying to work with
|
Several good observations and questions here @Mrinal-Thomas-Epic. I'll address each in a separate comment. > the data provider might not want to make a statement about whether the external record is a This would be possible if we made That said, the preferred outcome IMO is for the data creator to select the most appropriate subclass in cases like this, and I am optimistic that our documentation will allow them to do so. Do you anticipate it being a significant problem to require the data provider chooses an |
> is there a way to discretely represent the system (e.g. Yes. There is a 'sourceDataSet' attribute on The data example here illustrates this, and the relevant snippet is shown below: Does this address your need @Mrinal-Thomas-Epic? |
> Does The
If a situation arises where there is a document reports the central info conveyed in both a Does the above address your question/use case @Mrinal-Thomas-Epic? As an aside, what is a more common use case for SEPIO also provides a shortcut property that allows data creators to point directly to sources of evidence from the root These are likely not immediately relevant here, but please keep in mind the availability of these types of |
> It looks like I believe this was permitted because some of the more complex ClinGen curation pipelines around gene curation, and clinical actionability statements, had a use case for this type of thing. @larrybabb can provide details. However these models are not implemented yet, so it may make sense to remove this as a permissible type for the |
For |
In my mind, " a document that reports the central knowledge expressed in the statement" can refer to one of two use cases:
|
Originally posted by @Mrinal-Thomas-Epic in #234 (comment):
For our use case, we're thinking it would be nice to also have simpler options for
hasEvidenceItems
. Specifically, it would be nice to be able to have the option to put in aDocument
(for references to PubMed articles) or aCoding
(for references to external DBs). I can see an argument for wrapping theDocument
orCoding
within anotherStatement
, but think that could be more complicated than needed for this simple caseThe text was updated successfully, but these errors were encountered: