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

25 Design Doc for predprobDist.R #27

Merged
merged 17 commits into from
Dec 6, 2023
Merged

25 Design Doc for predprobDist.R #27

merged 17 commits into from
Dec 6, 2023

Conversation

audreyyeoCH
Copy link
Collaborator

No description provided.

@audreyyeoCH audreyyeoCH marked this pull request as draft December 1, 2023 16:20
@audreyyeoCH
Copy link
Collaborator Author

Need to break the function into several issues, currently PR in draft mode

@audreyyeoCH audreyyeoCH changed the title 25 General Clean predprobDist 25 Design Doc for predprobDist Dec 4, 2023
@audreyyeoCH audreyyeoCH changed the title 25 Design Doc for predprobDist 25 Design Doc for predprobDist.R Dec 4, 2023
@audreyyeoCH
Copy link
Collaborator Author

Hi @danielinteractive, I have managed to create two helper functions. Could you have a look ? I thought to start an issue for one of the helper functions, and start doing the test file as well for it

@danielinteractive danielinteractive self-assigned this Dec 6, 2023
Copy link
Collaborator

@danielinteractive danielinteractive left a comment

Choose a reason for hiding this comment

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

Thanks @audreyyeoCH yeah let's first get this PR finished and merged, afterwards can start new issues / PRs for the actual changes in the package then i.e. adding and using the new helper functions.

design/design-doc_predprobDist.Rmd Outdated Show resolved Hide resolved
design/design-doc_predprobDist.Rmd Show resolved Hide resolved
R/predprobDist.R Outdated Show resolved Hide resolved
inst/WORDLIST Outdated Show resolved Hide resolved
audreyyeoCH and others added 2 commits December 6, 2023 18:00
Co-authored-by: Daniel Sabanes Bove <[email protected]>
Co-authored-by: Daniel Sabanes Bove <[email protected]>
@danielinteractive danielinteractive marked this pull request as ready for review December 6, 2023 17:06
@audreyyeoCH audreyyeoCH merged commit 1212d38 into main Dec 6, 2023
3 checks passed
@audreyyeoCH audreyyeoCH deleted the 25_predprobDist branch December 6, 2023 17:16
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.

2 participants