-
Notifications
You must be signed in to change notification settings - Fork 6
Skips some tests that are currently causing problems for internal users #266
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
Changes from all commits
bcefe60
c5715e3
82def5d
afef5e8
4cf01a7
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,6 +6,7 @@ | |
| from groundlight_openapi_client.exceptions import NotFoundException | ||
|
|
||
|
|
||
| @pytest.mark.skip(reason="This is an expensive test, reset may take some time") | ||
| def test_reset_retry(gl_experimental: ExperimentalApi): | ||
| # Reset the detector, retrying in case the reset is still ongoing | ||
| det = gl_experimental.create_detector(f"Test {datetime.utcnow()}", "test_query") | ||
|
|
@@ -25,6 +26,7 @@ def test_reset_retry(gl_experimental: ExperimentalApi): | |
| raise Exception("Failed to reset detector") | ||
|
|
||
|
|
||
| @pytest.mark.skip(reason="This test does not work with strong 0 shot models, enabled by default based on your account") | ||
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This no longer works for internal users because GLAVA is so confident |
||
| def test_reset_training(gl_experimental: ExperimentalApi): | ||
| # If we reset a detector, we should have low confidence after the reset | ||
| low_confidence_threshold = 0.6 | ||
|
|
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -56,6 +56,12 @@ def test_counting_detector(gl_experimental: ExperimentalApi): | |
| assert count_iq.result.count is not None | ||
|
|
||
|
|
||
| @pytest.mark.skip( | ||
|
Collaborator
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. The SDK got ahead of reef, and what we've enabled for internal users, so this test is actually causing issues during development |
||
| reason=( | ||
| "General users currently currently can't use multiclass detectors. If you have questions, reach out" | ||
| " to Groundlight support, or upgrade your plan." | ||
| ) | ||
| ) | ||
| def test_multiclass_detector(gl_experimental: ExperimentalApi): | ||
| """ | ||
| verify that we can create and submit to a multi-class detector | ||
|
|
||
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.
This is likely the source of much of our current SDK test time