Skip to content

Add edge_pipeline_config parameter to detector creation#415

Merged
timmarkhuff merged 2 commits intomainfrom
tim/add-edge-pipeline-config
Mar 17, 2026
Merged

Add edge_pipeline_config parameter to detector creation#415
timmarkhuff merged 2 commits intomainfrom
tim/add-edge-pipeline-config

Conversation

@timmarkhuff
Copy link
Contributor

@timmarkhuff timmarkhuff commented Mar 14, 2026

Summary

Adds support for the edge_pipeline_config parameter when creating detectors, mirroring the existing pipeline_config parameter. This allows callers to specify which pipeline config to use for edge inference instead of always using the mode's default.

Corresponds to the backend change in zuuul, PR #6056.

Changes

  • Updated spec/public-api.yaml with the new edge_pipeline_config field on DetectorCreationInputRequest
  • Ran make generate to regenerate the OpenAPI client models
  • Added edge_pipeline_config parameter to all detector creation methods in client.py and experimental_api.py:
    • _prep_create_detector
    • create_detector
    • get_or_create_detector
    • create_binary_detector
    • create_counting_detector
    • create_multiclass_detector
    • create_bounding_box_detector
    • create_text_recognition_detector (ExperimentalApi)

Testing

Tested end-to-end against an eksdev environment running the zuuul branch from #6056:

from groundlight import Groundlight

gl = Groundlight()

det = gl.create_detector(
    name="edge-config-sdk-test",
    query="Is this image relevant?",
    pipeline_config="never-review",
    edge_pipeline_config="generic-cached-timm-efficientnetv2s-knn",
)
print(f"Created detector: {det.id}")
# Created detector: det_3AuicidzisgV5ja7fFhenB8RSIL

@timmarkhuff timmarkhuff changed the title Add edge_pipeline_config parameter Add edge_pipeline_config parameter to detector creation Mar 14, 2026
Copy link
Collaborator

@brandon-wada brandon-wada left a comment

Choose a reason for hiding this comment

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

LGTM

@timmarkhuff timmarkhuff merged commit 4e9f371 into main Mar 17, 2026
15 of 16 checks passed
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