NCATSTranslator / Relay

Autonomous relay system for NCATS Biomedical Data Translator
MIT License
5 stars 24 forks source link

Refactor Blocklist to be a map which includes a human-readable name #627

Closed MarkDWilliams closed 4 months ago

MarkDWilliams commented 5 months ago

Currently, the blocklist is only identifiers. As it grows, this may become harder to wrangle. Recommending that we change it to a map from id->name and just use the .keys() in place of the whole list in the remove_blocked() method

MarkDWilliams commented 4 months ago

To do this, you'll need to write a one off script to go through the blocklist.json, node norm it, get the preferred names, and then output a dictionary that maps the CURIE to the preferred label from node norm. Our blocklist code would work as is, but now on just blocklist.keys(). It won't need the names; those are just for human readability