Skip to content

Update blech_run_process workflow to generate parallel command on-the-fly - #790

Open
abuzarmahmood wants to merge 1 commit into
masterfrom
openhands/update-blech_run_process-workflow
Open

Update blech_run_process workflow to generate parallel command on-the-fly#790
abuzarmahmood wants to merge 1 commit into
masterfrom
openhands/update-blech_run_process-workflow

Conversation

@abuzarmahmood

Copy link
Copy Markdown
Member

Summary

This PR updates the blech_run_process workflow to no longer use temp files but to generate the parallel command on-the-fly.

Changes Made

Modified blech_run_process.sh to:

  1. Generate the electrode list dynamically from the layout file at runtime
  2. Read the max_parallel_cpu parameter from the .params file
  3. Generate and execute the parallel command directly instead of relying on the pre-generated temp/blech_process_parallel.sh file

Benefits

This change allows more flexible operation. If the number of parallel processes is changed in params, the temp files will no longer have to be updated manually. The workflow now:

  • Dynamically reads electrodes from the layout CSV file
  • Filters out EMG electrodes and those with "none"/"None"/"na" CAR groups
  • Uses the max_parallel_cpu parameter from the params file to determine parallelism

Testing

  • Verified bash syntax is correct using bash -n

Fixes

Closes #789

…-fly

This change removes the dependency on temp files for parallel processing.
Instead of using the pre-generated temp/blech_process_parallel.sh file,
the workflow now generates the parallel command dynamically based on:
- Electrodes from the layout file
- Parameters from the .params file (max_parallel_cpu)

This allows more flexible operation - if the number of parallel processes
is changed in params, the temp files will no longer have to be updated manually.

Co-authored-by: openhands <openhands@all-hands.dev>
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.

Update blech_run_process workflow to no longer use temp files but to generate parallel command on the fly

2 participants