Skip to content

Conversation

@kasaandras
Copy link
Contributor

  • Addition of new test cases for RACE Parametization

  • ADSL.RACE: Race as a Function Parameter

Pull Request

Fixes #nnn

* 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>
@kasaandras
Copy link
Contributor Author

recheck

@shajoezhu
Copy link
Contributor

hi @kasaandras , were you going to update the cache data. thanks

Copy link
Contributor

@Melkiades Melkiades left a 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>
@Melkiades
Copy link
Contributor

@shajoezhu could you take a look again?

Copy link
Contributor

@shajoezhu shajoezhu left a comment

Choose a reason for hiding this comment

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

lgtm! thanks guys!

@shajoezhu shajoezhu merged commit 12bcf79 into insightsengineering:main Nov 13, 2025
28 checks passed
@github-actions github-actions bot locked and limited conversation to collaborators Nov 13, 2025
@shajoezhu
Copy link
Contributor

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

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.

4 participants