Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

hassiebp
Copy link
Contributor

@hassiebp hassiebp commented Sep 4, 2025

Important

Modify getUserMetrics in traces.ts to include all observation types in metrics calculation.

  • Behavior:
    • Modify SQL query in getUserMetrics in traces.ts to include all observation types, not just 'GENERATION'.
  • Misc:
    • Remove AND o.type = 'GENERATION' condition from the query in getUserMetrics.

This description was created by Ellipsis for 4d0a59a. You can customize this summary. It will automatically update as commits are pushed.

@dosubot dosubot bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Sep 4, 2025
@dosubot dosubot bot added the bug Something isn't working label Sep 4, 2025
@hassiebp hassiebp enabled auto-merge September 4, 2025 10:56
@dosubot dosubot bot added the auto-merge This PR is set to be merged label Sep 4, 2025
@hassiebp hassiebp added this pull request to the merge queue Sep 4, 2025
@Steffen911 Steffen911 removed this pull request from the merge queue due to the queue being cleared Sep 4, 2025
@dosubot dosubot bot removed the auto-merge This PR is set to be merged label Sep 4, 2025
@hassiebp hassiebp merged commit 6bc9968 into main Sep 4, 2025
29 checks passed
@hassiebp hassiebp deleted the fix-user-metrics branch September 4, 2025 12:34
gongxh13 pushed a commit to gongxh13/langfuse that referenced this pull request Sep 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working size:XS This PR changes 0-9 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant