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

Add DrugTarget Commons processor #1267

Closed
wants to merge 1 commit into from
Closed

Add DrugTarget Commons processor #1267

wants to merge 1 commit into from

Conversation

cthoyt
Copy link
Collaborator

@cthoyt cthoyt commented Apr 1, 2021

Closes #1257

Drug Target Commons builds heavily on the subset of ChEBML that is about drug-like or clincally tested molecules.

Questions

  • Do we want to keep chemicals that are not named? We've had a few discussions about this already, and I think the consensus is no.
  • Do we want to keep InChI keys as db_refs?
  • How much processor-side processing should we do for targets that aren't given with uniprot identifiers, but either as a name that looks like a uniprot id, a name that looks like a gene symbol, a name that looks like a long name, or a name that looks like a comma-separated list of gene/family names? @bgyori I'm sure that code for parsing a field of mixed stuff exists already, and I think it would be good to reuse that if possible. Even if some refactoring/generalization is necessary, this would be another good addition to the indra.ontology.standardize submodule

@cthoyt cthoyt closed this Apr 18, 2021
@cthoyt cthoyt deleted the add-dtc-processor branch April 18, 2021 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Drug Target Commons as source
1 participant