Skip to content

Conversation

@JavierCladellas
Copy link
Collaborator

Added timeout field to configs:

"timeout":"<days>-<hours>:<minutes>:<seconds>" #e.g "0-0:15:0" for 15 minutes

Format is validated accordingly

The timeout is directly added to the slurm directive as

#SBATCH --time=0-0:15:0

@JavierCladellas JavierCladellas added enhancement A new feature or request benchmarking Issues related to benchmarking labels Nov 18, 2024
@JavierCladellas JavierCladellas self-assigned this Nov 18, 2024
@JavierCladellas JavierCladellas linked an issue Nov 18, 2024 that may be closed by this pull request
@netlify
Copy link

netlify bot commented Nov 18, 2024

Deploy Preview for benchmarking-polite-crostata-92f389 ready!

Name Link
🔨 Latest commit 4678567
🔍 Latest deploy log https://app.netlify.com/sites/benchmarking-polite-crostata-92f389/deploys/673c58aded04c1000808d993
😎 Deploy Preview https://deploy-preview-138--benchmarking-polite-crostata-92f389.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@JavierCladellas JavierCladellas changed the title 130 add timeout resolves: 130 add timeout Nov 18, 2024
@JavierCladellas JavierCladellas merged commit 3693b8e into master Nov 19, 2024
15 checks passed
@JavierCladellas JavierCladellas deleted the 130-add-timeout branch November 19, 2024 10:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

benchmarking Issues related to benchmarking enhancement A new feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add timeout

3 participants