Lee-W commented on code in PR #72150:
URL: https://github.com/apache/airflow/pull/72150#discussion_r4015488693


##########
providers/openai/docs/operators/openai.rst:
##########
@@ -53,6 +53,22 @@ The OpenAIResponseOperator requires the ``input_text`` 
prompt. Use the ``conn_id
 specify the OpenAI connection to use, and ``response_kwargs`` to pass through 
options such as
 ``tools``, ``conversation`` or ``previous_response_id``.
 
+Use ``max_output_tokens`` and ``max_tool_calls`` to cap generation per run -- 
both are templated,
+so a ceiling can vary by environment or Dag run without hardcoding it. 
``max_output_tokens`` caps
+the number of tokens generated; ``max_tool_calls`` caps the number of built-in 
tool calls the model

Review Comment:
   Added to the paragraph and docstring: hitting `max_tool_calls` changes no 
status and sets no `incomplete_details`, so there is no warning and nothing in 
`return_value` to tell it apart from a clean run.
   



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to