[OPIK-2240] [BE] Add new demo data to Opik #3132
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
thiagohora
changed the title
📋 PR Linter Failed❌ Invalid Title Format. Your PR title must include a ticket/issue number and at least one component tag (
Example: |
thiagohora
force-pushed
the
thiagohora/OPIK-2240_add_new_demo_data
branch
3 times, most recently
from
10753ca to
a0b78b3
Compare
📋 PR Linter Failed❌ Missing Section. The description is missing the |
1 similar comment
📋 PR Linter Failed❌ Missing Section. The description is missing the |
thiagohora
force-pushed
the
thiagohora/OPIK-2240_add_new_demo_data
branch
from
a0b78b3 to
5aea53d
Compare
thiagohora
force-pushed
the
thiagohora/OPIK-2240_add_new_demo_data_constant
branch
from
186440e to
23d661a
Compare
…hiagohora/OPIK-2240_add_new_demo_data
thiagohora
force-pushed
the
thiagohora/OPIK-2240_add_new_demo_data_constant
branch
2 times, most recently
from
3714567 to
5908579
Compare
BorisTkachenko
previously approved these changes
Aug 29, 2025
| @@ -72,6 +73,44 @@ def test_create_demo_data_structure(httpserver): | |||
| httpserver.expect_request("/v1/private/prompts/versions/retrieve", method="POST").respond_with_json(prompt) | |||
| httpserver.expect_request("/v1/private/prompts/versions", method="POST").respond_with_json(prompt) | |||
|
|
|||
| # Mock optimization endpoints that don't exist in the backend yet | |||
There was a problem hiding this comment.
Why they don't exist in BE?
There was a problem hiding this comment.
Because we first called to check if it existed, we then created it.
Base automatically changed from
thiagohora/OPIK-2240_add_new_demo_data_constant
to
main
August 29, 2025 12:15
thiagohora
dismissed
BorisTkachenko’s stale review
The base branch was changed.
There was a problem hiding this comment.
Pull Request Overview
This PR updates the demo data generation in Opik to include more comprehensive examples, including new optimization and thread feedback functionality. The changes enhance the demo experience for users with additional project types and more realistic data patterns.
Key changes:
- Adds new "Opik Demo Optimizer" project with optimization functionality
- Enhances thread feedback scoring with concurrent processing
- Improves time handling to preserve temporal relationships between traces and spans
Reviewed Changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
apps/opik-python-backend/src/opik_backend/demo_data_generator.py |
Major refactoring to add optimization project creation, improve time handling, and implement concurrent thread processing |
apps/opik-python-backend/tests/test_demo_data_generator.py |
Adds test mocking for new optimization and thread endpoints |
apps/opik-python-backend/tests/test_evaluator_python.py |
Removes unnecessary empty line |
apps/opik-python-backend/requirements.txt |
Updates opik version from 1.8.4 to 1.8.33 |
apps/opik-python-backend/src/opik_backend/demo_data_generator.py
Outdated
Show resolved
Hide resolved
BorisTkachenko
previously approved these changes
Sep 1, 2025
Fix the error message Co-authored-by: Copilot <[email protected]>
BorisTkachenko
approved these changes
Sep 2, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Details
Changing the Opik Demo data.
When adding new demo data to the project, this data will be created for every sign-up for OS and Clouse users.
Change checklist
Issues
Testing
Documentation
NA