-
Notifications
You must be signed in to change notification settings - Fork 82
Add UT for Data and Scorers library #26
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
…ys. it's a BaseModel.
…display_meter and make schema enforcement prompt use the actual type of the object instead of the raw name.
…s_to_ground_truth
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM nice unit tests
… tests because pytest was trying to use it as a Test class, but it can't because TestScorer inherited from a real class (with an __init__).
…/judgeval into alex/add-unit-tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM after fixing minor issues.
…alex/add-unit-tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add Tracer UT fixes from other PR
Add unit tests for the
data/
andscorers/
dirs.Make some small bugfixes after discovering malfunctions during testing.