Skip to content

Conversation

@langfuse-bot
Copy link
Collaborator

@langfuse-bot langfuse-bot commented Nov 17, 2025

Important

Remove score-related sections from API documentation and client code in reference.md and client.py.

  • API Documentation:
    • Removed Scores section from reference.md, including scores_avg and score_categories.
  • Client Code:
    • Removed Scores section from docstring in get_many() in client.py.
    • Removed Scores section from docstring in AsyncObservationsClient.get_many() in client.py.

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

Disclaimer: Experimental PR review

Greptile Summary

  • Removed scores_avg and score_categories filter options from observations API documentation
  • Auto-generated update syncing with upstream API specification changes from langfuse/langfuse commit 34d42bc

Confidence Score: 5/5

  • This PR is safe to merge with no risk
  • Auto-generated documentation update that removes obsolete filtering options from observations API; no code logic changes, only docstring updates in sync with upstream API spec
  • No files require special attention

Sequence Diagram

sequenceDiagram
    participant Client
    participant ObservationsAPI
    participant Database
    Client->>ObservationsAPI: "GET /api/public/observations"
    ObservationsAPI->>Database: "Query observations with filters"
    Database-->>ObservationsAPI: "Return observations"
    ObservationsAPI-->>Client: "Return ObservationsViews"
Loading

@hassiebp hassiebp merged commit 78cdbb1 into main Nov 17, 2025
11 checks passed
@hassiebp hassiebp deleted the api-spec-bot-34d42bc branch November 17, 2025 12:25
Copy link
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

2 files reviewed, no comments

Edit Code Review Agent Settings | Greptile
React with 👍 or 👎 to share your feedback on this new summary format

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.

3 participants