-
-
Notifications
You must be signed in to change notification settings - Fork 10
Tsk12 race parameters (#13) #333
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
Conversation
* Addition of new test cases for RACE Parametization * ADSL.RACE: Race as a Function Parameter
* Addition of new test cases for RACE Parametization * ADSL.RACE: Race as a Function Parameter * Stye and Linter fixe for RACE parametization * Linter fixe --------- Signed-off-by: DariusTETSA <46453678+DariusTETSA@users.noreply.github.com>
|
recheck |
|
hi @kasaandras , were you going to update the cache data. thanks |
Melkiades
left a comment
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.
Lgtm! Thanks for the addition ;)
* Addition of new test cases for RACE Parametization * ADSL.RACE: Race as a Function Parameter * Stye and Linter fixe for RACE parametization * Linter fixe * Have added two extra tests for when you only provide one for the Race parameters (race_list or race_prob). Also did some minor fixe on the Race parameter on the radsl * Update R/radsl.R Improve comment wording. Co-authored-by: AlexandraWall <Alexandrawall09@hotmail.co.uk> Signed-off-by: DariusTETSA <46453678+DariusTETSA@users.noreply.github.com> --------- Signed-off-by: DariusTETSA <46453678+DariusTETSA@users.noreply.github.com> Co-authored-by: AlexandraWall <Alexandrawall09@hotmail.co.uk>
|
@shajoezhu could you take a look again? |
shajoezhu
left a comment
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.
lgtm! thanks guys!
|
hi @kasaandras i m adding you to the ucb_collab team, which will have write access to this repo, there is an inivte on the way, you will then be able to trigger the build next time |
Addition of new test cases for RACE Parametization
ADSL.RACE: Race as a Function Parameter
Pull Request
Fixes #nnn