Skip to content

feat: eval - #68

Open
schiwekM wants to merge 33 commits into
mainfrom
feat/eval
Open

feat: eval#68
schiwekM wants to merge 33 commits into
mainfrom
feat/eval

Conversation

@schiwekM

@schiwekM schiwekM commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Add evaluation helpers to easily test agents.
Integrated with MLFlow to view results.

Have you...

  • Added relevant entry to the change log?

Comment thread package.json
@schiwekM
schiwekM marked this pull request as ready for review September 1, 2026 09:20
@schiwekM
schiwekM requested review from a team as code owners September 1, 2026 09:20
@schiwekM schiwekM assigned Akatuoro and unassigned SirSimon04 Sep 2, 2026
@schiwekM
schiwekM requested a review from Akatuoro September 3, 2026 07:08

@Akatuoro Akatuoro left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

comments + remove the goallab-prediction-agent

Comment thread package.json Outdated
Comment thread index.js Outdated
Comment thread CHANGELOG.md Outdated
Comment thread .docs/testing/evals.md Outdated
```js
import cds from "@sap/cds"
import { test } from "vitest"
import { Judge, TrajectoryJudge, ConverstationJudge, matchToolCall } from "@cap-js/agents"

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Suggested change
import { Judge, TrajectoryJudge, ConverstationJudge, matchToolCall } from "@cap-js/agents"
import { Judge, ConversationJudge, matchToolCall } from "@cap-js/agents/eval"

Only Judge, no others

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I adjusted the structure and the initial sample just includes the Judge now but the other two are still mentioned as they are still relevant.

Comment thread .docs/testing/evals.md Outdated
})
```

## Eval run lifecycle

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

remove everything from here on, or drastically reduce it. Contains a lot of details which are likely to change, which simply leads to outdated docs

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I adjusted the structure and minimized it but fewer details are I believe not possible as else the docs loose their value.

Comment thread .docs/testing/evals.md Outdated
Comment thread .docs/testing/evals.md Outdated
Comment thread .docs/testing/evals.md Outdated
Comment thread lib/testing/Judge.js
Comment thread srv/handlers/chat.js Outdated

/** Install srv.chat(query, contextId?, opts?) on an @agent service. Called from cds.on("serving"). */
export function registerChat(srv) {
srv.chat = async function chat(query, second, third) {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

reduce options as discussed yesterday. Also move this to lib/eval, the current implementation has a lot of details on the result object that are wanted in an eval context, but do not belong in a general api: metrics, toolCalls, query, additional messages for evaluating the trajectory....

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Sorry for overlooking. I removed the options but I left it in chat.js as the other options are already behind a profile flag just for testing.

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.

3 participants