Skip to content

feat: module for clinical trials function (parity with dgipy) - #50

Open
mcannon068nw wants to merge 5 commits into
mainfrom
clinical-trial-function
Open

mcannon068nw wants to merge 5 commits into
mainfrom
clinical-trial-function

Conversation

@mcannon068nw

Copy link
Copy Markdown
Contributor

Reviewer request to bring parity with dgiPy

using GDC-0199 clinical trials as an example: https://clinicaltrials.gov/search?intr=GDC-0199&viewType=Card

in RStudio console:

devtools::load_all()
trials <- getClinicalTrials("GDC-0199")
View(trials)

image

@mcannon068nw

Copy link
Copy Markdown
Contributor Author

Rohit is out at the moment, not sure who our best R person is so pulling in all eyes

@jsstevenson jsstevenson left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

just eyeballing it, this might produce results that are sliiightly different from the python lib but probably close enough

Comment thread R/clinical-trials.R
httr2::req_url_query(
`query.intr` = term,
pageSize = 1000,
format = "json"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

should set a 30s timeout here

@jsstevenson
jsstevenson self-requested a review October 8, 2026 14:46

@jsstevenson jsstevenson left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

also make sure to update docs/vignettes/etc (for bioconductor requirements) and rebuild them

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants