Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature] Expose this extension through an API #15

Closed
jordan-barrett-jm opened this issue Apr 11, 2023 · 4 comments · Fixed by #69
Closed

[Feature] Expose this extension through an API #15

jordan-barrett-jm opened this issue Apr 11, 2023 · 4 comments · Fixed by #69

Comments

@jordan-barrett-jm
Copy link
Contributor

I'd like to write a feature to expose this model through the segment webui API. This would be pretty straightforward to accomplish but ideally, the endpoint would not accept dots as the prompt, but text. When do you think the integration with Grounded DINO would be completed? It's something I've worked on before so if you haven't started yet I could probably knock it out today.

@jordan-barrett-jm
Copy link
Contributor Author

Also would be ideal for #12 to be merged so I can avoid conflicts

@continue-revolution
Copy link
Owner

continue-revolution commented Apr 11, 2023

I'd like to write a feature to expose this model through the segment webui API. This would be pretty straightforward to accomplish but ideally, the endpoint would not accept dots as the prompt, but text. When do you think the integration with Grounded DINO would be completed? It's something I've worked on before so if you haven't started yet I could probably knock it out today.

GroundingDINO integration almost completed, near the end of debug. I will soon review your code and merge it. Do you want to write API with GroundingDINO enabled? If yes, please stay tuned. I'll let you know when I finish integration.

@jordan-barrett-jm
Copy link
Contributor Author

Yep, would prefer Grounding DINO be enabled before starting on it. Will sit tight then

@continue-revolution
Copy link
Owner

GroundingDINO support done via merging #21

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 a pull request may close this issue.

2 participants