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

add more parallel options #152

Open
k-doering-NOAA opened this issue Jul 25, 2023 · 1 comment
Open

add more parallel options #152

k-doering-NOAA opened this issue Jul 25, 2023 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@k-doering-NOAA
Copy link
Collaborator

Suggestion On Iago's SSMSE JOSS review:

Packages 'foreach' and 'doParallel' are only 'Suggests' in DESCRIPTION, but foreach() is used in package code, although its presence is checked. If option 'parallel=TRUE' is chosen, the code sets up a parallel cluster using parallel::makeCluster, when other backends could be employed, e.g. doFuture. An alternative option would be to import the 'foreach' package, and leave the setup to the user. For example, a Linux user would choose a 'FORK' cluster rather than the default 'PSOCK', for efficiency reasons.

@nathanvaughan-NOAA is working on some other parallel code for another project, so we can build upon his work after it is done.

@k-doering-NOAA
Copy link
Collaborator Author

I was also recently made aware of parallelly which offers a enhanced option of parallel::detectCores() (parallelly::availableCores())

@k-doering-NOAA k-doering-NOAA added the enhancement New feature or request label Mar 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants