Skip to content

Feat: Add locate all function#118

Merged
mlikasam-askui merged 4 commits intomainfrom
feat-add-locate-all
Aug 15, 2025
Merged

Feat: Add locate all function#118
mlikasam-askui merged 4 commits intomainfrom
feat-add-locate-all

Conversation

@mlikasam-askui
Copy link
Contributor

@mlikasam-askui mlikasam-askui commented Aug 14, 2025

Breaking changes:

  • The locate function in locator models must now return a list of points instead of a single point.

@programminx-askui
Copy link
Collaborator

Hello @adi-wan-askui ,

can you take the PR over? @mlikasam-askui is next week in off.

Copy link
Contributor

@adi-wan-askui adi-wan-askui left a comment

Choose a reason for hiding this comment

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

Well done :)

Have you checked that all the e2e tests for locate() run through?

@mlikasam-askui mlikasam-askui merged commit 8a065a1 into main Aug 15, 2025
1 check passed
@mlikasam-askui mlikasam-askui deleted the feat-add-locate-all branch August 15, 2025 13:35
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.

3 participants