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

sagemaker streaming #449

Merged
merged 2 commits into from
Oct 6, 2023
Merged

sagemaker streaming #449

merged 2 commits into from
Oct 6, 2023

Conversation

dartpain
Copy link
Contributor

@dartpain dartpain commented Oct 6, 2023

  • What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
    Adds proper sagemaker endpoint handling with respone streaming
  • Why was this change needed? (You can also link to an open issue here)
    It allows to stream responses now
  • Other information:

@vercel
Copy link

vercel bot commented Oct 6, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
docs-gpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 6, 2023 0:23am
nextra-docsgpt ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 6, 2023 0:23am

@codecov
Copy link

codecov bot commented Oct 6, 2023

Codecov Report

Merging #449 (4f5e363) into main (aa9a024) will increase coverage by 0.06%.
Report is 22 commits behind head on main.
The diff coverage is 17.85%.

❗ Current head 4f5e363 differs from pull request most recent head 4957285. Consider uploading reports for the commit 4957285 to get more accurate results

@@            Coverage Diff             @@
##             main     #449      +/-   ##
==========================================
+ Coverage   15.97%   16.03%   +0.06%     
==========================================
  Files          59       59              
  Lines        2673     2725      +52     
==========================================
+ Hits          427      437      +10     
- Misses       2246     2288      +42     
Files Coverage Δ
application/core/settings.py 100.00% <100.00%> (ø)
application/llm/sagemaker.py 22.22% <11.53%> (-31.12%) ⬇️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@dartpain dartpain merged commit cd79330 into main Oct 6, 2023
4 of 11 checks passed
@dartpain dartpain deleted the feature/sage-streaming branch January 19, 2024 14:13
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