-
Notifications
You must be signed in to change notification settings - Fork 0
Check for equivalence of term labels for duplicate TermURLs across nodes #68
Copy link
Copy link
Open
Labels
_flag:stale[BOT ONLY] Flag issue that hasn't been updated in a while and needs to be triaged again[BOT ONLY] Flag issue that hasn't been updated in a while and needs to be triaged again
Description
federation-api/app/api/crud.py
Lines 163 to 165 in 9d685db
| for list_of_terms in cross_node_results: | |
| for term in list_of_terms[data_element_URI]: | |
| unique_terms_dict[term["TermURL"]] = term |
If we overwrite previous instances here, would it be reasonable to check first if the term is the same as the one we've already stored? If we ever encounter a situation where some jokester has a neurobagel node where they just make up their own TermURLs for our terms, we could end up with a tricky to debug situation if all that determines what we return here is the order in which we unpack the responses from nodes.
Originally posted by @surchs in #65 (comment)
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
_flag:stale[BOT ONLY] Flag issue that hasn't been updated in a while and needs to be triaged again[BOT ONLY] Flag issue that hasn't been updated in a while and needs to be triaged again
Type
Projects
Status
No status