10000 Fix #585 - add dynamic tooltips by mattroumaya · Pull Request #632 · Gilead-BioStats/gsm · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Fix #585 - add dynamic tooltips #632

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Aug 5, 2022
Merged

Fix #585 - add dynamic tooltips #632

merged 2 commits into from
Aug 5, 2022

Conversation

mattroumaya
Copy link
Contributor
@mattroumaya mattroumaya commented Aug 2, 2022 •
edited
Loading

Overview

Fix #585
Add GroupLabel and GroupID to tooltip

Test Notes/Sample Code

plt <- function(p) {
  p %>% plotly::ggplotly(tooltip = 'tooltip')
}

dfInput <- AE_Map_Adam()
SafetyAE <- AE_Assess(dfInput)
dfBounds <- Analyze_Poisson_PredictBounds(SafetyAE$dfTransformed, c(-5, 5))
Visualize_Scatter(SafetyAE$dfFlagged, dfBounds) %>% plt


Consent_Input <- Consent_Map_Raw()
Consent_Assess <- Consent_Assess(Consent_Input)
Visualize_Count(Consent_Assess$dfAnalyzed) %>% plt

@mattroumaya mattroumaya self-assigned this Aug 2, 2022
@mattroumaya mattroumaya requested a review from samussiah August 2, 2022 18:58
@mattroumaya
ffwd
72c74a2 
Merge branch 'dev' of github.com:Gilead-BioStats/gsm into fix-585

# Please enter a commit message to explain why this merge is necessary,
# especially if it merges an updated upstream into a topic branch.
#
# Lines starting with '#' will be ignored, and an empty message aborts
# the commit.
@samussiah samussiah merged commit a122a49 into dev Aug 5, 2022
@samussiah samussiah deleted the fix-585 branch August 5, 2022 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@samussiah samussiah samussiah approved these changes

Assignees

@mattroumaya mattroumaya

Labels
None yet
Projects
None yet 48FE
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Bugfix: Update Visualize_* functions to reference grouping variable.
2 participants
@mattroumaya @samussiah
0