-
Notifications
You must be signed in to change notification settings - Fork 89
Update datasets.md #158
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update datasets.md #158
Conversation
Added recommended dataset profiles for four usecases
📦 Build Artifacts Available |
📦 Build Artifacts Available |
📦 Build Artifacts Available |
📦 Build Artifacts Available |
There was a problem hiding this 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 adds recommended dataset profiles for four use cases to the documentation and cleans up an extra blank line in the CLI utility.
- Added dataset profile examples (prompt/output token settings) for Chat, RAG, Summarization, and Code Generation in
docs/datasets.md
. - Removed an unnecessary blank line before the
get_metavar
method insrc/guidellm/utils/cli.py
.
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
File | Description |
---|---|
src/guidellm/utils/cli.py | Remove extra blank line before get_metavar method |
docs/datasets.md | Add recommended dataset profiles for four common use cases |
📦 Build Artifacts Available |
📦 Build Artifacts Available |
Added recommended dataset profiles for four usecases --------- Co-authored-by: Mark Kurtz <[email protected]> Signed-off-by: dalthecow <[email protected]>
Added recommended dataset profiles for four usecases --------- Co-authored-by: Mark Kurtz <[email protected]> Signed-off-by: dalthecow <[email protected]>
Added recommended dataset profiles for four usecases