Skip to content

Commit 420febf

Browse files
tgasser-nvPouyanpi
authored andcommitted
Pass OutputRailsStreamingConfig into _run_output_rails_in_streaming as mandatory argument as all 2 calls check for None
1 parent 7b776ba commit 420febf

File tree

1 file changed

+8
-4
lines changed

1 file changed

+8
-4
lines changed

nemoguardrails/rails/llm/llmrails.py

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,11 @@
8080
from nemoguardrails.logging.verbose import set_verbose
8181
from nemoguardrails.patch_asyncio import check_sync_call_from_async_loop
8282
from nemoguardrails.rails.llm.buffer import get_buffer_strategy
83-
from nemoguardrails.rails.llm.config import EmbeddingSearchProvider, RailsConfig
83+
from nemoguardrails.rails.llm.config import (
84+
EmbeddingSearchProvider,
85+
OutputRailsStreamingConfig,
86+
RailsConfig,
87+
)
8488
from nemoguardrails.rails.llm.options import (
8589
GenerationLog,
8690
GenerationOptions,
@@ -1350,6 +1354,7 @@ def stream_async(
13501354
):
13511355
return self._run_output_rails_in_streaming(
13521356
streaming_handler=generator,
1357+
output_rails_streaming_config=self.config.rails.output.streaming,
13531358
messages=messages,
13541359
prompt=prompt,
13551360
)
@@ -1405,6 +1410,7 @@ def task_done_callback(task):
14051410
# returns an async generator
14061411
return self._run_output_rails_in_streaming(
14071412
streaming_handler=streaming_handler,
1413+
output_rails_streaming_config=self.config.rails.output.streaming,
14081414
messages=messages,
14091415
prompt=prompt,
14101416
)
@@ -1633,6 +1639,7 @@ def __setstate__(self, state):
16331639
async def _run_output_rails_in_streaming(
16341640
self,
16351641
streaming_handler: AsyncIterator[str],
1642+
output_rails_streaming_config: OutputRailsStreamingConfig,
16361643
prompt: Optional[str] = None,
16371644
messages: Optional[List[dict]] = None,
16381645
stream_first: Optional[bool] = None,
@@ -1735,9 +1742,6 @@ def _prepare_params(
17351742
**action_params,
17361743
}
17371744

1738-
output_rails_streaming_config = self.config.rails.output.streaming
1739-
if output_rails_streaming_config is None:
1740-
raise ValueError("Output rails streaming config is not available")
17411745
buffer_strategy = get_buffer_strategy(output_rails_streaming_config)
17421746
output_rails_flows_id = self.config.rails.output.flows
17431747
stream_first = stream_first or output_rails_streaming_config.stream_first

0 commit comments

Comments
 (0)