Skip to content

Conversation

@clutchski
Copy link
Contributor

No description provided.

@github-actions
Copy link

github-actions bot commented May 14, 2025

Braintrust eval report

Autoevals (matt/no-btcore-dep-1747235488)

Score Average Improvements Regressions
NumericDiff 74.4% (0pp) 🟢 6 🔴 4
Start 1747235487.86s 🟡 🟡
End 1747235491.74s 🟡 🟡
Duration 3.87s (+2.55s) 🟡 🔴 100
Llm_duration 2.53s 🟡 🟡
Prompt_tokens 279.25tok (+0tok) 🟡 🟡
Completion_tokens 19.14tok (-0.75tok) 🟢 32 🔴 14
Total_tokens 298.38tok (-0.75tok) 🟢 32 🔴 14
Estimated_cost 0$ 🟡 🟡

Comment on lines +36 to +39
print(
"The error field is deprecated, as errors are now propagated to the caller. The field will be removed in a future version of the library",
sys.stderr,
)
Copy link

Choose a reason for hiding this comment

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

The print statement is using sys.stderr as a positional argument instead of as the file parameter. This will cause the message to print incorrectly. The statement should be modified to:

print(
    "The error field is deprecated, as errors are now propagated to the caller. The field will be removed in a future version of the library",
    file=sys.stderr
)
Suggested change
print(
"The error field is deprecated, as errors are now propagated to the caller. The field will be removed in a future version of the library",
sys.stderr,
)
print(
"The error field is deprecated, as errors are now propagated to the caller. The field will be removed in a future version of the library",
file=sys.stderr,
)

Spotted by Diamond

Is this helpful? React 👍 or 👎 to let us know.

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.

2 participants