feat: new cli for batch jobs #754
Conversation
Deploying instructor with
|
| Latest commit: |
50eabf2
|
| Status: | ✅ Deploy successful! |
| Preview URL: | https://7a5f9188.instructor.pages.dev |
| Branch Preview URL: | https://add-batch-cli.instructor.pages.dev |
![ellipsis-dev[bot]](https://avatars.githubusercontent.com/in/64358?s=60&v=4){kind=small}
There was a problem hiding this comment.
❌ Changes requested. Reviewed everything up to 3badff5 in 1 minute and 15 seconds
More details
- Looked at
211lines of code in3files - Skipped
0files when reviewing. - Skipped posting
0drafted comments based on config settings.
Workflow ID: wflow_3LI3kzPyGPA6FGJO
Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.
There was a problem hiding this comment.
❌ Changes requested. Incremental review on 7a63dfb in 1 minute and 56 seconds
More details
- Looked at
23lines of code in1files - Skipped
0files when reviewing. - Skipped posting
0drafted comments based on config settings.
Workflow ID: wflow_5kWT7MemEldzO2Vz
Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.
There was a problem hiding this comment.
👍 Looks good to me! Incremental review on f0f3e54 in 3 minutes and 11 seconds
More details
- Looked at
49lines of code in1files - Skipped
0files when reviewing. - Skipped posting
0drafted comments based on config settings.
Workflow ID: wflow_bPuD4CewQk6OgWvI
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.
There was a problem hiding this comment.
👍 Looks good to me! Incremental review on db0ba28 in 2 minutes and 3 seconds
More details
- Looked at
49lines of code in2files - Skipped
0files when reviewing. - Skipped posting
2drafted comments based on config settings.
1. instructor/cli/batch.py:78
- Draft comment:
The use of...intyper.Optionis unnecessary when a help message is provided. It should be removed to avoid potential confusion.
file_path: str = typer.Option(help="File containing the batch job requests"),
- Reason this comment was not posted:
Confidence of 30% on close inspection, compared to threshold of 85%.
2. instructor/cli/batch.py:100
- Draft comment:
The use of...intyper.Optionis unnecessary when a help message is provided. It should be removed to avoid potential confusion.
batch_id: str = typer.Option(help="Batch job ID to cancel"),
- Reason this comment was not posted:
Confidence of 80% on close inspection, compared to threshold of 85%.
Workflow ID: wflow_6WUeznJX5ZWrqr2v
You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.
|
please add a lil recording and doc pages before we're ready to merge |
|
This is a video recording of the cli in action where we use a generated batch_recording.mp4Also added a docs page which shows a new snippet on how to generate these .jsonl files |
This reverts commit 79f6904.
|
When can we expect the next release to include this? Excited to use this batch CLI command |
|
@slavakurilyak we just released v1.3.4 which has it! |
This targets #604 by implementing two main new features
BatchJobclass which allows users to create.jsonlfiles which are compatible with OpenAI's new batch job functionSummary:
This PR introduces a new
BatchJobclass and CLI commands for managing batch jobs with OpenAI's API.Key points:
BatchJobclass ininstructor/batch.pyfor creating.jsonlfiles for batch jobs.instructor/cli/batch.pyto list, create, and cancel batch jobs.instructor/cli/cli.py.Generated with ❤️ by ellipsis.dev