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

Compiled workers - Horace #794

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Compiled workers - Horace #794

wants to merge 1 commit into from

Conversation

mducle
Copy link
Member

@mducle mducle commented Mar 21, 2022

Adds a compiled worker which works with herbert, parpool and mpiexec_mpi type clusters (not verified on slurm_mpi yet but should also work there...). These changes require the equivalent Herbert PR#434 to work.

The changes include removing a check in @hpc_config that a worker must be a Matlab command - if it is not a Matlab function, and it is a valid filename or system command it will be accepted (see the Herbert PR for details).

The compiled worker is not built by default. At the moment to build it you must do:

mkdir build && cd build && cmake .. && cmake --build . --target build_worker

Then specify the built worker in @parallel_config:

ppc = parallel_config; ppc.worker = '<path/to/Horace>/build/admin/compiled_worker.exe';

And run gen_sqw etc to see that it works.

@mducle mducle requested a review from abuts March 21, 2022 08:51
@mducle mducle changed the base branch from rel_3_6 to master April 4, 2022 15:59
@mducle mducle changed the base branch from master to main June 15, 2022 22:26
@oerc0122 oerc0122 force-pushed the compiled_workers_horace branch from 6577b07 to fc0906d Compare January 4, 2024 14:51
@oerc0122 oerc0122 force-pushed the compiled_workers_horace branch from fc0906d to 164e2df Compare January 4, 2024 14:58
@oerc0122 oerc0122 changed the base branch from main to master January 4, 2024 15:00
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

Successfully merging this pull request may close these issues.

2 participants