Skip to content

Conversation

@kasaandras
Copy link
Contributor

Pull Request

Fixes #nnn

@github-actions
Copy link
Contributor


🎉 Thank you for your contribution! Before this PR can be accepted, we require that you all read and agree to our Contributor License Agreement.
You can digitally sign the CLA by posting a comment on this Pull Request in the format shown below. This agreement will apply to this PR as well as all future contributions on this repository.


I have read the CLA Document and I hereby sign the CLA


0 out of 6 committers have signed the CLA.
@DariusTETSA
@Wall Alexandra
@pharma-dev-hub
@penard Nils
@kasaandras
@iolowilliams
Wall Alexandra, Penard Nils seem not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You can retrigger this bot by commenting recheck in this Pull Request. Posted by the CLA Assistant Lite bot.

@Melkiades
Copy link
Contributor

Thanks for your PR!

0 out of 6 committers have signed the CLA.
@DariusTETSA
https://github.com/Wall Alexandra
@pharma-dev-hub
@penard Nils
@kasaandras
@iolowilliams

This will make things complicated. I suggest starting with a new PR with only one commit from one user.

Could you run also a styler and lintr? Thanks!

Comment on lines +9242 to +9249
1 2021-10-13 01:00:00 269 CYCLE 2 DAY 1 2
2 2021-11-11 00:00:00 298 END OF INDUCTION 999
3 2020-10-09 00:00:00 -100 SCREENING -1
3 2020-10-09 01:00:00 -100 SCREENING -1
4 2021-01-17 00:00:00 0 BASELINE 0
5 2021-10-13 00:00:00 269 CYCLE 2 DAY 1 2
5 2021-10-13 01:00:00 269 CYCLE 2 DAY 1 2
6 2021-12-12 00:00:00 329 CYCLE 4 DAY 1 4
7 2021-11-11 00:00:00 298 END OF INDUCTION 999
8 2022-05-02 00:00:00 470 FOLLOW UP 999
8 2022-05-02 01:00:00 470 FOLLOW UP 999
Copy link
Contributor

Choose a reason for hiding this comment

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

hi @kasaandras , why is the time shift? thanks

@shajoezhu
Copy link
Contributor

hi @kasaandras , can you fix and pass the cicd checks. Thanks!

Comment on lines +19 to +21
#' @param race_list character vector of RACE
#' @param race_prob numeric vector of probabilities of being assigned to a RACE
#' @param female_prob (`proportion`)\cr Probability of a subject being female, male is calculated by `1-female_prob`.
Copy link
Contributor

Choose a reason for hiding this comment

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

Please try to keep the convention:

  • start with type of the data
  • Full sentence starting with a capital and ends with a dot.

@shajoezhu
Copy link
Contributor

hi @kasaandras , hope all is well. just want to check in and see how we doing on this PR, thanks

@pawelru
Copy link
Contributor

pawelru commented Jun 10, 2025

closing as per separate email conversation
we are awaiting a new one

@pawelru pawelru closed this Jun 10, 2025
@github-actions github-actions bot locked and limited conversation to collaborators Jun 10, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants