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

Skip to content

Conversation

@anticorrelator
Copy link
Contributor

@anticorrelator anticorrelator commented Sep 24, 2025

  • Adds an unbind method to Evaluators to remove any associated input mapping bind
  • the bind_evaluator function now returns a copy of the original evaluator to prevent undesired mutation

Note

Adds Evaluator.unbind() and updates bind_evaluator to return a deep-copied, bound evaluator instead of mutating the original.

  • Evaluators (packages/phoenix-evals/src/phoenix/evals/evaluators.py):
    • Add Evaluator.unbind() to clear any bound input_mapping.
    • Update bind_evaluator to deepcopy the provided evaluator, bind the copy, and return it (prevents mutating the original).
    • Import copy to support deep copying.

Written by Cursor Bugbot for commit 9c8f23e. This will update automatically on new commits. Configure here.

@anticorrelator anticorrelator requested a review from a team as a code owner September 24, 2025 15:37
@github-project-automation github-project-automation bot moved this to πŸ“˜ Todo in phoenix Sep 24, 2025
@dosubot dosubot bot added the size:S This PR changes 10-29 lines, ignoring generated files. label Sep 24, 2025
@github-project-automation github-project-automation bot moved this from πŸ“˜ Todo to πŸ‘ Approved in phoenix Sep 24, 2025
@anticorrelator anticorrelator force-pushed the dustin/evaluator-binding-improvements branch from 39040a8 to 9c8f23e Compare October 2, 2025 16:51
@anticorrelator anticorrelator merged commit 2989a91 into main Oct 2, 2025
38 checks passed
@anticorrelator anticorrelator deleted the dustin/evaluator-binding-improvements branch October 2, 2025 17:05
@github-project-automation github-project-automation bot moved this from πŸ‘ Approved to βœ… Done in phoenix Oct 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:S This PR changes 10-29 lines, ignoring generated files.

Projects

Status: βœ… Done

Development

Successfully merging this pull request may close these issues.

4 participants