Skip to content

Fix: Failed run due to old bindpaths; increased pipeline walltime#107

Merged
kelly-sovacool merged 7 commits intomainfrom
iss-106
Sep 10, 2025
Merged

Fix: Failed run due to old bindpaths; increased pipeline walltime#107
kelly-sovacool merged 7 commits intomainfrom
iss-106

Conversation

@samarth8392
Copy link
Contributor

@samarth8392 samarth8392 commented Sep 10, 2025

Changes

  • Removed old bindpaths and fixed /gs10 doesn't exist error
  • Increased pipeline walltime for slurm submission to 100 hours
  • Minor documentation updates for setup and running LOGAN

Issues

Fixes #106
Fixes #88

PR Checklist

(Strikethrough any points that are not applicable.)

  • This comment contains a description of changes with justifications, with any relevant issues linked.
  • [ ] Write unit tests for any new features, bug fixes, or other code changes. testing framework not yet implemented
  • [ ] Update docs if there are any API changes.
  • [ ] If a new nextflow process is implemented, define the process container and stub.
  • Update CHANGELOG.md with a short description of any user-facing changes and reference the PR number. Guidelines: https://keepachangelog.com/en/1.1.0/

@samarth8392 samarth8392 marked this pull request as ready for review September 10, 2025 15:20
@kelly-sovacool kelly-sovacool merged commit 6b890d2 into main Sep 10, 2025
5 checks passed
@kelly-sovacool kelly-sovacool deleted the iss-106 branch September 10, 2025 16:07
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.

LOGAN stub/dryrun failed Show logan run usage in docs instead of nextflow run

2 participants