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

Cell wikidata #329

Merged
merged 7 commits into from
Aug 7, 2024
Merged

Cell wikidata #329

merged 7 commits into from
Aug 7, 2024

Conversation

cbizon
Copy link
Contributor

@cbizon cbizon commented Aug 7, 2024

This hits wikidata to find CL/UMLS concordances via wikidata IDs. It writes a concord table between CL and UMLS.

It could also bring in wikidata id's but not sure if that's terribly useful.

Does a pretty good job; there are still lots of nodes that are either Cl or UMLS but cleans up many unmerged cliques.

@cbizon cbizon requested a review from gaurav August 7, 2024 13:25
gaurav added 2 commits August 7, 2024 16:21
Replaced GET -> POST, reformatted SPARQL query.
Copy link
Collaborator

@gaurav gaurav left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM -- thanks, Chris!

@gaurav gaurav merged commit d0502c6 into master Aug 7, 2024
@gaurav gaurav deleted the cell_wikidata branch August 7, 2024 20:47
gaurav added a commit that referenced this pull request Sep 26, 2024
* Updated RxNorm from 07012024 to 08052024.
* Updated ubergraph_iri_stem_to_prefix_map with additional IRI stems.
* Bug fix: `chemicals.write_pubchem_ids()` was failing on reaching the end of the file. Rewrote the code to account for this possibility.
* Updated the UMLS leftover output to include the `ic`, `clique_identifier_count`, `taxa`, `shortest_name_length` and `preferred_name` fields (still doesn't fix #126)
* Changed PubChem Compound download from FTP to HTTPS (using wget).
* Added a `--timestamping` option for wget recursive downloads to try to prevent re-downloading PubMed files if it hasn't changed.
* Updated method to create a prefix map from `curies`.
* Merged changes from #329 (this should go away once we merge the PRs before this and then merge it with master)
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.

2 participants