Skip to content
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

Seed handling #16

Open
astamm opened this issue Dec 12, 2024 · 0 comments
Open

Seed handling #16

astamm opened this issue Dec 12, 2024 · 0 comments

Comments

@astamm
Copy link
Collaborator

astamm commented Dec 12, 2024

This is a bad practice to set the seed via an optional argument.
It is usually advised to let the user decide and use {withr} for handling seeds without modifying the user's local environment.
For the plausibility function, it is important to assess its value for each value of the parameters of interest using the same permutations.
@ManonSimonot can you investigate the possible of removing the seed argument in favour of the argument nperm which could be:

  • either an integer value in which case nperm permutations are randomly drawn;
  • or a list of permutations that would have been previously sampled and stored;
  • or a vector of integers giving the random indices of permutations to select within the list of all possible permutations (more memory-efficient than second option).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant