JuliaParallel / JuliaParallel/MPIClusterManagers.jl

Update README.md

Open
#25 1 comment 0 reactions 0 assignees View on GitHub
Dominant language
Julia
Stars
47
Forks
7
PR merge metrics
No merged PRs in 30d

Description

[README.md](https://github.com/JuliaParallel/MPIClusterManagers.jl/blob/master/README.md) needs to be updated.

For example it states the following usage `MPIManager(;np=Sys.CPU_THREADS, mpi_cmd=false, launch_timeout=60.0)` which is wrong.
- [v0.1.0](https://github.com/JuliaParallel/MPIClusterManagers.jl/tree/v0.1.0) has an optional parameter `mpirun_cmd`. [Constructor signature on v0.1.0](https://github.com/JuliaParallel/MPIClusterManagers.jl/blob/a1ac47dd28472f4aeb827ccd4e9985dc5c7109a9/src/mpimanager.jl#L53)
- [v0.2.0](https://github.com/JuliaParallel/MPIClusterManagers.jl/tree/v0.2.0) does not offer such parameter. [Constructor signature on v0.2.0](https://github.com/JuliaParallel/MPIClusterManagers.jl/blob/0efc89d36233d9e825c2f930787f76985dcb767e/src/mpimanager.jl#L52)

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.