Added an Add as Demonstration button to the inference TryWithVariant modal
should use the same pattern as the existing HumanFeedbackForm so wasn't too complicated
[!IMPORTANT] Added a button to submit demonstration feedback in the inference modal and updated UI tests to handle credentials.
- UI Components:
- Added
DemonstrationFeedbackButtoninDemonstrationFeedbackButton.tsxfor submitting demonstration feedback.- Updated
VariantResponseModalinVariantResponseModal.tsxto includeDemonstrationFeedbackButton.- Routes:
- Modified
route.tsxto handle demonstration feedback submission and navigation.- Added
prepareDemonstrationFromVariantOutput()to process variant output for feedback.- Testing:
- Updated
.github/workflows/ui-tests-e2e.ymlto run UI E2E tests requiring credentials.- Added e2e test in
observability.inferences.$inference_id.spec.tsfor adding demonstration feedback via the "Try with variant" flow.This description was created by
for 6bf8da8cff16da69326055c005affe8e36269e8a. You can customize this summary. It will automatically update as commits are pushed.
/regen-fixtures
/regen-fixtures
All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.
/regen-fixtures
recheck
/regen-fixtures
/regen-fixtures
/regen-fixtures
Moved button: