potiuk commented on code in PR #76244:
URL: https://github.com/apache/airflow/pull/76244#discussion_r4443218527
##########
providers/common/ai/docs/operators/llm.rst:
##########
@@ -206,6 +206,19 @@ With structured output:
:start-after: [START howto_decorator_llm_structured]
:end-before: [END howto_decorator_llm_structured]
+The returned prompt is rendered as a Jinja template, like the operator's
``prompt``
+argument, so ``{{ ds }}`` and other template variables work in it. Text you
insert into
+the returned string yourself, for example with an f-string, is rendered as
part of the
+template too. To include text that comes from a function argument, such as a
document or
+an upstream task's output, reference the argument by name in the template
instead of
+formatting it into the string. The function's arguments are available when the
prompt is
Review Comment:
Resolving this, since 52de6de covers all the points above: the bound call is
merged in all six decorators (`call_arguments` in `utils/prompt.py`, without
`apply_defaults()`), and the examples, Usage docstrings and this paragraph now
use `{{ ... }}`. There are tests for the positional call and for the `ti=None`
default. Feel free to reopen if anything still looks off.
--
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]
