Function calling upgrade

[davorrunje]

Why are these changes needed?

Two improvements related to recent feedback from Discord:

• async/sync mixing of functions and chat initiations works now for all combinations. Previously it was silently ignored in some cases which caused an exception to be raised later making it hard to debug.
• an additional utility function register_function was added to simplify the registration of functions without decorators.

def currency_calculator(
  base: Annotated[Currency, "Base currency: amount and currency symbol"],
  quote_currency: Annotated[CurrencySymbol, "Quote currency symbol"] = "USD",
) -> Currency:
  quote_amount = exchange_rate(base.currency, quote_currency) * base.amount
  return Currency(amount=quote_amount, currency=quote_currency)

autogen.agentchat.register_function(
    currency_calculator,
    caller=chatbot,
    executor=user_proxy,
    description="Currency exchange calculator.",
)

Related issue number

Checks

• I've included any doc changes needed for https://microsoft.github.io/autogen/. See https://microsoft.github.io/autogen/docs/Contribute#documentation to build and test documentation locally.
• I've added tests (if relevant) corresponding to the changes introduced in this PR.
• I've made sure all auto checks have passed.

[codecov-commenter]

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (0107b52) 32.69% compared to head (9b007bd) 45.70%.

Files	Patch %	Lines
autogen/agentchat/conversable_agent.py	84.00%	2 Missing and 2 partials ⚠️

Additional details and impacted files

@@             Coverage Diff             @@
##             main    #1443       +/-   ##
===========================================
+ Coverage   32.69%   45.70%   +13.00%     
===========================================
  Files          42       42               
  Lines        5019     5039       +20     
  Branches     1151     1220       +69     
===========================================
+ Hits         1641     2303      +662     
+ Misses       3248     2552      -696     
- Partials      130      184       +54     

Flag	Coverage Δ
unittests	45.62% <85.18%> (+12.96%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ekzhu]

In notebooks, when calling initiate_chat and a_initiate_chat we can use the with Cache.disk() as cache context to make user aware of the cache feature. So we can gradually get rid of the legacy cache_seed setting in llm_config.

[sonichi]

I guess the test failure is due to a recent change #1471

[afourney]

I guess the test failure is due to a recent change #1471

Yes. Just remove the "Tool Call Id: ...\n" lines from the content fields of the test.

[davorrunje]

I guess the test failure is due to a recent change #1471

Yes. Just remove the "Tool Call Id: ...\n" lines from the content fields of the test.

Yup, that was it. Thanx!