-
Notifications
You must be signed in to change notification settings - Fork 13.5k
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
Runnable astream_log does not pass kwargs to _astream_log_implementation #19054
Labels
🤖:bug
Related to a bug, vulnerability, unexpected error with an existing feature
Ɑ: Runnables
Related to Runnables
Comments
dosubot
bot
added
Ɑ: Runnables
Related to Runnables
🤖:bug
Related to a bug, vulnerability, unexpected error with an existing feature
labels
Mar 14, 2024
eyurtsev
pushed a commit
that referenced
this issue
Mar 14, 2024
…log (#19055) - **Description:** When calling the `_stream_log_implementation` from the `astream_log` method in the `Runnable` class, it is not handing over the `kwargs` argument. Therefore, even if i want to customize APIHandler and implement additional features with additional arguments, it is not possible. Conversely, the `astream_events` method normally handing over the `kwargs` argument. - **Issue:** #19054 - **Dependencies:** - **Twitter handle:** if your PR gets announced, and you'd like a mention, we'll gladly shout you out! Co-authored-by: hyungwookyang <hyungwookyang@worksmobile.com>
rahul-trip
pushed a commit
to daxa-ai/langchain
that referenced
this issue
Mar 27, 2024
…log (langchain-ai#19055) - **Description:** When calling the `_stream_log_implementation` from the `astream_log` method in the `Runnable` class, it is not handing over the `kwargs` argument. Therefore, even if i want to customize APIHandler and implement additional features with additional arguments, it is not possible. Conversely, the `astream_events` method normally handing over the `kwargs` argument. - **Issue:** langchain-ai#19054 - **Dependencies:** - **Twitter handle:** if your PR gets announced, and you'd like a mention, we'll gladly shout you out! Co-authored-by: hyungwookyang <hyungwookyang@worksmobile.com>
bechbd
pushed a commit
to bechbd/langchain
that referenced
this issue
Mar 29, 2024
…log (langchain-ai#19055) - **Description:** When calling the `_stream_log_implementation` from the `astream_log` method in the `Runnable` class, it is not handing over the `kwargs` argument. Therefore, even if i want to customize APIHandler and implement additional features with additional arguments, it is not possible. Conversely, the `astream_events` method normally handing over the `kwargs` argument. - **Issue:** langchain-ai#19054 - **Dependencies:** - **Twitter handle:** if your PR gets announced, and you'd like a mention, we'll gladly shout you out! Co-authored-by: hyungwookyang <hyungwookyang@worksmobile.com>
gkorland
pushed a commit
to FalkorDB/langchain
that referenced
this issue
Mar 30, 2024
…log (langchain-ai#19055) - **Description:** When calling the `_stream_log_implementation` from the `astream_log` method in the `Runnable` class, it is not handing over the `kwargs` argument. Therefore, even if i want to customize APIHandler and implement additional features with additional arguments, it is not possible. Conversely, the `astream_events` method normally handing over the `kwargs` argument. - **Issue:** langchain-ai#19054 - **Dependencies:** - **Twitter handle:** if your PR gets announced, and you'd like a mention, we'll gladly shout you out! Co-authored-by: hyungwookyang <hyungwookyang@worksmobile.com>
hinthornw
pushed a commit
that referenced
this issue
Apr 26, 2024
…log (#19055) - **Description:** When calling the `_stream_log_implementation` from the `astream_log` method in the `Runnable` class, it is not handing over the `kwargs` argument. Therefore, even if i want to customize APIHandler and implement additional features with additional arguments, it is not possible. Conversely, the `astream_events` method normally handing over the `kwargs` argument. - **Issue:** #19054 - **Dependencies:** - **Twitter handle:** if your PR gets announced, and you'd like a mention, we'll gladly shout you out! Co-authored-by: hyungwookyang <hyungwookyang@worksmobile.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
🤖:bug
Related to a bug, vulnerability, unexpected error with an existing feature
Ɑ: Runnables
Related to Runnables
Checked other resources
Example Code
pass
Error Message and Stack Trace (if applicable)
No response
Description
When calling the
_stream_log_implementation
from theastream_log
method in theRunnable
class, it is not handing over thekwargs
argument. Therefore, even if i want to customize APIHandler and implement additional features with additional arguments, it is not possible. Conversely, theastream_events
method normally handing over thekwargs
argument.System Info
pass
The text was updated successfully, but these errors were encountered: