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

[feat] Streaming Update #798

Merged
merged 106 commits into from
Jun 6, 2024
Merged

[feat] Streaming Update #798

merged 106 commits into from
Jun 6, 2024

Conversation

CalebCourier
Copy link
Collaborator

Notable Changes:

  • Escape validator names in on-fail attributes

  • Add required_model_auth to Manifest class

  • Enable passing serializeable metadata to api

  • API streaming support

  • Refactor guard save & enable guard.use syntax for api

  • Fix Notebook Runs (Fix Notebook Runs #793)

  • Loosen otel requirements

  • Use semver for package versions in checks

nichwch added 30 commits May 7, 2024 12:34
@CalebCourier CalebCourier marked this pull request as ready for review June 6, 2024 13:47
@CalebCourier
Copy link
Collaborator Author

Notebooks are failing for reasons unrelated to this PR, but we can't be assured of these changes until we fix them. Working on that now.

@CalebCourier CalebCourier changed the base branch from main to 0.4.5-dev June 6, 2024 18:19
@@ -144,7 +144,7 @@ async def async_run(

return call_log

@async_trace(name="step")
@async_trace(name="async_step")
Copy link
Contributor

Choose a reason for hiding this comment

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

We need to maybe also revert this at some point? But currently it breaks if it uses 'step' due to some async issues

@CalebCourier CalebCourier merged commit 2de04f4 into 0.4.5-dev Jun 6, 2024
@CalebCourier CalebCourier deleted the feat/streaming-update branch June 6, 2024 18:38
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