Skip to content

Fix for issue #397#398

Closed
mb2055 wants to merge 1 commit intodevelfrom
fix_397
Closed

Fix for issue #397#398
mb2055 wants to merge 1 commit intodevelfrom
fix_397

Conversation

@mb2055
Copy link
Copy Markdown
Contributor

@mb2055 mb2055 commented Mar 21, 2025

Fix for issue #397 .

This fix ensures that each node input file has a unique identifier, preventing conflicts when multiple node instances are running simultaneously.

Fix for the output file is not ideal but I can't think of a better way to do it since we need to read the file_prefix from the args dictionary without knowing which of the arguments (if any) will be used to set the file_prefix when validate is called. The solution here is simply to assume that this will be set using an argument called file_prefix and that if this argument does not exist then the default name of output.yaml is used. It does not account for any situation in which the file_prefix is set using a value that is not the file_prefix argument.

Also adds a getNodeDirectory utility function that returns the current value of _node_dir .

@mb2055 mb2055 requested a review from lohedges March 21, 2025 14:57
@mb2055 mb2055 had a problem deploying to biosimspace-build March 21, 2025 14:58 — with GitHub Actions Failure
@mb2055 mb2055 temporarily deployed to biosimspace-build March 21, 2025 14:58 — with GitHub Actions Inactive
@mb2055 mb2055 had a problem deploying to biosimspace-build March 21, 2025 14:58 — with GitHub Actions Failure
@mb2055 mb2055 had a problem deploying to biosimspace-build March 21, 2025 14:58 — with GitHub Actions Failure
@mb2055 mb2055 temporarily deployed to biosimspace-build March 21, 2025 14:58 — with GitHub Actions Inactive
@mb2055 mb2055 had a problem deploying to biosimspace-build March 21, 2025 15:45 — with GitHub Actions Failure
@mb2055 mb2055 had a problem deploying to biosimspace-build March 21, 2025 15:45 — with GitHub Actions Failure
@mb2055 mb2055 had a problem deploying to biosimspace-build March 21, 2025 15:45 — with GitHub Actions Failure
@lohedges
Copy link
Copy Markdown
Contributor

Sorry for the delay with this, have been trying to think of a cleaner solution. Two options that I can think of:

  1. Allow the user to pass an ID or input file name to the run function.
  2. Run the process within a temporary workspace, allowing the user to specify a work_dir option as a kwarg to run. This would necessitate the use of absolute paths for inputs, though, but this could be documented.

@lohedges lohedges deleted the fix_397 branch July 1, 2025 13:05
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