TranslatorSRI / NodeNormalization

Service that produces Translator compliant nodes given a curie
MIT License
9 stars 6 forks source link

Adds the loading of the info-content data #99

Closed PhillipsOwen closed 2 years ago

PhillipsOwen commented 2 years ago

this PR adds the loading of a new cache for the information content data.

PhillipsOwen commented 2 years ago

I don't see where you are getting the ic out of redis?

When we do get it out, we may have a case where both conflated entities have their own ic, in which case we should choose the minimum one.

Do you really want to set all the databases to 0 in the config?

this part just gets the data into redis. i have not worked on getting it out of redis for the website. my idea is that we get the data loaded over the weekend and then work on pulling it out for the website monday.

all the redis caches use db 0. presumably there is only 1 db per redis instance.

YaphetKG commented 2 years ago

Connections, and redis setup look good. For local testing maybe having different db (0-6) on the default config might be a good idea (since the default config refers to a single redis instance )