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

loadRegistry assigns exported objects into the global namespace #304

Open
sebffischer opened this issue Dec 2, 2024 · 1 comment
Open

Comments

@sebffischer
Copy link

sebffischer commented Dec 2, 2024

This is not documented in loadRegistry.
But as @berndbischl noted, it is documented in batchExport.

library(batchtools)
path = tempfile()
reg = makeExperimentRegistry(path)
#> No readable configuration file found
#> Created registry in '/var/folders/ft/n79895td0xn0gpr6ny8jyh800000gn/T/RtmpAFvW7R/file8eef2b73136b' using cluster functions 'Interactive'

batchExport(list(a = 123))
#> Exporting new objects: 'a' ...


a = 345

loadRegistry(path)
#> Reading registry in read-only mode.You can inspect results and errors, but cannot add, remove, submit or alter jobs in any way.If you need write-access, re-load the registry with `loadRegistry([...], writeable = TRUE)`.
#> No readable configuration file found
#> Experiment Registry
#>   Backend   : Interactive
#>   File dir  : /var/folders/ft/n79895td0xn0gpr6ny8jyh800000gn/T/RtmpAFvW7R/file8eef2b73136b
#>   Work dir  : /private/var/folders/ft/n79895td0xn0gpr6ny8jyh800000gn/T/RtmpJ9OtV1/reprex-8e786d06a68d-frail-tick
#>   Jobs      : 0
#>   Problems  : 0
#>   Algorithms: 0
#>   Seed      : 3916
#>   Writeable : FALSE

a
#> [1] 123

Created on 2024-12-02 with reprex v2.1.1

@sebffischer
Copy link
Author

I would even argue that it would be more intuitive to not do this at all, at least by default.

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