[EAGLE-7291] Use PATs for API calls that involve external models, part 2#251
Merged
tiit-clarifai merged 1 commit intomasterfrom Apr 10, 2026
Merged
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Python gRPC client test suite to authenticate with Personal Access Tokens (PATs) for requests that implicitly invoke external (cross-application) embedder models via image inputs, ensuring tests can access models from clarifai/main.
Changes:
- Switched image-embedding-based input search tests to use
metadata(pat=True)and addeduser_app_id=user_app_id()to the affected requests. - Updated a model predict call (with image input) in the model CRUD test to use PAT auth and explicit
user_app_id.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| tests/client/test_model_crud.py | Uses PAT auth + user_app_id for image-based PostModelOutputs that triggers an external embedder. |
| tests/client/test_inputs_searches.py | Uses PAT auth + user_app_id for image-ranking PostInputsSearches queries that require external embeddings. |
| tests/client/test_input_search_legacy.py | Uses PAT auth + user_app_id for legacy image-based PostSearches embedding searches. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
nitinbhojwani
approved these changes
Apr 10, 2026
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Search and custom predict operations with an image input use the embedder model from the base workflow of the application. The test applications use the general-image-embedding model from clarifai/main, so they need to use a Personal Access Token to access the model from another application.