-
Notifications
You must be signed in to change notification settings - Fork 8
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
change fixedBlockRand
to randomize_by_fixed_block
#62
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Instead of rand_by_fixed_block
, let's use randomize_by_fixed_block
.
Good suggestion! @lili-ling-msd, let's change the function name to |
fixedBlockRand
to rand_by_fixed_block
fixedBlockRand
to randomize_by_fixed_block
Merge branch '32-fixedblockrand-rand_by_fixed_block' of https://github.com/Merck/simtrial into 32-fixedblockrand-rand_by_fixed_block # Conflicts: # R/rand_by_fixed_block.R
Hi @nanxstats, could you please approve this PR? Since I initially created this PR as a demo for Lili and Xintong, I am not allowed to approve it now. Thanks! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the patch - approved.
No description provided.