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

feat!: use regbot for clinical trial fetching #89

Merged
merged 4 commits into from
Dec 31, 2024

Conversation

jsstevenson
Copy link
Contributor

@jsstevenson jsstevenson commented Dec 26, 2024

progress on #59
close #86
close #85

@jsstevenson jsstevenson added the priority:medium Medium priority label Dec 26, 2024
korikuzma
korikuzma previously approved these changes Dec 27, 2024
Copy link

@korikuzma korikuzma left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Minor suggestions

Copy link

This PR is stale because it has been open 3 day(s) with no activity. Please review this PR.

@github-actions github-actions bot added the stale label Dec 30, 2024
@jsstevenson
Copy link
Contributor Author

also tagging @mcannon068nw

korikuzma
korikuzma previously approved these changes Dec 30, 2024
def _add_study_to_output(output: dict[str, list], drug_name: str, study: Study) -> None:
"""Update `output` in-place with results from study

:param output: in-progress raw columnar data

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

missing drug_name

@jsstevenson jsstevenson merged commit 659d5a4 into main Dec 31, 2024
18 checks passed
@jsstevenson jsstevenson deleted the refactor-clin-trials branch December 31, 2024 13:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
2 participants