main to dev#45
Merged
Merged
Conversation
* Set up release infrastructure. (#42) * Set up release infrastructure. Note that this is still intended to merge into `dev`. After this, we should follow its advice and create an `rc` branch to PR into `main`. But also note that R CMD Check freaks out if there isn't a vnum anywhere in news, so I'm prematurely tagging the expected vnum there on purpose. - Closes #40. * Update pkgdown-pr.yaml to match gsm.app. * {gsm.ae} v0.1.0 RC This initial release provides a dashboard that summarizes trends in Site- and Country-level adverse event and severe adverse event data. The data is presented as a series of panels, which can be expanded to show additional information (as demonstrated in the "Prevalence" panel). ## New Features - Summary Panel: The "Summary"" panel presents summary data for AE and SAE at the Study-, Group- (Site or Country), and Participant-level, reacting to the selections of the user. If multiple snapshots are available, the current data is compared to the previous snapshot. (#9, #12, #21, #23, #24, #25, #26, #34, #36, #37) - Prevalence Panel: The "Prevalence" panel presents plots showing the prevalence (as % of all AEs) for categorical variables such as whether the AE is "Serious?" or the tagged "Toxicity Grade", with bars by Study, Group (Site or Country), and Participant, reacting to the selections of the user. Expanding this panel demonstrates the possibility of different views in this dashboard based on whether or not the panel is expanded. In this case, the view adds a table of all values for the selected categorical variable, sorted by prevalence at the Study level. (#10, #11, #22, #29, #33, #35) - Timeline Panel: The "Timeline" panel shows counts of adverse events by month for data variables (such as "AE Entry Date", "AE Start Date", and "AE End Date"), faceted at the Study-, Group- (Site or Country), and Participant-level, reacting to the selections of the user. (#16) * Update shinyapps-deploy/terminate GHA. * Fix if. * Run pkgdown check on main, too. And simplify the action, I think. * Remove typo Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * Test with dev gsm.app. Don't merge until gsm.app is pushed to main, and then update this Remotes reference. * Bump gsm.app version requirement. * update shiny and gsm.app dependency versions in Description * Fix `NA` bug in `mod_AETimeline.R`. * Filter out missing dates and don't filter out old events. * Fix axes to protect against squishing, and (temporarily) remove labels in timeline. I don't usually like to leave commented-out code like this, but I'd like to keep the info I have there in order to continue the fix in the next iteration. * Add y-axis labels Re-remove those when/if we add labels back. --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Laura Maxwell <laura.maxwell@atorusresearch.com> Co-authored-by: Spencer Childress <spencer.c.childress@gmail.com>
Codecov ReportAll modified and coverable lines are covered by tests ✅
🚀 New features to boost your workflow:
|
|
🚀 Deployed App: https://rinpharma.shinyapps.io/gsm-ae-pr-45 |
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Reconcile dev with main.