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

Refactor based on discussion about status endpoints and how they shou… #1

Merged
merged 1 commit into from
Nov 25, 2024

Conversation

Layoric
Copy link
Member

@Layoric Layoric commented Nov 25, 2024

…ld be more consistently typed.

  • Split GenerationResponse to ArtifactGenerationResponse and TextGenerationResponse.
  • Added ffmpeg to use artifact generation response.
  • Got specific APIs to return specific generation response and use consistent Results.

Down stream tests and apps will need to be updated to prep for release.

…ld be more consistently typed.

- Split GenerationResponse to ArtifactGenerationResponse and TextGenerationResponse.
- Added ffmpeg to use artifact generation response.
- Got specific APIs to return specific generation response and use consistent `Results`.

Down stream tests and apps will need to be updated to prep for release.
@Layoric Layoric merged commit c24d30f into main Nov 25, 2024
2 checks passed
@Layoric Layoric deleted the job-status-refactor branch November 25, 2024 06:34
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.

1 participant