Skip to content

Conversation

@Th3Bust3r
Copy link

Added 'stratified' as an alternative sampling method. This stratifies the locations in rank_by_loc and then randomizes the cell assignment within each strata.

The current 'systematic' method only randomizes sample(1:k,k) once, and then repeats that sequence down the rank_by_loc. This produces identical groupings of markets in data_aux for MultiCellMarketSelection each time we run it, even though the cell number labels may differ. This is a potentially significant source of bias when running MultiCellMarketSelection for subsequent experiments with the same number of cells. By randomizing at each strata, we can ensure the market groupings in data_aux will be truly different by setting a different seed.

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants