Skip to content

Expose priming group#411

Merged
brandon-wada merged 5 commits intomainfrom
brandon/detector_priming
Mar 10, 2026
Merged

Expose priming group#411
brandon-wada merged 5 commits intomainfrom
brandon/detector_priming

Conversation

@brandon-wada
Copy link
Collaborator

Users can now create detectors belonging to a specific detector priming group with a known priming group id. Detectors in these groups will be able to start with a Groundlight defined pretrained model that is well suited for the task the priming group is built around

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

It's been a little while since we ran make generate, slight changes to the npm metadata file

gl.delete_detector(fake_detector_id) # type: ignore


def test_create_detector_with_invalid_priming_group_id(gl: Groundlight):
Copy link
Member

Choose a reason for hiding this comment

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

no happy path test is strange.

the detector (limited to 1KB). This metadata can be used to store additional
information like location, purpose, or related system IDs. You can retrieve this
metadata later by calling `get_detector()`.
:param priming_group_id: Optional ID of an existing PrimingGroup to associate with this detector.
Copy link
Member

Choose a reason for hiding this comment

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

We should callout that a 3rd party won't know what valid ones are and not to use it.

metadata later by calling `get_detector()`.
:param class_names: The name or names of the class to use for the detector. Only used for multi-class
and counting detectors.
:param priming_group_id: Optional ID of an existing PrimingGroup to associate with this detector.
Copy link
Member

Choose a reason for hiding this comment

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

We should callout that a 3rd party won't know what valid ones are and not to use it.

the detector (limited to 1KB). This metadata can be used to store additional
information like location, purpose, or related system IDs. You can retrieve this
metadata later by calling `get_detector()`.
:param priming_group_id: Optional ID of an existing PrimingGroup to associate with this detector.
Copy link
Member

Choose a reason for hiding this comment

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

We should callout that a 3rd party won't know what valid ones are and not to use it.

the detector (limited to 1KB). This metadata can be used to store additional
information like location, purpose, or related system IDs. You can retrieve this
metadata later by calling `get_detector()`.
:param priming_group_id: Optional ID of an existing PrimingGroup to associate with this detector.
Copy link
Member

Choose a reason for hiding this comment

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

We should callout that a 3rd party won't know what valid ones are and not to use it.

Copy link
Member

@sunildkumar sunildkumar left a comment

Choose a reason for hiding this comment

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

approved without conviction, as discussed on slack.

@brandon-wada brandon-wada merged commit 0b22416 into main Mar 10, 2026
8 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