Fix clinicaltrials error by adding missing kwargs #482
2
−5
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
AidanCooper
requested review from
EugeneLightsOn,
malexw,
scottmx81,
tianjing-li and
a team
as code owners
AidanCooper
had a problem deploying
to
development
GitHub Actions
Failure
BeatrixCohere
had a problem deploying
to
development
GitHub Actions
Failure
|
Hey @BeatrixCohere, I think this should be good to merge? Let me know if there's anything else you need from me |
AidanCooper
force-pushed
the
fix-clinicaltrials
branch
from
0820923
to
d4d4c86
Compare
AidanCooper
had a problem deploying
to
development
GitHub Actions
Failure
|
Hey! You'll need to run |
ClinicalTrials.call() raises an error due an unexpected keyword argument, 'session'. Resolved by adding **kwargs to the call method. This PR also converts the method to async for consistency with the other community tools.
AidanCooper
force-pushed
the
fix-clinicaltrials
branch
from
d4d4c86
to
65fecac
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I'm not sure when this bug was introduced, but
ClinicalTrials.call()raises an error due an unexpected keyword argument, 'session'. Resolved by adding**kwargsto the call method. This PR also converts the method to async for consistency with the other community tools.AI Description
The
callmethod in theclinicaltrials.pyfile has been modified to include theasynckeyword, indicating that it is now an asynchronous function. This change allows the function to execute other code while waiting for external resources, improving the overall efficiency of the program.Code Changes:
callmethod has been modified to include theasynckeyword.**kwargsparameter has been added to thecallmethod to allow for flexible argument passing.