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

change: move claude3-provider-common code to anthropic-model-provider #348

Merged
merged 2 commits into from
Jan 20, 2025

Conversation

iwilltry42
Copy link
Contributor

@iwilltry42 iwilltry42 commented Jan 15, 2025

@iwilltry42 iwilltry42 force-pushed the change/anthropic-shared branch from 9bf203a to f893bd9 Compare January 20, 2025 11:40
@iwilltry42
Copy link
Contributor Author

Rebased and opened the related PR on the enterprise-tools repo: https://github.com/obot-platform/enterprise-tools/pull/2

Comment on lines +53 to +56
return StreamingResponse(
"data: " + mapped_response.model_dump_json() + "\n\n",
media_type="application/x-ndjson",
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this mean that these don't do streaming?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I have a Follow-Up PR to switch between streaming and non-streaming in the works.
This is mostly a copy-paste of the old module

@iwilltry42 iwilltry42 merged commit f075bce into obot-platform:main Jan 20, 2025
1 check passed
@iwilltry42 iwilltry42 deleted the change/anthropic-shared branch January 20, 2025 16:54
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