Skip to content

246 update evaluation docs#247

Open
dkkdark wants to merge 3 commits into
mainfrom
246-update-documentation
Open

246 update evaluation docs#247
dkkdark wants to merge 3 commits into
mainfrom
246-update-documentation

Conversation

@dkkdark

@dkkdark dkkdark commented Apr 7, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@dkkdark dkkdark requested a review from NoB0 April 7, 2026 10:31
@github-actions

github-actions Bot commented Apr 7, 2026

Copy link
Copy Markdown
Current Branch Main Branch
Coverage Badge Coverage Badge

@NoB0 NoB0 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Some comments to look at and update to do based on last changes on #243

Comment thread docs/source/evaluation.rst Outdated
Comment thread docs/source/evaluation.rst
Comment thread docs/source/evaluation.rst Outdated
Comment thread docs/source/evaluation.rst
Comment thread docs/source/evaluation.rst Outdated
@dkkdark dkkdark requested a review from NoB0 April 21, 2026 14:35
Comment thread docs/source/evaluation.rst Outdated
Comment thread docs/source/evaluation.rst Outdated
Comment thread docs/source/evaluation.rst Outdated
Comment thread docs/source/evaluation.rst
@dkkdark dkkdark requested a review from NoB0 June 16, 2026 07:43

@NoB0 NoB0 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM apart from a nit. To be merged after PR #243.

UserSimCRS evaluates conversational recommender systems (CRSs) on previously generated synthetic dialogues. The evaluation pipeline loads dialogues from a JSON file, computes one or more metrics, and stores the results as JSON together with the copy of the configuration used.

Currently, we provide the following evaluation scripts:
A default evaluation configuration is provided in `config/default/config_evaluation.yaml`.

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Nit: Move this to the end of the Configuration section.

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