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

Update detector client for use of /api/v1/text/contents #37

Closed
1 of 2 tasks
alanbraz opened this issue May 16, 2024 · 1 comment · Fixed by #62
Closed
1 of 2 tasks

Update detector client for use of /api/v1/text/contents #37

alanbraz opened this issue May 16, 2024 · 1 comment · Fixed by #62
Assignees

Comments

@alanbraz
Copy link
Collaborator

alanbraz commented May 16, 2024

Description

As a developer, I want to call any detector's API, so that I can use it correctly in my workflow.

Detector API is being revisited and we are adding more endpoints to nicely group together detectors in logical bins. This will result in changes in orchestrator.

Discussion

API updated and will need to refactor orchestrator code to accommodate them.

Acceptance Criteria

Note: this story only covers handing of classification type APIs. For other endpoints that we add to detector API, we will need to add respective APIs in orchestrator as well, which will be covered in different story.

@evaline-ju
Copy link
Collaborator

Removing blocked label - the switch to use /api/v1/text/contents instead of the current /api/v1/detector is unblocked for the existing orchestrator endpoints (classification / streaming-classification)

@evaline-ju evaline-ju changed the title Update Detectors API Update detector client for use of /api/v1/text/contents May 31, 2024
@evaline-ju evaline-ju self-assigned this Jun 4, 2024
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.

3 participants