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

compile_dll() sets the .Random.seed #185

Closed
MichaelChirico opened this issue Apr 2, 2024 · 2 comments
Closed

compile_dll() sets the .Random.seed #185

MichaelChirico opened this issue Apr 2, 2024 · 2 comments

Comments

@MichaelChirico
Copy link
Contributor

Copied from r-lib/pkgload#272

In R --vanilla

ls(all=TRUE)
# character(0)

# any (?) package with src/ code, I've tried roxygen2, Rcpp, and data.table
#   but _not_ on lint, which doesn't have src/ code
pkgload::load_all()

ls(all=TRUE)
# [1] ".Random.seed"

Is that intended/unavoidable behavior? It may harm reproducibility. OTOH, I can't think of a place where this causes an issue, it's just something I noticed when tinkering around on another problem. So don't feel obligated to devote too much effort to this.

@MichaelChirico
Copy link
Contributor Author

MichaelChirico commented Apr 2, 2024

Traced down to how {processx} behaves and filed an issue there:

r-lib/processx#386

Given the root cause, it may be up to {pkgbuild} to decide whether this behavior is working as intended & possibly document it, or else be careful to restore state on exit.

@MichaelChirico
Copy link
Contributor Author

downstream fixed, nothing to do here

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