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
New terms, issue #1176 #1231
New terms, issue #1176 #1231
Conversation
So far so good.
|
Thanks. I'll take care of the next step. It would have been better not to commit the deletion of |
I updated the imports and made some changes to the table. Please use Protege to add 'aspiration specimen collection' to There are still some problems to fix for the aspirate stuff. |
@apmody there are a bunch of changes to biobank-specimens.tsv that are making it difficult to see what's changed. Is there a reason quotes were added to the definition source field? Since these are TSVs, not CSVs, I don't think we need to enclose these in quotes. Side note, there's also some trailing space on some of those values (e.g. |
I'm pretty sure that I ran |
Do we want to keep them or should I clean that up? |
I think it's right now, and will be stable going forward. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The definitions need to be cleaned up a bit. I gave some examples on the first two, but mainly I think it would be good for these definitions to match the general pattern of this sheet:
A specimen that is collected with X from Y
Where X
is the method of collection (e.g., swab) and Y
is the anatomical area (e.g. cervix)
Also make sure all the definitions start with a capital "A".
I also think it would be good to have definition sources for them. See my comment about sources on line 70.
@apmody please disregard the review, I will update the definitions. Thanks! |
After updating the definitions, I re-ran |
New terms from issue #1176
Calling
make all
in following error that needs to be resolved:MANCHESTER PARSE ERROR the expression 'specimen and (is_specified_output_of some 'collecting specimen from organism') and ('derives from' some trachea)' at row 70, column 8 in table "src/ontology/templates/biobank-specimens.tsv" cannot be parsed: encountered unknown 'trachea'