Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Workflow B: ARAX not sending >100 input CURIES during expansion #317

Open
karafecho opened this issue Sep 30, 2021 · 7 comments
Open

Workflow B: ARAX not sending >100 input CURIES during expansion #317

karafecho opened this issue Sep 30, 2021 · 7 comments
Assignees

Comments

@karafecho
Copy link
Contributor

This issue is to report that ARAX is not calling KPs when there are more than 100 input CURIEs in the expansion.

@dkoslicki
Copy link
Member

Yes, that is by design. We have found that most KPs are not performant enough to handle such queries within the default time-out. So instead of waiting for them to time out, we do not decide to even attempt. @amykglen may have more insight

@karafecho
Copy link
Contributor Author

karafecho commented Oct 4, 2021

Yes, I understand that the restriction is by design. I'm wondering, however, if you might be able to suggest a workaround that would allow ARAX to pull in additional KPs? Perhaps by extending the default timeout? Better leveraging the async query option? Or, is this strictly a KP issue in your mind?

@dkoslicki
Copy link
Member

I think it's mainly a KP performance issue, but @amykglen may be able to shed more light on possible workarounds.

@amykglen
Copy link

amykglen commented Oct 5, 2021

yes, we're going to discuss this at our team meeting tomorrow - will post back here after that.

@karafecho
Copy link
Contributor Author

Great! Thanks, David and Amy!

@amykglen
Copy link

amykglen commented Oct 8, 2021

ok, ARAX now sends queries to KPs even if they have >100 input curies. we're also revisiting our default timeouts and expect to increase those within the next week!

@karafecho
Copy link
Contributor Author

This is great! Thanks, Amy.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants