kaxil commented on code in PR #71462:
URL: https://github.com/apache/airflow/pull/71462#discussion_r3765827123
##########
providers/anthropic/docs/operators/anthropic.rst:
##########
@@ -175,15 +177,34 @@ model requests once its tracked list cost reaches the
ceiling:
agent_id="agt_...",
environment_id="env_...",
message="Summarise yesterday's incidents.",
- session_kwargs={
- "budget": {
- "type": "limit",
- # Minor units as an integer string: "2500" is $25.00.
- "max_list_cost": {"amount": "2500", "currency": "USD"},
- }
- },
+ budget=25.00, # $25.00
Review Comment:
Changed to `# 25.00 USD`. Putting the currency after the number avoids the
`$25,00` ambiguity you flagged without the odd `USD$` doubling.
##########
providers/anthropic/tests/unit/anthropic/operators/test_agent.py:
##########
@@ -199,6 +199,67 @@ def test_deferrable_defers_with_trigger(self,
mock_hook_prop):
hook.wait_for_session.assert_not_called()
+class TestBudgetParam:
+ @staticmethod
+ def _op(**kwargs):
Review Comment:
Applied, along with its five call sites in that class.
--
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]
