Skip to content
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

Adds PipeEvaluator #8

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Adds PipeEvaluator #8

wants to merge 1 commit into from

Conversation

simedw
Copy link
Contributor

@simedw simedw commented Aug 11, 2023

Let's you chain multiple evaluators together, main reason is to first try cheap and faster evaluator, such as string-match, and only if they fail go for more expensive ones like semantic similarity.

@simedw simedw requested a review from andreaazzini August 11, 2023 17:07
Comment on lines +5 to +6
def noop():
pass
Copy link
Member

Choose a reason for hiding this comment

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

Is this used anywhere else? If this is to prevent auto-ordering these lines, we have used the # Adding an empty comment to force import order to avoid circular imports comment above, we could potentially use that again?

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