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.

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

Develop fixes tested on IPCI and RMD #9

Merged
merged 45 commits into from
Nov 20, 2023
Merged

Develop fixes tested on IPCI and RMD #9

merged 45 commits into from
Nov 20, 2023

Conversation

cebarboza
Copy link
Collaborator

  • Bug fixes
  • RMD passing
  • eventCohorts 101 to 114
  • Submitting for review

Ross Williams and others added 30 commits November 15, 2023 02:03
added initial functionality to run the treatment patterns analysis. N…
737763c 
…eeds to be updated for the cohorts
updated run study and renamed arguments for consistency
074ce40
put tp trigger in codeToRun and runStudy
77f4d30
removed print statement from createCohorts
f46f161
@MaximMoinat
add launching of Shiny app
e0f7055
@MaximMoinat
add treatment cohorts
6c71853
@MaximMoinat
update treatment cohorts
f91d7a3
@MaximMoinat
update treatment cohorts
d0ff72d
@MaximMoinat
update cohort definition set
94c1ae0
added treatmentpatterns analysis
a69e035
Merge branch 'develop' of https://github.com/ohdsi-studies/EhdenAlopecia
1ccafed 
 into develop
@rossdwilliams
fix typo
a8e949c
@cebarboza
cohorts only with counts
2b147de
added subfolder saving
d30d80b
Merge branch 'develop' of https://github.com/ohdsi-studies/EhdenAlopecia
02913e2 
 into develop
@cebarboza
edhen friday
3134205
@cebarboza
fixes style
06aa5dc
@cebarboza
fixes imports
304bbef
@cebarboza
passing check
67944d5
@cebarboza
code to run here
cec5be3
@cebarboza
gha badge rmd
5154824
@cebarboza
check develop
449392c
@cebarboza
commit
bdbc865 
Merge branch 'develop' of https://github.com/ohdsi-studies/EhdenAlopecia into develop

# Conflicts:
#	R/runStudy.R
@cebarboza
remotes added cr, cg and cd
d424dbb
@cebarboza
description remotes correction
f42979b
@cebarboza
fixing badge
ee503ac
@cebarboza
cohort
84b1760
@cebarboza
fix outputSubDir
f90f42f
@cebarboza
execute Treatment not segmented
7bc1a94
@cebarboza
suggested here fix
853223a
@cebarboza cebarboza merged commit 6e08b97 into main Nov 20, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rossdwilliams rossdwilliams rossdwilliams approved these changes

@MaximMoinat MaximMoinat Awaiting requested review from MaximMoinat

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@cebarboza @rossdwilliams @MaximMoinat