search

RTXteam / RTX

Software repo for Team Expander Agent (Oregon State U., Institute for Systems Biology, and Penn State U.)
https://arax.ncats.io/
MIT License
33 stars 21 forks source link

Limit to 250 results #2288

Closed dkoslicki closed 4 days ago

dkoslicki commented 3 months ago

Tyler stipulates that ARAs should return no more than 250 results. I think we can address this in the query graph interpreter (change all the filter results from 500 to 250)

edeutsch commented 3 months ago

Where/when was this stipulated? I know it was discussed, and we decided to try it to see if it improved speed of the system at all. But it seems like there should be some timing tests before we all do this.

dkoslicki commented 3 months ago

https://ncatstranslator.slack.com/archives/C068DQ6688L/p1718113199119099

edeutsch commented 3 months ago

Great, thanks, I was not subscribed to that channel. I think Translator might be over-channeling a bit. I am now subscribed. Slightly ironic that the only decision there posted is one that I remember a bit differently. I've asked for clarification from Tyler. We may not want to proceed on this quite yet until we're all in agreement.

dkoslicki commented 4 days ago

Closing as implemented and the tests now look at top X% or in top ~30 results