Vijay Ramesh

Thanks for the feedback I'll get on those changes over this weekend!

ok @ekzhu all your feedback should be addressed.

In that case I think we'd want to pull out cache_seed as well, which I can see was renamed after openai introduced the seed param, so it's also a bit...

ok updated per your feedback @ekzhu

marking it as draft while I do some full external testing (using my branch in another project and making sure it behaves as I'd expect end to end)

ok great verified locally ```python def healthcheck(self): joker_agent = autogen.AssistantAgent( name="Joker", llm_config=self.oai_config, system_message=AgentPrompts.healthcheck()["Joker"], ) joker_user_proxy = autogen.UserProxyAgent( "user_proxy", code_execution_config=False, system_message=AgentPrompts.healthcheck()["user_proxy"], human_input_mode="NEVER", llm_config=self.oai_config, max_consecutive_auto_reply=1, ) with Cache.redis() as cache: joker_user_proxy.initiate_chat( joker_agent,...

Docs were already added see https://github.com/microsoft/autogen/pull/1222/files#diff-623ecb7c953c4e7dc29eb44ac6de40b7a3c7f27929833c732a418bc1b419ec27R288

Ah I think I lost your change when updating from upstream main

OK cool moving the tests actually greatly simplifies things, now they just will run with the normal build and openai jobs. build job ran and reported coverage to https://app.codecov.io/github/microsoft/autogen/commit/354157af2f6907668994dd344599ec594d74cd97/tree/autogen/cache

I've submitted a pull request (https://github.com/clement/django-wurfl/pull/5) with at least a partial solution - instead of simply trying to match against 1/3rd of the UA string, it tries to match against...