-
-
Notifications
You must be signed in to change notification settings - Fork 379
Fixed metamorphic tests for LLM #1185
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
Fixed metamorphic tests for LLM #1185
Conversation
GSK-1311 Error when running auto-generated test suite for LLM model
https://demo.giskard.ai/main/projects/1451/testing/1511/overview The model is created using |
mattbit
left a comment
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.
Good for me!
| from giskard.ml_worker.utils.logging import timer | ||
| from giskard.models.base import BaseModel | ||
| from giskard.models.utils import fix_seed | ||
| from giskard.scanner.llm.utils import LLMImportError |
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.
Maybe we should move LLMImportError outside of scanner now that we are integrating it more with the rest of the codebase. But not super important.
…erated-test-suite-for-llm-model
|
@kevinmessiaen would be nice to add a simple test if you have time, using |
| except ImportError as err: | ||
| raise LLMImportError() from err | ||
|
|
||
| scorer = evaluate.load("bertscore") |
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.
It's better to add evaluate.load("bertscore") to the try-except block since it relies on another extra dependency that we add. In case a person already has evaluate (previously installed for example) but not bert-score he'd also get the same LLMImportError with explanations
…erated-test-suite-for-llm-model
…erated-test-suite-for-llm-model
…erated-test-suite-for-llm-model
…erated-test-suite-for-llm-model
|
Kudos, SonarCloud Quality Gate passed! |








Description
Fixed metamorphic tests for LLM
Type of Change