Closed courtneythaxton closed 4 years ago
@courtneythaxton Do you mean "Secondary Affiliation" instead of Secondary Approver"? Will this be a means for curators to note that another "Affiliation" was part of the Approval process?
@courtneythaxton so for the dual credit on the website... would you want it to say "Affiliation X (in collaboration with Affiliation Y", or "Affiliation X AND Affiliation Y", or something else?
@courtneythaxton Do you mean "Secondary Affiliation" instead of Secondary Approver"? Will this be a means for curators to note that another "Affiliation" was part of the Approval process?
@matt wright, Maybe we should leave it as "Secondary Contributor." We need to leave it general as there are multiple scenarios in which this "Secondary functions is important."
For instance, with the diagnostics lab potentially curating in the GCI, you can imagine them as a "Biocuration core" as they will not have experts to approve their curation under their affiliation. In this case we would need a ClinGen affiliated EP to approve their curations for publication. In another example, we may have two EPs working together to approve a curation, i.e. for syndromic genes, etc. Each of these affiliations would contribute to the curation and approval. We would also want this information reflected on the website.
I think we will bring it up on the next GCI call and get some more specifications.
@marinadistefano @erinriggs @jennygoldstein
@courtneythaxton so for the dual credit on the website... would you want it to say "Affiliation X (in collaboration with Affiliation Y", or "Affiliation X AND Affiliation Y", or something else?
I believe we would want it as simple as possible. Affiliation X and Affiliation Y contributed to this gene-disease clinical validity classification. We can use the notes sections to provide more detailed information as to the exact relationship. This section may need to be included on the Evidence Summary for transparency.
@wrightmw @marinadistefano @erinriggs @jennygoldstein
It will require some work on the website site. Currently the data shared between the two systems assumed one affiliate but we can update that to handle multiple and then then we need to allow the curation to be associated with more than one affiliate in the database. Both of those items are rather straight forward on our side.
We should carefully consider is if we do or don't need to set a 'primary' affiliate as the producer of the curation because it won't be fun to reverse course down the line. I can think of two display items related to the website to consider; (1) A side effect of having everyone be an equal collaborator is when we start listing the total curations EP have done it will cause some curations will be counted by both groups. So if someone goes to the website they may see 1000 curations completed but on the EP page it may look like 1050 were done. (Assuming 50 were shared). This many not be a concern but I want to mention it. (2) We should also consider how we want to display the affiliation order on the curation details page. Do we simply list them alphabetically if we don't choose a primary curator? Again, just mentioning to help work through the scenarios.
I can't speak to what is needed on the GCI side and I'll let @wrightmw @bryanwulf @jimmyzhen chime in for that.
Hi @sgoehringer Thank. you for the heads up on some consideration for this change and how it might affect the website. I think we (the Gene Curation Small group) have decided to talk over this some more with the Gene Curation Working Group and PIs to get some specifications. We will be sure to bring up these issues as part of the discussion.
For (1) I understand that there may be overlap and this might not be a huge concern, but notable to discuss.
For (2) This will also be good to discuss. So far we can think of two main scenarios for the use of a secondary approver/ affiliation/ contributor. The first is when a Biocuration core has performed the curation, and the expert approval goes to one of the already established ClinGen EPs for final approval, as the cores do not have experts associated with them. The second example is when two (or possibly more) ClinGen EPs work together to contribute to a curation and classification, for example with syndromic genes.
For the biocuration core example, I think it would be nice to distinguish the group that curated the information, versus the group that gave the final approval.
For the second example, since both affiliations maybe contributing to the curation and approving, this would be good to distinguish.
Any thoughts on this @mattwright @jimmyzhen @bryanwulf @erinriggs @marinadistefano @jennygoldstein
Hi @mattwright @jimmyzhen @sgoehringer we reviewed the "secondary approver" specs in the Gene Curation Small group yesterday (@erinriggs @marinadistefano @jennygoldstein) and have made some adjustments which I will outline.
1) We would like to keep it to just one column for the approver list, but allow multiple choices from the EP/WGs currently curating. On the GCI Affiliation Approval v3 figure, part C, The approver(s) section should be the drop down list that allows multiple choices. We realized that there may some some instances in which more than two groups may want to curate and approve on a gene-disease relationship, especially syndromes in which multiple organs are affected. WE felt this might be the easiest way instead of adding more columns of approvers.
2) In the "GCI Affiliation Approval v3 figure.pdf", part E, the groups can use the approver comment section to outline the specifics on the approval process and and the group(s) efforts.
GCI Affiliation approval v3.pdf
3) For the Website recognition, we have provided a mockup, in which we would like the groups who contributed to the curation to be acknowledged in the "Expert curation" box. Further details of the approval and curation can be provided in the Evidence summary, which we will direct in the Evidence summary outline and SOP.
Website dual recognition v2.pdf
@sgoehringer For the website, we acknowledge point 1 and accept that there may be some overlap in the gene-disease curation records. We did discuss that it would be great to distinguish and display the total number of approved gene-disease curations performed by ClinGen affiliated groups somewhere on the website.
For point 2, at this time how about we label them in alphabetical order for now.
We will make statement updates in the SOP coordinating with this change. Specifically that in cases in which multiple EPs/WGs want to contribute to the curation and/or approval that they must coordinate together for one entry in the GCI. We do not foresee the need to have the secondary group curate overtop of the current groups curation to add to the approval. This type of function will be reserved for groups that are "recruiting" and adding new information onto an existing curation and giving a new approval, not a joint approval.
Please let us know if you need anymore specs, or details.
Thanks @courtneythaxton
@courtneythaxton @marinadistefano @ErinRiggs @jennygoldstein
So, following on from the call just now. If you add EPs to the Approver(s) pulldown then you can specify them as the Approver (you can do this already). We could add the logic that if the Approver is an EP then this field is exported to the website (upon publishing). We would need to collaborate with the website team on this because the Approver field is not currently exported to the website.
Further, we could add an additional pulldown of Contributor(s). If Contributors are selected then these would be exported to the website (upon publishing). Because Contributors would be a new field then we would have to collaborate with the website team to show these.
You would need to specify with the website team how you would want the EP Approvers and/or Contributors to be displayed on the website.
I will be on the next small GCI WG call, so we can discuss this then also :-)
?Perfect - adding in Scott so he is aware. On the website end, we have previously discussed the desire to display the approving EP, though the desire to display contributers is new. I think we can approach the display of these two things in a similar manner, though, so hopefully it won't be too difficult.
Scott, are you able to join us on our Gene Curation small group on Monday at 3p E? Might be good to have everyone on together to discuss.
Erin Rooney Riggs, MS, CGC
Assistant Professor
Certified Genetic Counselor
Autism & Developmental Medicine Institute
Geisinger
eriggs@geisinger.edumailto:eriggs@geisinger.edu
From: Matt W. Wright notifications@github.com Sent: Wednesday, October 24, 2018 9:57 AM To: ClinGen/clincoded Cc: Riggs, Erin; Mention Subject: [External] Re: [ClinGen/clincoded] Secondary approval list for curation Cores (#1796)
@courtneythaxtonhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fcourtneythaxton&data=02%7C01%7Ceriggs%40geisinger.edu%7C7766ba1800514b5653c308d639c0f6ac%7C37d46c567c664402a16055c2313b910d%7C0%7C0%7C636759898265401014&sdata=xH7hqKWpY04dDKWK3wiHaFYrpgU%2BV%2F0ofHg3mYQhiIM%3D&reserved=0 @marinadistefanohttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmarinadistefano&data=02%7C01%7Ceriggs%40geisinger.edu%7C7766ba1800514b5653c308d639c0f6ac%7C37d46c567c664402a16055c2313b910d%7C0%7C0%7C636759898265401014&sdata=ScpSe7DSN3KBEoXMPhJzeW3OVEpznBzL0ZsT9hh5EWs%3D&reserved=0 @ErinRiggshttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FErinRiggs&data=02%7C01%7Ceriggs%40geisinger.edu%7C7766ba1800514b5653c308d639c0f6ac%7C37d46c567c664402a16055c2313b910d%7C0%7C0%7C636759898265401014&sdata=5kGcDISDrBbDF2qACklS1FvIrWLNCBRCNi5lDjr7N2A%3D&reserved=0 @jennygoldsteinhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fjennygoldstein&data=02%7C01%7Ceriggs%40geisinger.edu%7C7766ba1800514b5653c308d639c0f6ac%7C37d46c567c664402a16055c2313b910d%7C0%7C0%7C636759898265401014&sdata=i%2Fu%2FEz6YDo0%2BocbLrdqqY6%2FyZk8YnibW4SB6eFvTUnc%3D&reserved=0
So, following on from the call just now. If you add EPs to the Approver(s) pulldown then you can specify them as the Approver (you can do this already). We could add the logic that if the Approver is an EP then this field is exported to the website (upon publishing). We would need to collaborate with the website team on this because the Approver field is not currently exported to the website.
Further, we could add an additional pulldown of Contributor(s). If Contributors are selected then these would be exported to the website (upon publishing). Because Contributors would be a new field then we would have to collaborate with the website team to show these.
You would need to specify with the website team how you would want the EP Approvers and/or Contributors to be displayed on the website.
I will be on the next small GCI WG call, so we can discuss this then also :-)
- You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FClinGen%2Fclincoded%2Fissues%2F1796%23issuecomment-432692682&data=02%7C01%7Ceriggs%40geisinger.edu%7C7766ba1800514b5653c308d639c0f6ac%7C37d46c567c664402a16055c2313b910d%7C0%7C0%7C636759898265401014&sdata=oaG%2F1h1vgvlbnupvfPs6c0qsfliXk41v%2BZQsyFfhkBo%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAcqZlOTzCs3U-6r4JrQqYnpdMj3kUMABks5uoH_AgaJpZM4XK2Mb&data=02%7C01%7Ceriggs%40geisinger.edu%7C7766ba1800514b5653c308d639c0f6ac%7C37d46c567c664402a16055c2313b910d%7C0%7C0%7C636759898265401014&sdata=V%2FJlqWlLSMkU83n%2Bubx0w2Vz5RhHiaLBLkOR%2B1SvVYE%3D&reserved=0.
IMPORTANT WARNING: The information in this message (and the documents attached to it, if any) is confidential and may be legally privileged. It is intended solely for the addressee. Access to this message by anyone else is unauthorized. If you are not the intended recipient, any disclosure, copying, distribution or any action taken, or omitted to be taken, in reliance on it is prohibited and may be unlawful. If you have received this message in error, please delete all electronic copies of this message (and the documents attached to it, if any), destroy any hard copies you may have created and notify me immediately by replying to this email. Thank you. Geisinger Health System utilizes an encryption process to safeguard Protected Health Information and other confidential data contained in external e-mail messages. If email is encrypted, the recipient will receive an e-mail instructing them to sign on to the Geisinger Health System Secure E-mail Message Center to retrieve the encrypted e-mail.
Thank you for keeping me in the loop and I was aware that this was being considered. Do we have a timeline (rough estimates are totally fine) on when the updates are expected in the GCI so we can work this into our development schedule?
Also, I can join a Monday call at 3pm to discuss further. Please share the invite when you have a chance.
@courtneythaxton, @marinadistefano, @ErinRiggs, @jennygoldstein, @wrightmw
Okay, I will work on a mock up for the Multiple contributors section at this point. It would be good to know @jimmyzhen @wrightmw whether a multiple choice, single drop down would work column would work, or whether we would need a separate dropdown for each contributor? Thank you all for your help and advice on this.
@ErinRiggs @marinadistefano @jennygoldstein @sgoehringer
@courtneythaxton @wrightmw,
At the very basic level, multiple selection in a single HTML <select>
(aka dropdown) is supported. In this case, the user can select multiple options from the dropdown menu while holding down either the Command
key (Mac) or the Control
key (Windows).
The implementation I envision for this, from the UI and UX perspective, might be something similar to the following:
If needed, we can organize the contributors into 2 groups in one single dropdown, such as expert panels and non-expert panels.
Thanks @jimmyzhen this is really helpful to think about, and glad that the multiple selection will work. I do think we will need to think about how many of these drop downs we want.
I do agree that it seems that we would need a Contributing Expert panel and a Contributing non-expert panel that would distinguish the entities contributing to the curation, and we might also want at least one Approving EP drop down, but possibly a second one that would indicate another EP contributed to the approval. WE will sort this out in the next Small group call (@erinriggs @marinadistefano @jennygoldstein).
@jimmyzhen VCI WG confirmed they are happy with the same functionality added into the VCI.
I wanted to add the new mockup of the contributor and approver classification for the GCI modal. We discussed that the drop down menus for the contributor and approver should be the official affiliation names of the GCEPs, not individual names. As @jimmyzhen showed above, we would like both contributor and approver to have multiple choice selection.
I have also included an updated website display. I added a section to highlight the groups contributing to the curation, and created it as smaller text, that way the approval of the classification is the larger, more noticeable.
let me know what you all think or if I should make any adjustment or if we need to discuss more specifications @ErinRiggs @Marinadistefano @jennygoldstein @wrightmw
Thanks! @courtneythaxton
Website dual recognition v3.pdf Secondary Affiliation approval v5.pdf
Thanks for doing this Courtney. This looks good. The only question I have is if we had decided that only one group or multiple groups would be the approver(s)? I only ask because you have multiple contributors, but it's unclear whether we would have multiple approvers (just because the dropdown text is plural)?
Marina
From: courtneythaxton notifications@github.com Sent: Tuesday, November 13, 2018 9:48 AM To: ClinGen/clincoded Cc: DiStefano, Marina,Ph.D.; Mention Subject: Re: [ClinGen/clincoded] Secondary approvers and contributors in the GCI/VCI (#1796)
External Email - Use Caution
I wanted to add the new mockup of the contributor and approver classification for the GCI modal. We discussed that the drop down menus for the contributor and approver should be the official affiliation names of the GCEPs, not individual names. As @jimmyzhenhttps://github.com/jimmyzhen showed above, we would like both contributor and approver to have multiple choice selection.
I have also included an updated website display. I added a section to highlight the groups contributing to the curation, and created it as smaller text, that way the approval of the classification is the larger, more noticeable.
let me know what you all think or if I should make any adjustment or if we need to discuss more specifications @ErinRiggshttps://github.com/ErinRiggs @marinadistefanohttps://github.com/marinadistefano @jennygoldsteinhttps://github.com/jennygoldstein @wrightmwhttps://github.com/wrightmw
Thanks! @courtneythaxtonhttps://github.com/courtneythaxton
Website dual recognition v3.pdfhttps://github.com/ClinGen/clincoded/files/2576697/Website.dual.recognition.v3.pdf Secondary Affiliation approval v5.pdfhttps://github.com/ClinGen/clincoded/files/2576698/Secondary.Affiliation.approval.v5.pdf
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ClinGen/clincoded/issues/1796#issuecomment-438292078, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Acqgeq5PgDjBFsEN1PmQaeYF_UY1wFcNks5uutu_gaJpZM4XK2Mb.
The information in this e-mail is intended only for the person to whom it is addressed. If you believe this e-mail was sent to you in error and the e-mail contains patient information, please contact the Partners Compliance HelpLine at http://www.partners.org/complianceline . If the e-mail was sent to you in error but does not contain patient information, please contact the sender and properly dispose of the e-mail.
That is a good question Marina. I'm not sure it was exactly decided by the large group. I believe towards the end we felt that more than one approver would be nice, and would be used in a limited amount of cases. I do think there was some discussion on whether one affiliation should be able to add a secondary approval (as we have outlined), or whether it needs to be enabled in the GCI for two separate affiliation to approve the same GDM record under their own affiliations.
I believe for now our solution is the most straight forward, and I do not think multiple groups would “abuse” (for lack of a better word) the ability to choose a second affiliation. Meaning I don’t think a group would choose a secondary approver without reviewing the classification first. However, I do think it should be confined to the ClinGen Expert panels to choose this option, and not for outside entities (such as counsel, illuminate, or the community).
What do others think?
Courtney
From: marinadistefano notifications@github.com<mailto:notifications@github.com> Reply-To: ClinGen/clincoded reply@reply.github.com<mailto:reply@reply.github.com> Date: Monday, November 19, 2018 at 8:47 AM To: ClinGen/clincoded clincoded@noreply.github.com<mailto:clincoded@noreply.github.com> Cc: Courtney Thaxton courtney_thaxton@med.unc.edu<mailto:courtney_thaxton@med.unc.edu>, Mention mention@noreply.github.com<mailto:mention@noreply.github.com> Subject: Re: [ClinGen/clincoded] Secondary approvers and contributors in the GCI/VCI (#1796)
Thanks for doing this Courtney. This looks good. The only question I have is if we had decided that only one group or multiple groups would be the approver(s)? I only ask because you have multiple contributors, but it's unclear whether we would have multiple approvers (just because the dropdown text is plural)?
Marina
From: courtneythaxton notifications@github.com<mailto:notifications@github.com> Sent: Tuesday, November 13, 2018 9:48 AM To: ClinGen/clincoded Cc: DiStefano, Marina,Ph.D.; Mention Subject: Re: [ClinGen/clincoded] Secondary approvers and contributors in the GCI/VCI (#1796)
External Email - Use Caution
I wanted to add the new mockup of the contributor and approver classification for the GCI modal. We discussed that the drop down menus for the contributor and approver should be the official affiliation names of the GCEPs, not individual names. As @jimmyzhenhttps://github.com/jimmyzhen showed above, we would like both contributor and approver to have multiple choice selection.
I have also included an updated website display. I added a section to highlight the groups contributing to the curation, and created it as smaller text, that way the approval of the classification is the larger, more noticeable.
let me know what you all think or if I should make any adjustment or if we need to discuss more specifications @ErinRiggshttps://github.com/ErinRiggs @marinadistefanohttps://github.com/marinadistefano @jennygoldsteinhttps://github.com/jennygoldstein @wrightmwhttps://github.com/wrightmw
Thanks! @courtneythaxtonhttps://github.com/courtneythaxton
Website dual recognition v3.pdfhttps://github.com/ClinGen/clincoded/files/2576697/Website.dual.recognition.v3.pdf Secondary Affiliation approval v5.pdfhttps://github.com/ClinGen/clincoded/files/2576698/Secondary.Affiliation.approval.v5.pdf
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ClinGen/clincoded/issues/1796#issuecomment-438292078, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Acqgeq5PgDjBFsEN1PmQaeYF_UY1wFcNks5uutu_gaJpZM4XK2Mb.
The information in this e-mail is intended only for the person to whom it is addressed. If you believe this e-mail was sent to you in error and the e-mail contains patient information, please contact the Partners Compliance HelpLine at http://www.partners.org/complianceline . If the e-mail was sent to you in error but does not contain patient information, please contact the sender and properly dispose of the e-mail.
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ClinGen/clincoded/issues/1796#issuecomment-439898149, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AfBeVWhnY38ITA1DOzMcspN8Mj1VHAv0ks5uwraIgaJpZM4XK2Mb.
Hi all,
I have outlined some specs for the multiple contributor and multiple approvers for a classification in the GCI (potentially to enable in the VCI), after the March 7 GCI/VCI call. Please see the attached word document.
One suggestions was to provide a controlled drop down list of choices to indicate how "Contributors" contributed to the curation and/or classification. Help would be appreciated to populate this list. Some initial suggestions include: performed the curation, performed the procuration, provided evidence as an outside entity (e.g. Counsyl spreadsheets).
@wrightmw @cgpreston @ErinRiggs @marinadistefano @jennygoldstein @sgoehringer Multiple contributor and approver specs_March 2019.docx
Thank you for making this document Courtney. I think it overall looks good and is consistent with what we discussed. The only question I have (and maybe we can just discuss it again quickly on Monday's GCI call with Matt) is about the dropdown for contributor comments vs the free text and how that would work. Would it just be the dropdown text, the free text only, or both?
From: courtneythaxton notifications@github.com Sent: Monday, March 18, 2019 9:49:02 PM To: ClinGen/clincoded Cc: DiStefano, Marina,Ph.D.; Mention Subject: Re: [ClinGen/clincoded] Secondary approvers and contributors in the GCI/VCI (#1796)
External Email - Use Caution
Hi all,
I have outlined some specs for the multiple contributor and multiple approvers for a classification in the GCI (potentially to enable in the VCI), after the March 7 GCI/VCI call. Please see the attached word document.
One suggestions was to provide a controlled drop down list of choices to indicate how "Contributors" contributed to the curation and/or classification. Help would be appreciated to populate this list. Some initial suggestions include: performed the curation, performed the procuration, provided evidence as an outside entity (e.g. Counsyl spreadsheets).
@wrightmwhttps://github.com/wrightmw @cgprestonhttps://github.com/cgpreston @ErinRiggshttps://github.com/ErinRiggs @marinadistefanohttps://github.com/marinadistefano @jennygoldsteinhttps://github.com/jennygoldstein @sgoehringerhttps://github.com/sgoehringer Multiple contributor and approver specs_March 2019.docxhttps://github.com/ClinGen/clincoded/files/2981147/Multiple.contributor.and.approver.specs_March.2019.docx
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHubhttps://github.com/ClinGen/clincoded/issues/1796#issuecomment-474166202, or mute the threadhttps://github.com/notifications/unsubscribe-auth/Acqgeu60k6P38SP0B9hocs5hAsPHJ4Mpks5vYEIOgaJpZM4XK2Mb.
The information in this e-mail is intended only for the person to whom it is addressed. If you believe this e-mail was sent to you in error and the e-mail contains patient information, please contact the Partners Compliance HelpLine at http://www.partners.org/complianceline . If the e-mail was sent to you in error but does not contain patient information, please contact the sender and properly dispose of the e-mail.
Hi All, After the Gene curation small call today, we felt that the use of the standardized drop down selection may not be readily used and therefore decided to go ahead with the specs uploaded here. In the future if the PIs really feel that we should include these standardized text we can work on them. At this point in time, the text in these boxes is not forwarded to the website team nor displayed on the curation summaries, and we would like for it to be maintained in that way.
thanks, @courtneythaxton
@marinadistefano @wrightmw @ErinRiggs @jennygoldstein @cgpreston @markmandell
@courtneythaxton @ErinRiggs @jennygoldstein
Can you please confirm the answer to this issue? Currently for many GCEPs, they have elected to have a small number of people who are "Approvers"... for instance, in the HL GCEP there are 3 Approvers... when someone goes to approve then they have select one of these 3 Approvers. These individual approvers are not made public. Do you still want these groups to be able to chose individual approvers? And would we want a selected secondary approver to be able to pick individual approvers?
I think we would want the "main" group to continue to be able to approve things the way they normally would (i.e., if they normally select individual approvers, they should be able to select individual approvers here). I don't think it is necessary to identify individual approvers for the secondary group.
Thanks for the clarification @ErinRiggs :-)
Here is the instance for this ticket: https://1796-mm-secondaryapprovers.demo.clinicalgenome.org/dashboard/
There seemed to a be a consensus to keep the "normal" classification Approver input, in addition to adding what was included in the spec. The additional info should be displayed in the Approval Preview, the Approval "snapshots" toward the bottom of the page, as well as the Evidence Summary. Let me know if there are any issues/feedback!
Hi @markmandell @wrightmw
This is looking good. I like the ability to expand or hide the screen.
First note: The contributor vs Approver sections are unaligned, could we align them? (see pic)
For the approval summary in the GCI, I was able to add multiple contributor and approvers! See pic
I believe that the GCWG set that the approving group has the responsibility for giving credit to the contributors and approvers outside of their affiliation.
I did want to ask, for GDMs that may be transferred from a biocuration core, is there a way to highlight the contribution? This may be too much right now, and that is okay if it cannot be done, but thought I would ask
The unaligned inputs might be a browser issue, I'll definitely look into it and fix! I'll test it out to see. Just wanted to include a screenshot from the panel when I went to the same instance to show what it looks like on my end.
Also seems like the original Approvers input is missing from your screenshot, I'll refresh the instance to see if that works. I will also be sure to get with Matt and discuss highlighting the contributions, and I will take a look into that as well :)
Thanks @markmandell IT may be my browser, I'm on my single, little Mac screen!
@courtneythaxton 1) Now that we have "Curation Approver(s)" it's confusing to also have the "Approver". How about we change "Approver" to something like "Individual Approver" or "Affiliation Approver" or something else to make the terminology distinct? 2) The button that opens up the form says "Open Contributor(s) Form", do you think it would be more clear if it said "Open Contributor(s)/Curation Approvers(s) Form"... but that's getting too long, so how about just "Open Attribution Form"? 3) Do think there needs to some contextual help here to explain what these mean? Like a mouseover or 'i" information icon, that says when someone should be credited as a contributor or approver?
@markmandell This looks great. I noticed one error in the way this works. When you save an Approved record then on the classification page this sits above the provisional saved. When I migrate away and come back to the classification summary, then the Approval has disappeared.
@courtneythaxton just reading through this thread, I noticed your question further up. The process shown within the UI is all about capture of these data. How it gets used for accreditation on the website is a downstream issue. Currently in this v1 the record itself would remain a UNC biocuration core owned record if they created it. When selecting an Approver, you are just designating that they did the final Approval. If we sent the contributor and approver details to the website then they could credit the biocuration core there? Perhaps the contributor could be auto-filled with UNC biocuration core?
@courtneythaxton do you want the approvers to be all VCEPs/GCEPs as they are now listed, or should this just be GCEPs only?
Hi @wrightmw @markmandell
I agree to change the "Approver" to "Affiliation Approver"
I like the "Open Attributions Form"
I like the idea of the info button or hover over. I will work on this text with the small group.
Thank you for the downstream clarification. I was just wanting to consider when to get Scott G and Tristan in the picture, but most likely once we finalize this function before release.
Both VCEPs and GCEPs in the approvers list.
Final Approval Date can be made required in a future release (see #2043)
@sgoehringer When this ticket goes live (likely R29), then we'll have contributors/approvers to add to our JSON we provide to you. Can you please work with @markmandell to confirm what format you'd require these to be in within the JSON file, so that we don't break the pipeline by changing the file.
Also @courtneythaxton @ErinRiggs @jennygoldstein, if you want these data displayed on the website then you'll have to let Scott know how you want these to appear.
Example of current application of this ticket:
There is now only one Curation Approver allowed. However, multiple Curation Contributors are still allowed.
On the large GC WG call today, it was suggested that the following text changes be made:
Is that correct @ErinRiggs . Other than those text changes, I don't think there is anything else that needs to be done with this ticket (is that right?), so maybe we could sign off on this (after the text changes) and do the code review on it, and try to get this into the next release?
@markmandell Text changes above would apply to the publication panel also.
@markmandell @bryanwulf In most cases an affiliation would only have a "ClinGen affiliation" associated with it, and we export that to the streaming service/website. However, if the user selected an alternative Approver (note: currently on this mockup this is the "Curation Approver" but changing to "Classification Approver") then instead should that be the only one exported to the streaming service/website. That is, the default we export is the "ClinGen affiliation" unless an alternative Approver has been selected, and then should we export that alone. I suspect @sgoehringer would only want to receive one Approver from our output?
@wrightmw Sounds good, I'll make the changes and update the instance.
As for what's being sent to the website, it looks to me like we would be sending both. But if one needs to be prioritized over the other or we only want to send one approver, we can decide which is best!
Thanks @markmandell
@sgoehringer would you be ok with us sending you both? And you would just display one of them? i.e. display the "ClinGen Affiliation" as the Approver by default, but if there is an alternative approver then display that instead?
@bryanwulf @markmandell "Classification Approver(s)" used to be multi-choice with a type ahead functionality, but this field got changed by request to just a single "Classification Approver" and a pulldown was instituted instead. Finding the one you are looking for in the pulldown is not very user friendly because it's a long list (and will get longer) and they are not in any perceivable order. Would it be a quick fix to make this list alphabetical?
@markmandell The type ahead functionality is working for me now! So looks good.
@markmandell @bryanwulf Will the contributors appear on the website once published? If not, then we still need to have an indicator to our users that this will appear at a later time.
@markmandell @bryanwulf So, for instance... here where I've provided "Classification Contributors" there needs to be some text letting folks know that "Classification Contributors can be published but will not appear on the ClinGen website until it has been updated to support this new field":
@cgpreston Do you have any suggested tweaks for this message?
@wrightmw, I suspect some people will be interested in knowing if the website will then retroactively show any Classification Contributors for curations published between this feature going live in the VCI and the website support for the field.
If the website implementation is retroactive, maybe something like "Classification Contributors may be added and the curations published in the VCI now, the classification contributors will not appear on the ClinGen website until the website supports displaying this new field.". If it's not retroactive then I think we need different language.
Support for the display will be retroactive.
That looks great... here again with very minor tweaks: "Classification Contributors may be added and then published in the VCI now, however Classification Contributors will not appear on the ClinGen website until the website supports displaying this new field.".
Would you all consider this language? "At present, designation of Classification Contributors is restricted to the VCI/GCI. In the future, Classification Contributors will appear on curation summaries published to the ClinGen website and/or ERepo to facilitate recognition."
I like the version Courtney shared.
Sent from my iPhone
On Dec 17, 2019, at 9:24 PM, courtneythaxton notifications@github.com wrote:
Would you all consider this language? "At present, designation of Classification Contributors is restricted to the VCI/GCI. In the future, Classification Contributors will appear on curation summaries published to the ClinGen website and/or ERepo to facilitate recognition."
— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or unsubscribe.
The Biocuration cores (and possible other curation EPs) will need the ability to have a secondary expert panel list for approval of curations, as the cores themselves do not have a broad list of expert for approval, and many of the curations will be sent to other EPs for approval. Recognition for the curation affiliation and approval EP affiliation should translate to the website.
Specifications: (1) Under Approve classification, provide a "Secondary Approver" dropdown list. This list should include all current Affiliations. Potentially add a "Secondary Approver comments section in which the name(s) of the experts for that affiliation can be logged. Secondary Affiliation approval v2.pdf
(2) We would also like this "dual" credit to be recognized on clinical genome.org gene-disease validity classifications (2a) Addition of an area in the classification matrix under "calculated classification" that says curation performed by the following Affiliation: list per the primary approver dropdown (2b) Under the "expert curation (date): Provide the Classification (approved date) and the following text: This gene-disease validity classification was approved by the "name of secondary approver affiliation." Website dual recognition.pdf
If these needs more specification, please advise @wrightmw @jimmyzhen @jennygoldstein @erinriggs @marinadistefano @sgoehringer