Add MergeRequestReader to chat
-
Review changes -
-
Download -
Patches
-
Plain diff
What does this MR do and why?
Addresses one popular context the chat currently does not support: https://gitlab.com/gitlab-org/ai-powered/duo-chat/discussions/-/issues/3+
Specifically this MR addresses Support Merge Requests as context for Duo Chat ... (#464587 - closed) • Lesley Razzaghian • 17.5 • Needs attention
Evaluation results
Here are the evaluation results from a collective LLM judge on the master branch
Here are the results on this branch
Here are the stats (averages)
Master branch
Correctness: 3.65
Readability: 3.75
Comprehensiveness: 3.43
This branch
Correctness: 3.71
Readability: 3.75
Comprehensiveness: 3.53
The improved results in the existing evaluation are likely not statistically significant, but at least it proves these changes do not degrade existing questions.
MR acceptance checklist
Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.
Screenshots or screen recordings
Screenshots are required for UI changes, and strongly recommended for all other merge requests.
Before | After |
---|---|
How to set up and validate locally
Numbered steps to set up and validate the change are strongly suggested.
- In rails console enable the experiment fully
Feature.enable(:ai_merge_request_reader_for_chat)
- Visit merge request and ask question, for example: summarize this Merge request
What needs to be done
Task | Status | Notes |
---|---|---|
Run CEF locally on this branch vs master to ensure no degradation | ![]() |
See above in 'Evaluation results' |
Create basic dataset for MR eval | ![]() |
Here |
Change Chat REST API to accept MR requests | ![]() |
|
Add seed data for merge requests to be able to test locally in CEF | ![]() |
I believe this can be a followup |
Make this work with 'v2_chat_agent_integration' feature flag on | ![]() |
This MR just needs to be merged after this one has been rolled out |
Ask model validation team to add this dataset to daily runs, merge this MR and monitor eval results | ![]() |
Asked them to add it discussion |
Merge request reports
- version 273af7133b
- version 265798c802
- version 25f7a4a706
- version 24688e6699
- version 238cd77899
- version 22c61a6136
- version 219f984583
- version 2015c74637
- version 199df2f00a
- version 18ce8abf87
- version 17420b2a63
- version 1674835e3c
- version 15aa90ca3b
- version 1483fe5ae7
- version 139e816f39
- version 1225347bc0
- version 111e3752b7
- version 1050970cd3
- version 9a8174caf
- version 8f08bcf6d
- version 7e474ad71
- version 65e581138
- version 59aaa20d7
- version 4c0f11a92
- version 38037691b
- version 2be7969f9
- version 11ee53a36
- master (base)
- latest versionee4a078c34 commits,
- version 273af7133b32 commits,
- version 265798c80231 commits,
- version 25f7a4a70630 commits,
- version 24688e669929 commits,
- version 238cd7789928 commits,
- version 22c61a613627 commits,
- version 219f98458326 commits,
- version 2015c7463725 commits,
- version 199df2f00a24 commits,
- version 18ce8abf8723 commits,
- version 17420b2a6322 commits,
- version 1674835e3c20 commits,
- version 15aa90ca3b19 commits,
- version 1483fe5ae717 commits,
- version 139e816f3915 commits,
- version 1225347bc014 commits,
- version 111e3752b714 commits,
- version 1050970cd313 commits,
- version 9a8174caf12 commits,
- version 8f08bcf6d10 commits,
- version 7e474ad719 commits,
- version 65e5811388 commits,
- version 59aaa20d77 commits,
- version 4c0f11a926 commits,
- version 38037691b4 commits,
- version 2be7969f93 commits,
- version 11ee53a362 commits,
- Side-by-side
- Inline
There are no changes yet
No changes between version 4 and version 4