Skip to content

Report generation: Adding user feedback support in the form of thumbs up and thumbs down button in the UI#54

Merged
lotif merged 26 commits into
mainfrom
marcelo/thumbs
Feb 17, 2026
Merged

Report generation: Adding user feedback support in the form of thumbs up and thumbs down button in the UI#54
lotif merged 26 commits into
mainfrom
marcelo/thumbs

Conversation

@lotif
Copy link
Copy Markdown
Collaborator

@lotif lotif commented Feb 17, 2026

Summary

Adding thumbs up/thumbs down button to the gradio demo UI to demonstrate how to log user feedback to langfuse.

Clickup Ticket(s): NA

Type of Change

  • 🐛 Bug fix (non-breaking change that fixes an issue)
  • ✨ New feature (non-breaking change that adds functionality)
  • 💥 Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • 📝 Documentation update
  • 🔧 Refactoring (no functional changes)
  • ⚡ Performance improvement
  • 🧪 Test improvements
  • 🔒 Security fix

Changes Made

  • Using gradio state to keep langfuse's trace id at the end of the agent's execution
  • Adding a new row in the UI to capture user feedback in the form of thumbs up and thumbs down buttons that get displayed at the end of the agent's execution
  • Adding events to record the user feedback into langfuse

Testing

  • Tests pass locally (uv run pytest tests/)
  • Type checking passes (uv run mypy <src_dir>)
  • Linting passes (uv run ruff check src_dir/)
  • Manual testing performed (describe below)

Manual testing details:

Clicked the buttons and checked they work in langfuse

Checklist

  • Code follows the project's style guidelines
  • Self-review of code completed
  • Documentation updated (if applicable)
  • No sensitive information (API keys, credentials) exposed

@lotif lotif changed the title Marcelo/thumbs Report generation: Adding user feedback support in the form of thumbs up and thumbs down button in the UI Feb 17, 2026
@lotif lotif requested review from amrit110 and fcogidi February 17, 2026 16:09
Comment thread implementations/report_generation/demo.py Outdated
@lotif lotif changed the base branch from marcelo/online-eval to main February 17, 2026 16:45
@lotif lotif changed the base branch from main to marcelo/online-eval February 17, 2026 16:51
Base automatically changed from marcelo/online-eval to main February 17, 2026 18:56
@lotif lotif merged commit 03e4086 into main Feb 17, 2026
3 checks passed
@lotif lotif deleted the marcelo/thumbs branch February 17, 2026 19:05
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