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

Medication Source Code Verification: Student 2 #42

Closed
1 of 7 tasks
callahantiff opened this issue May 15, 2019 · 18 comments
Closed
1 of 7 tasks

Medication Source Code Verification: Student 2 #42

callahantiff opened this issue May 15, 2019 · 18 comments
Assignees
Labels
source code verification verification of source string to source code mappings

Comments

@callahantiff
Copy link
Owner

callahantiff commented May 15, 2019

Student (GitHub Username): @avnmatre
Verification Number: 2

Assigned Domain Expert: @trinklek

Verification Task Description: DropBox Paper Documentation

Phenotype Descriptions: project wiki

Verification Assignments:
UPDATE: counts below updated to reflect today's meeting.

Verification Assignments:

@callahantiff callahantiff added the source code verification verification of source string to source code mappings label May 15, 2019
@callahantiff callahantiff self-assigned this May 15, 2019
@avanmatre
Copy link

@trinklek Can you look at this range? I chose partial/appropriate + clinically relevant because the dosages of dex/amphetamine in the source_name column don't correspond to the true dose of the med (row 31 would correspond to a 10mg dose, yes?). https://docs.google.com/spreadsheets/d/1C9WOk9jKgYh7R50Rb8Ck3pV0sMikrbNrJDmGBnytOCE/edit#gid=0&range=31:34

@trinklek
Copy link
Collaborator

@avanmatre I agree that row 31 is referring to 10mg of Adderall, but I don't think there is any disconcordance. Are you comparing columns D and G and thinking column D is referring to 2.5mg? I think column D is just collapsing each of the salt forms of amphetamine and dextroamphetamine for simplicity. It looks like a number of columns do this and I don't think it is wrong. If the collapsing of the salt forms in column D is what has you stuck, I think we should explore different references to see if this is an appropriate collapsing/categorization - I am leaning towards it is appropriate.

@avanmatre
Copy link

@trinklek I am comparing columns D and G - I understand that the salt forms are collapsed but row #31 for instance is showing 2.5/2.5 in column D which doesn't clearly correlate with a 10mg dose whereas column G correlates with a 10mg dose. Columns D and G don't EXACTLY match so I wasn't sure about indicating exactly vs partial match. I agree it is appropriate, just unsure to what end "exactly" needs to be followed.

@avanmatre
Copy link

@trinklek Compare my previous question to https://docs.google.com/spreadsheets/d/1C9WOk9jKgYh7R50Rb8Ck3pV0sMikrbNrJDmGBnytOCE/edit#gid=0&range=126:126
where column D adds to 5mg dose

@trinklek
Copy link
Collaborator

@avanmatre Great comparison! Row 31 and row 126 definitely are inconsistent in content for column D.

@callahantiff Do you only want the source string (column B) compared to the source name (column D), or do you also want standard name (column G) compared/evaluated?

@callahantiff
Copy link
Owner Author

@avanmatre Great comparison! Row 31 and row 126 definitely are inconsistent in content for column D.

@callahantiff Do you only want the source string (column B) compared to the source name (column D), or do you also want standard name (column G) compared/evaluated?

Hey @avanmatre and @trinklek! Sorry for being late to the party! To answer your question, the initial plan was only to have you verify the string in column B to the result returned in column D. The mapping between column D and column G is done by the OMOP community. I think your point is very valid and definitely worth capturing in the comments column now that you identified it!

With that said, our primary focus is on verifying the mappings between column B and column D. If you happen to notice something worth mentioning in the other columns, note, but don't spend too much time on it. Does that sound OK? What do you think about that plan @trinklek?

@trinklek
Copy link
Collaborator

@avanmatre and @callahantiff I completely agree with the plan to focus on Columns B and D.

@callahantiff
Copy link
Owner Author

callahantiff commented May 30, 2019

@avanmatre - Just a reminder, once you have finished verifying each file, can you please do the following:

  • Check the box that corresponds to the file above?
  • Close the specific issue (hyperlinked above) and include a brief comment to confirm its completion? This is also a great time to add any additional information that you think is important for me to know or keep in mind as I move onto the next phase of the analysis.
  • Add the date of completion to the Dropbox Paper table (here).

Thanks and great work so far! 💪

@callahantiff
Copy link
Owner Author

@avanmatre

UPDATE: counts below updated to reflect today's meeting.

Verification Assignments:

@avanmatre
Copy link

avanmatre commented Jun 25, 2019 via email

@callahantiff
Copy link
Owner Author

@avanmatre - Thanks so much for your email. I completely understand, your residency is WAY more important. Please don't worry about it. Are there hours that we still need to pay you for? Aside from that, there is nothing further needed at this time. Thanks for being willing to help me with this work. If you are still interested in submitting something for your conference, I'm still very open to discussing that. Just let me know.

Once you confirm whether or not we have outstanding hours to pay you for, I will close out all issues assigned to you for mapping.

@callahantiff
Copy link
Owner Author

@trinklek - I spoke with @mgkahn today regarding this. It was brought up that there is a chance that not having two reviewers could bite us with reviewers, especially since we will have two medical students reviewing the non-medication domains. Do you buy the following argument -- we need two reviewers for the non-medication domain because the two reviewers are students. We don't need two reviewers for the medication domain because the first reviewer is not a student, but a licensed pharmacist with lots of relevant experience. Do you think that argument would hold or, should we be safe and find a second reviewer?

@avanmatre
Copy link

avanmatre commented Jul 2, 2019 via email

@trinklek
Copy link
Collaborator

trinklek commented Jul 2, 2019

@trinklek - I spoke with @mgkahn today regarding this. It was brought up that there is a chance that not having two reviewers could bite us with reviewers, especially since we will have two medical students reviewing the non-medication domains. Do you buy the following argument -- we need two reviewers for the non-medication domain because the two reviewers are students. We don't need two reviewers for the medication domain because the first reviewer is not a student, but a licensed pharmacist with lots of relevant experience. Do you think that argument would hold or, should we be safe and find a second reviewer?

I do think having a second reviewer would be ideal. KD is absolutely brilliant, but there are many nuances to this. I also think the perception of a pharmacist is not as strong as other clinician types given the huge spectrum of types of pharmacists adn variable training (e.g., walgreens versus clinical), thus your reviewer may wonder if the pharmacist was really credible - I absolutely would at least. I am confident a second reviewer will not be 100% consistent with what KD found, so I wonder if you are OK with someone doing a 10-20% sample or if you want a second person to review everything. If you are OK reporting an imperfect, but still positive interrater agreement, then I would go with a second reviewer looking at 10-20%, but if you want to come up with final mapping, then I would have a second person review all of it and I serve as the mediator for any disagreements so there...I am not 100% sure who the second reviewer should/could be, but my new fellow who started 7/1 is a possibility. I think you need a pharmacist (not student) for this type of work and not just any pharmacist - I am happy to reach out to folks if you want, or discuss further. Ideally we can find a trainee for cheap or a practicing clinical pharmacist who is willing to do at no cost, but I do want to find someone that I know is credible.

@callahantiff
Copy link
Owner Author

@trinklek - I spoke with @mgkahn today regarding this. It was brought up that there is a chance that not having two reviewers could bite us with reviewers, especially since we will have two medical students reviewing the non-medication domains. Do you buy the following argument -- we need two reviewers for the non-medication domain because the two reviewers are students. We don't need two reviewers for the medication domain because the first reviewer is not a student, but a licensed pharmacist with lots of relevant experience. Do you think that argument would hold or, should we be safe and find a second reviewer?

I do think having a second reviewer would be ideal. KD is absolutely brilliant, but there are many nuances to this. I also think the perception of a pharmacist is not as strong as other clinician types given the huge spectrum of types of pharmacists adn variable training (e.g., walgreens versus clinical), thus your reviewer may wonder if the pharmacist was really credible - I absolutely would at least. I am confident a second reviewer will not be 100% consistent with what KD found, so I wonder if you are OK with someone doing a 10-20% sample or if you want a second person to review everything. If you are OK reporting an imperfect, but still positive interrater agreement, then I would go with a second reviewer looking at 10-20%, but if you want to come up with final mapping, then I would have a second person review all of it and I serve as the mediator for any disagreements so there...I am not 100% sure who the second reviewer should/could be, but my new fellow who started 7/1 is a possibility. I think you need a pharmacist (not student) for this type of work and not just any pharmacist - I am happy to reach out to folks if you want, or discuss further. Ideally we can find a trainee for cheap or a practicing clinical pharmacist who is willing to do at no cost, but I do want to find someone that I know is credible.

@trinklek - Sorry I somehow missed this! Thank you for the great information! OK, let's move forward with finding someone credible and trustworthy. Perhaps we can reassess whether they do 10-20% or all terms depending on who shows interest? We would also be able to pay them for their time, same rate as before.

@trinklek
Copy link
Collaborator

@trinklek - I spoke with @mgkahn today regarding this. It was brought up that there is a chance that not having two reviewers could bite us with reviewers, especially since we will have two medical students reviewing the non-medication domains. Do you buy the following argument -- we need two reviewers for the non-medication domain because the two reviewers are students. We don't need two reviewers for the medication domain because the first reviewer is not a student, but a licensed pharmacist with lots of relevant experience. Do you think that argument would hold or, should we be safe and find a second reviewer?

I do think having a second reviewer would be ideal. KD is absolutely brilliant, but there are many nuances to this. I also think the perception of a pharmacist is not as strong as other clinician types given the huge spectrum of types of pharmacists adn variable training (e.g., walgreens versus clinical), thus your reviewer may wonder if the pharmacist was really credible - I absolutely would at least. I am confident a second reviewer will not be 100% consistent with what KD found, so I wonder if you are OK with someone doing a 10-20% sample or if you want a second person to review everything. If you are OK reporting an imperfect, but still positive interrater agreement, then I would go with a second reviewer looking at 10-20%, but if you want to come up with final mapping, then I would have a second person review all of it and I serve as the mediator for any disagreements so there...I am not 100% sure who the second reviewer should/could be, but my new fellow who started 7/1 is a possibility. I think you need a pharmacist (not student) for this type of work and not just any pharmacist - I am happy to reach out to folks if you want, or discuss further. Ideally we can find a trainee for cheap or a practicing clinical pharmacist who is willing to do at no cost, but I do want to find someone that I know is credible.

@trinklek - Sorry I somehow missed this! Thank you for the great information! OK, let's move forward with finding someone credible and trustworthy. Perhaps we can reassess whether they do 10-20% or all terms depending on who shows interest? We would also be able to pay them for their time, same rate as before.

@callahantiff I had an idea today of someone today. For this person, if they review a 10-20% sample, will participation in publication be an option? Your answer will help me pitch it correctly.

@callahantiff
Copy link
Owner Author

@trinklek - I spoke with @mgkahn today regarding this. It was brought up that there is a chance that not having two reviewers could bite us with reviewers, especially since we will have two medical students reviewing the non-medication domains. Do you buy the following argument -- we need two reviewers for the non-medication domain because the two reviewers are students. We don't need two reviewers for the medication domain because the first reviewer is not a student, but a licensed pharmacist with lots of relevant experience. Do you think that argument would hold or, should we be safe and find a second reviewer?
I do think having a second reviewer would be ideal. KD is absolutely brilliant, but there are many nuances to this. I also think the perception of a pharmacist is not as strong as other clinician types given the huge spectrum of types of pharmacists adn variable training (e.g., walgreens versus clinical), thus your reviewer may wonder if the pharmacist was really credible - I absolutely would at least. I am confident a second reviewer will not be 100% consistent with what KD found, so I wonder if you are OK with someone doing a 10-20% sample or if you want a second person to review everything. If you are OK reporting an imperfect, but still positive interrater agreement, then I would go with a second reviewer looking at 10-20%, but if you want to come up with final mapping, then I would have a second person review all of it and I serve as the mediator for any disagreements so there...I am not 100% sure who the second reviewer should/could be, but my new fellow who started 7/1 is a possibility. I think you need a pharmacist (not student) for this type of work and not just any pharmacist - I am happy to reach out to folks if you want, or discuss further. Ideally we can find a trainee for cheap or a practicing clinical pharmacist who is willing to do at no cost, but I do want to find someone that I know is credible.
@trinklek - Sorry I somehow missed this! Thank you for the great information! OK, let's move forward with finding someone credible and trustworthy. Perhaps we can reassess whether they do 10-20% or all terms depending on who shows interest? We would also be able to pay them for their time, same rate as before.

@callahantiff I had an idea today of someone today. For this person, if they review a 10-20% sample, will participation in publication be an option? Your answer will help me pitch it correctly.


@trinklek - Sorry I missed this, apparently these types of messages were getting sent to spam. Absolutely, reviewing a sample of 10-20% totally constitutes participation in publications that use this work. 😄

@callahantiff
Copy link
Owner Author

Closing and archiving this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
source code verification verification of source string to source code mappings
Projects
None yet
Development

No branches or pull requests

3 participants