NotificationsYou must be signed in to change notification settings
Fork964
Star14.1k

add test for input_boxes#1141

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

Open

sabman wants to merge1 commit intohuggingface:main

base:main

Choose a base branch

fromdecision-labs:test-sam-with-input_boxes

Open

add test for input_boxes#1141

sabman wants to merge1 commit intohuggingface:mainfromdecision-labs:test-sam-with-input_boxes

Conversation

Copy link

sabman commentedJan 12, 2025

The model should in theory supportinput_boxes as per this issue:https://huggingface.co/Xenova/slimsam-77-uniform/discussions/2#65c022977c6ad1994b893a8d but there's not test or example to verify this. I also couldn't get the model to accept theinput_boxes on the main branch. Getting error:

Error: An error occurred during model execution: "Missing the following inputs: input_points, input_labels.

add test for input_boxes

fb65ca4

The model should in theory support input_boxes as per this issue:https://huggingface.co/Xenova/slimsam-77-uniform/discussions/2#65c022977c6ad1994b893a8d but there's not test or example to verify this.