-
Notifications
You must be signed in to change notification settings - Fork 855
chore(alibi-detect-server): [INFRA-1646] Upgrade alibi-detect-server #6935
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
Merged
tyndria
merged 37 commits into
SeldonIO:release-1.19.0-prep
from
tyndria:INFRA-1613-alibi-detect-tonya
Dec 3, 2025
Merged
Changes from 32 commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
4556496
Fix `get_local_repo` to make it work on macOS
tyndria 5915eba
Update python and lock
tyndria aa27d7e
Update numpy to "1.26.4"
tyndria 1afdd3d
remove `extras=["all"]` from alibi-detect dependency definition and p…
tyndria 47eb56f
Fix build seldonio/conda-ubi9 image
tyndria 284e2ad
Upgrade pip-licenses = "5.5.0"
tyndria 6082bba
Fix building `alibi-detect-server` image
tyndria 49d602b
Fix invoking `make test`
tyndria 135e211
Upgrade tensorflow to fix some tests
tyndria 19ea84e
skip if rclone is unavailable
tyndria df5d8e3
Updated poetry.lock
tyndria 44e2c60
Update GH action for alibi detect tests
tyndria 3eb84f0
Minor packages upgrades
tyndria bab6063
Fix minor image warnings
tyndria 7338a90
Remove fix for the solved `pystan` issue
tyndria 29e45c7
Added missing "tensorflow-probability" and "tf-keras" dependencies to…
tyndria 614c5af
Upgrade rclone to v1.71.2 to fix CVEs
tyndria 2cd77bc
Fix poetry installation after conda base image is changed; Introduced…
tyndria 54e7ca6
Fix alibidetect_tests workflow
tyndria 963cd66
Try to run workflow in a fork (expect to be failing)
tyndria 9ed6584
Upgrade `black` to fix lint job
tyndria 3757875
upgrade protobuf, tf, tf-keras, grpcio-tools
tyndria c5e3d98
Upgrade protobuf patch version to move from yanked version
tyndria c066833
Use ubi9 base image for the alibi-detect-server image
tyndria db60069
Return missing option -p for mkdir command in Dockerfile
tyndria 63c6d92
Return type check stage before tests execution and fix one error
tyndria 40f5fb9
Testing with alibi-detect rc
tyndria f95cd8d
Upgraded mypy to fix type_check action
tyndria 6e16926
Pass TF_USE_LEGACY_KERAS to fix alibi-detect e2e tests on top of the …
tyndria 5e02623
Update readme and mark GPU image as deprecated (only repo
tyndria 1358dff
Fixed helper commands to run quick sanity checks and updated readme
tyndria 01f3fd6
Try to clean up disk space to fix ci for alibi-detect-server
tyndria 20edd64
Clean up
tyndria e979306
Deprecate components/drift-detection/nvidia-triton-cifar10/cifar10_dr…
tyndria 4886cf9
Added a note about rclone setup to readme
tyndria 0b26e09
Update lint command to use `poetry`; Committing formatted files
tyndria 25c3433
Revert a code to build conda base image on mac
tyndria File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,3 +1,41 @@ | ||
| # Alibi-Detect CloudEvents Server | ||
|
|
||
| Extends the [Seldon CloudEvents Server](https://github.com/SeldonIO/seldon-models/tree/master/servers/cloudevents) to allow Alibi Detect models to be loaded and events processed. | ||
|
|
||
| ## Maintanence | ||
|
|
||
| In order to build and test integration with the Alibi Detect library, ensure to execute the following commands after pulling changes: | ||
|
|
||
| ```bash | ||
|
|
||
| make dev_install | ||
| make test | ||
|
|
||
|
|
||
| ``` | ||
|
|
||
| This will install the Alibi Detect library in the development environment and run the tests. | ||
|
|
||
| There is also a possibility to quickly run sanity checks in this way: | ||
|
|
||
| ```bash | ||
|
|
||
| make run-outlier-detector-tensorflow # to run the outlier detector server with tensorflow locally | ||
| make curl-detector-tensorflow # to send a test request to the server | ||
| make curl-tensorflow-outlier-detector-scores # to send a test request to get outlier scores | ||
|
|
||
| make run-outlier-detector-v2 # to run the outlier detector server with kfserving protocol locally | ||
| make curl-detector-v2 # to send a test request to the server | ||
| make curl-detector-v2-outlier # to send a test request to get outlier scores | ||
|
|
||
| make run-metrics-server # to run the metrics server locally | ||
| make curl-metrics-server-feedback # to submit a test feedback with truth | ||
| make curl-metrics-server-get # to get metrics from the server | ||
| ``` | ||
|
|
||
| In order to run the integration tests in the Kubernetes cluster, build the Docker image first: `make docker-build`. | ||
| Then, as part of the cluster and ecosystem setup in the `testing/scripts` folder, [this test](../../testing/scripts/test_alibi_detect_server.py) can be executed to validate the Alibi Detect server functionality. | ||
|
|
||
| ## Deprecation Notice for GPU image (starting from Core 1.19.0 release) | ||
|
|
||
| GPU image for Alibi Detect server is deprecated and won't be maintained starting from Seldon Core 1.19.0 release. |
File renamed without changes.
File renamed without changes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Are those make commands running fine after the addition of
TF_USE_LEGACY_KERAS=1🤔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.
They are
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.
confirmed