[OPIK-4972] [BE] feat: update demo data to only create Opik Demo Agent Observability project#5861
Conversation
…t Observability project - Remove demo evaluation, optimizer, and playground project creation - Clean up unused imports and DemoDataContext fields in demo_data_generator.py - Update DEMO_PROJECT_NAME constant to match the single demo project name Implements OPIK-4972: [M3] Frontend: Update demo data with project-scoped, environment-tagged traces
…from demo_data.py
|
🔄 Test environment deployment process has started Phase 1: Deploying base version You can monitor the progress here. |
|
✅ Test environment is now available! To configure additional Environment variables for your environment, run [Deploy Opik AdHoc Environment workflow] (https://github.com/comet-ml/comet-deployment/actions/workflows/deploy_opik_adhoc_env.yaml) Access Information
The deployment has completed successfully and the version has been verified. |
|
🔄 Test environment deployment process has started Phase 1: Deploying base version You can monitor the progress here. |
|
✅ Test environment is now available! To configure additional Environment variables for your environment, run [Deploy Opik AdHoc Environment workflow] (https://github.com/comet-ml/comet-deployment/actions/workflows/deploy_opik_adhoc_env.yaml) Access Information
The deployment has completed successfully and the version has been verified. |
aadereiko
left a comment
There was a problem hiding this comment.
The FE part looks good to go. Just make sure that Demo Project isn't queried from anywhere
|
🌙 Nightly cleanup: The test environment for this PR ( |
andrescrz
left a comment
There was a problem hiding this comment.
Please double check the open comment.
|
🌙 Nightly cleanup: The test environment for this PR ( |
1 similar comment
|
🌙 Nightly cleanup: The test environment for this PR ( |
|
🌙 Nightly cleanup: The test environment for this PR ( |
Python Backend Tests Results165 tests 162 ✅ 3m 10s ⏱️ Results for commit f681bf7. ♻️ This comment has been updated with latest results. |
|
🌙 Nightly cleanup: The test environment for this PR ( |
- Bump opik to 1.10.56 in opik-python-backend/requirements.txt - Upgrade all dependencies in opik-sandbox-executor-python/requirements.txt to exact versions resolved from opik==1.10.56 dependency graph (notable: litellm 1.83.0, openai 2.30.0, python-dotenv 1.2.2) - Add ENV LITELLM_MODE=PRODUCTION to sandbox Dockerfile to prevent litellm from calling load_dotenv() at import time; litellm defaults LITELLM_MODE to "DEV" and calls load_dotenv() in that mode, whose find_dotenv() uses frame inspection (sys._getframe) that fails in a .pyc-only context
…andbox Update pinned versions to match those resolved from opik==1.10.56: - certifi: 2024.12.14 -> 2026.2.25 - charset-normalizer: 3.4.1 -> 3.4.6 - click: 8.1.8 -> 8.3.1 - idna: 3.10 -> 3.11 - MarkupSafe: 3.0.2 -> 3.0.3 - packaging: 24.2 -> 26.0 - requests: 2.32.3 -> 2.33.1 - uuid6: 2024.7.10 -> 2025.0.1
…esult schema opik 1.10.56 added a category_name field to ScoreResult. Update the expected dicts in test_success parametrize to include category_name: None.
…None opik 1.10.56 added category_name to ScoreResult but the Docker image in CI may use an older opik version without it. Assert category_name is None when present and compare remaining fields, so both executor variants pass.
|
🌙 Nightly cleanup: The test environment for this PR ( |
6 similar comments
|
🌙 Nightly cleanup: The test environment for this PR ( |
|
🌙 Nightly cleanup: The test environment for this PR ( |
|
🌙 Nightly cleanup: The test environment for this PR ( |
|
🌙 Nightly cleanup: The test environment for this PR ( |
|
🌙 Nightly cleanup: The test environment for this PR ( |
|
🌙 Nightly cleanup: The test environment for this PR ( |
andrescrz
left a comment
There was a problem hiding this comment.
LGTM, I recommend decoupling the PRs for the demo data and the upgrades and fixes on the images.
There was a problem hiding this comment.
I recommend sending these changes below (dependencies, image changes) in a separated PR.
Not a blocker.
|
🌙 Nightly cleanup: The test environment for this PR ( |
Details
Simplifies demo data creation to a single project. Previously, four demo projects were created on signup: Opik Demo Agent Observability, Opik Demo Assistant, Opik Demo Optimizer, and playground. This change removes the three extra projects and keeps only "Opik Demo Agent Observability" as the single demo project.
create_demo_evaluation_project,create_demo_optimizer_project, andcreate_demo_playground_projectfrom the Python backend demo data generatorDemoDataContextdataclass fieldsDEMO_PROJECT_NAMEfrontend constant to match the single demo project nameopikSDK to1.10.56(aligns with opik 2.0 SDK) in bothopik-python-backendandopik-sandbox-executor-python, updating all pinned dependencies to their resolved versionsENV LITELLM_MODE=PRODUCTIONto the sandbox Dockerfile to prevent litellm from callingload_dotenv()at import time in a.pyc-only contextDemoData.java(Java backend BI exclusion list) is intentionally kept unchanged for backward compatibility with existing workspaces that already have the old demo projects.Change checklist
Issues
AI-WATERMARK
AI-WATERMARK: yes
Testing
pytest tests/— 69/69 Python backend tests passednpm run test— 1254/1254 frontend tests passed (2 pre-existing failures inlangchain/mapper.test.tsandopenai/mapper.test.tsare unrelated: missingreact-h5-audio-playerdependency)npm run lint— passed cleanlyDEMO_PROJECT_NAMEconstant update is reflected inuseDemoProjecthookDocumentation
N/A