search

NCATSTranslator / TranslatorArchitecture

MIT License
9 stars 11 forks source link

Response for node inference #60

Closed cbizon closed 3 years ago

cbizon commented 3 years ago

Based on slack poll responses.

andrewsu commented 3 years ago

Just to confirm, this PR is meant to reflect what is graphically shown on slide 4 here? https://docs.google.com/presentation/d/1FrAVxIqwrpqlTrcRkdvhRFG9Tysw9o2XPAk6SiL11aE/edit#slide=id.gd577682d07_0_0

suihuang-ISB commented 3 years ago

same question as @andrewsu. The SLACK survey options mapping to the slides was not entirely clear because slide 3 is part of the problem description, and NOT one of the proposed solutions which start with slide 4. (Slide 3 could also be one solution albeit not ideal, if admitting fuzzy mapping, right?)

cbizon commented 3 years ago

Yes, the text is meant to correspond to slide 4. If that's not clear, then perhaps the wording could be clarified? Or maybe we need to include some example queries and responses for this section?

andrewsu commented 3 years ago

For posterity, this is a screenshot from the slides that shows a graphical representation of what this PR is meant to express.

image

cbizon commented 3 years ago

Thanks @andrewsu I think that's nicer...