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

add gee prototype module with example #664

Merged
merged 14 commits into from
Nov 30, 2022
Merged

add gee prototype module with example #664

merged 14 commits into from
Nov 30, 2022

Conversation

danielinteractive
Copy link
Contributor

@danielinteractive danielinteractive commented Oct 28, 2022

complements insightsengineering/tern.gee#2

Please don't update from main or merge. We want to base this on the current release.

@danielinteractive
Copy link
Contributor Author

Hi @shajoezhu fyi - this is a first prototype module. I would like to use this to show to the project folks next week, and then work with @cicdguy to deploy an initial version internally for project team playing around with it. your team can then help us to further improve and productionize it

@shajoezhu
Copy link
Contributor

brilliant! thanks @danielinteractive , just looping @JiaLiu0001 into this thread as well, who will be developing the teal app for the crovab team.

@shajoezhu
Copy link
Contributor

shajoezhu commented Nov 11, 2022

error with

Listening on http://127.0.0.1:7133
Warning: Error in teal.modules.clinical:::arm_ref_comp_observer: unused arguments (datasets = datasets, dataname = parentname)
  3: runApp
  2: print.shiny.appobj
  1: <Anonymous>

fixed

@shajoezhu shajoezhu mentioned this pull request Nov 12, 2022
10 tasks
@mhallal1 mhallal1 mentioned this pull request Nov 23, 2022
merges into
#664

The aim here is to:
1) Add `qenv`
2) Add reporter

I also created the `template` for this module to be in line with other
tmc modules.
R/tm_a_gee.R Outdated
substitute(
expr = {
model_fit <- tern.gee::fit_gee(
vars = tern.gee:::vars_gee(
Copy link
Collaborator

Choose a reason for hiding this comment

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

does vars_gee need to be exported from tern.gee?

edelarua and others added 5 commits November 25, 2022 05:57
close #694

<!--- Replace `#nnn` with your issue link for reference. -->

Fixes #nnn

Signed-off-by: Joe Zhu <sha.joe.zhu@gmail.com>
Co-authored-by: Emily de la Rua <59304861+edelarua@users.noreply.github.com>
Signed-off-by: Joe Zhu <sha.joe.zhu@gmail.com>
@shajoezhu shajoezhu added the sme label Nov 30, 2022
@github-actions
Copy link
Contributor

github-actions bot commented Nov 30, 2022

Unit Tests Summary

    1 files    33 suites   4s ⏱️
146 tests 146 ✔️ 0 💤 0
162 runs  162 ✔️ 0 💤 0

Results for commit 1bf1ebf.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@edelarua edelarua left a comment

Choose a reason for hiding this comment

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

LGTM once all checks are passing

@shajoezhu shajoezhu merged commit 0189b7a into main Nov 30, 2022
@shajoezhu shajoezhu deleted the 2_prototype branch November 30, 2022 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants