Closed matentzn closed 4 years ago
@matentzn
I also believe that we need a better approach for the
cov:internal_covoc_id
. If there can be multiple, shouldn't there be one primary id? @paolaroncaglia My suggestion:1. `cov:internal_covoc_id` for the 1 main internal id 2. `cov:alt_covoc_id` for the remaining alternative ones.
But its up to you.
Submitters sometimes entered requests for a same term in two different tabs, or even in the same tab but in separate rows, e.g. one row for singular form of term and another row for plural. In such cases, I "merged", and the decision as to which id is primary is really arbitrary. We could say that the singular form should have primary id and that in duplications across tabs, the primary id should be for the tab that's most relevant (not always applicable though). Going back to that would take time though. They need all ids so they don't lose info wrt their originary vocabulary. How important is it to address this issue before tomorrow's release please? :-)
Thats ok then, really up to you!
@zoependlington Shall we just leave the ID as is then? Not even a split? Leaving:
property_value: http://purl.obolibrary.org/obo/covoc/internal_covoc_id "DIS_37||BMV_77" xsd:string
Maybe they prefer this, or we can instruct them how to edit the field themselves.
But again, whatever you prefer. No strong feelings here.
(however I would add the SPLIT to the drug-disease thing for sure)
I already added the split for both, but I can remove it again if we decide we don't want it split for the IDs
Nono, if you did, leave it! Just wanted to avoid work. Its fine! We dont change anything. If the split works, we can close this issue here.
Example 1:
Example 2:
As you can see, the SPLIT || is not configured correctly for these (cov:internal_covoc_id, cov:drug_associated_with_disease).
I also believe that we need a better approach for the
cov:internal_covoc_id
. If there can be multiple, shouldn't there be one primary id? @paolaroncaglia My suggestion:cov:internal_covoc_id
for the 1 main internal idcov:alt_covoc_id
for the remaining alternative ones.But its up to you.