Skip to content

Conversation

@mattf
Copy link
Contributor

@mattf mattf commented Jun 25, 2025

What does this PR do?

mark inference.{embeddings,chat_completion,completion} deprecated per llamastack/llama-stack#2365

Test Plan

pytest tests/test_deprecated_inference.py

extra_body: Body | None = None,
timeout: float | httpx.Timeout | None | NotGiven = NOT_GIVEN,
) -> ChatCompletionResponse | Stream[ChatCompletionResponseStreamChunk]:
warnings.warn(
Copy link
Contributor

Choose a reason for hiding this comment

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

hm, this is actually auto-generated code. I think we'd need to look into stainless to see if this can be done.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

is this something i can do?

Copy link
Contributor

Choose a reason for hiding this comment

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

@mattf unfortunately not since stainless is controlled by us (meta) right now. we are working to make this be community owned also.

@ashwinb
Copy link
Contributor

ashwinb commented Jul 17, 2025

I think we might be able to get this landed. But could you make the same PR for the https://github.com/llamastack/llama-stack-client-python/ repository? I will archive this repository now.

@ashwinb
Copy link
Contributor

ashwinb commented Sep 26, 2025

Closing this since these changes need to be generated via the Stainless SDK

@ashwinb ashwinb closed this Sep 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants