docs: add docs for anonymizer replace evaluation#181
Conversation
Signed-off-by: memadi <memadi@nvidia.com>
Greptile SummaryThis PR adds a dedicated
Confidence Score: 5/5Documentation-only change with no executable code paths affected; safe to merge. All three changed files are Markdown/YAML docs. The model-override YAML snippet in evaluation.md was verified against the actual _merge_selections() implementation and is correct. The four judge role names match evaluate.yaml exactly. Previously flagged issues (trailing newline, empty section header, grammar nit) are resolved in the current file state. No files require special attention. Important Files Changed
|
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Signed-off-by: memadi <memadi@nvidia.com>
Co-authored-by: lipikaramaswamy <31832945+lipikaramaswamy@users.noreply.github.com>
Summary
Why a separate page instead of adding to
replace.md?Evaluation is conceptually independent from any single anonymization strategy. Replace and Rewrite both have evaluation — each with different mechanics (post-hoc vs. built-in), different metrics, and different model roles. Keeping evaluation in its own page:
What's documented in
evaluation.md?This PR is related to the changes made in PR #158
Type of Change
Testing
make testpasses locallymake checkpasses locally (format + lint + typecheck + lock-check)Documentation
make docs-buildpasses locallyRelated Issues
Closes #98