Skip to content

Conversation

@nhz2
Copy link
Member

@nhz2 nhz2 commented Sep 16, 2025

This allows running a subset of jobs in a single process.

This is useful to avoid job or process limits on Zaratan.

The tests were generated by Claude

@nhz2 nhz2 requested a review from Copilot September 16, 2025 21:12
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR extends the batch functionality to support ranges of jobs instead of just single job indices or running all jobs. This allows users to run a subset of jobs in a single process, which is useful for avoiding job or process limits on compute clusters like Zaratan.

  • Updates CLI parsing to accept range syntax (e.g., "1:3", "begin:end", "2:3:10")
  • Replaces single batch index with batch range in data structures and processing logic
  • Adds comprehensive test coverage for various range formats and error conditions

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
src/cli-parsing.jl Implements range parsing logic and updates CLI option structure
src/run-sim.jl Updates job processing to iterate over batch range instead of single job
test/test_arg-parsing.jl Adds extensive test cases for range parsing and error handling
README.md Updates documentation to reflect new range functionality

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@nhz2 nhz2 merged commit 8a48598 into main Sep 16, 2025
9 checks passed
@nhz2 nhz2 deleted the streams branch September 16, 2025 21:42
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