Skip to content

feat: timeouts#457

Open
tristan-f-r wants to merge 9 commits intoReed-CompBio:mainfrom
tristan-f-r:timeout-arg
Open

feat: timeouts#457
tristan-f-r wants to merge 9 commits intoReed-CompBio:mainfrom
tristan-f-r:timeout-arg

Conversation

@tristan-f-r
Copy link
Collaborator

@tristan-f-r tristan-f-r commented Jan 13, 2026

Adds timeout to algorithms as a demonstration of passing through errors. Closes #316.

Caveats:

  • This does not work with singularity (singularity has no docker wait equivalent and to implement timeouts in singularity would probably require constant polling of a detached thread)
  • ML requires at least one pathway, and failing pathways can break ML-work. How do we want to handle downstream analysis when errors occur (including in the future heuristic errors?)

@tristan-f-r tristan-f-r added the enhancement New feature or request label Jan 13, 2026
@read-the-docs-community
Copy link

read-the-docs-community bot commented Jan 13, 2026

Documentation build overview

📚 spras | 🛠️ Build #31218767 | 📁 Comparing c2febff against latest (18f2cf8)


🔍 Preview build

Show files changed (3 files in total): 📝 3 modified | ➕ 0 added | ➖ 0 deleted
File Status
genindex.html 📝 modified
fordevs/modules.html 📝 modified
fordevs/spras.html 📝 modified

@tristan-f-r tristan-f-r added the P-high This is a blocker for many PRs/issues/features label Jan 13, 2026
@github-actions github-actions bot added the merge-conflict This PR has merge conflicts. label Jan 31, 2026
@github-actions github-actions bot removed the merge-conflict This PR has merge conflicts. label Jan 31, 2026
@ntalluri
Copy link
Collaborator

ntalluri commented Feb 5, 2026

This does not work with singularity (singularity has no docker wait equivalent and to implement timeouts in singularity would probably require constant polling of a detached thread)

This is a problem; if this is something we are going to use for the benchmarking study we need this to work with singularity because CHTC only uses singularity/apptainer.

@tristan-f-r
Copy link
Collaborator Author

tristan-f-r commented Feb 5, 2026

I originally assumed this was a more esoteric PR to test the error-handling workflow. Though, based on the meeting just now, I'll look into a nice way to get this working with Singularity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request P-high This is a blocker for many PRs/issues/features

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Generic spras limit option on containers

2 participants