Skip to content
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

Docs: Update evaluation.md #1649

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Docs: Update evaluation.md #1649

wants to merge 1 commit into from

Conversation

thedmail
Copy link
Contributor

Fixes some rendering issues with code blocks and numbered (ordered) lists.

Description here... Help the reviewer by:

  • linking to an issue that includes more details
  • if it's a new feature include samples of how to use the new feature
  • (optional if issue link is provided) if you fixed a bug include basic bug details

Checklist (if applicable):

Fixes some rendering issues with code blocks and numbered (ordered) lists.
@thedmail thedmail requested a review from ssbushi January 22, 2025 23:15
@@ -278,11 +297,15 @@ The `eval:flow` command runs inference-based evaluation on an input dataset.
This dataset may be provided either as a JSON file or by referencing an existing
dataset in your Genkit runtime.

Copy link
Contributor

@ssbushi ssbushi Jan 23, 2025

Choose a reason for hiding this comment

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

Side note:

Could you please fix the note in the Schema validation section (Right - Line 260):

These examples may fail when the running an evaluation.

should be
These examples may fail when running an evaluation.

Copy link
Collaborator

Choose a reason for hiding this comment

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

please follow https://www.conventionalcommits.org/en/v1.0.0/ for pr title and commit message

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants