Log cache value message only once using lru_cache #1024

kumaranvpl · 2025-02-18T13:54:31Z

Why are these changes needed?

Related issue number

Closes #1022

Checks

I've included any doc changes needed for https://docs.ag2.ai/. See https://docs.ag2.ai/docs/contributor-guide/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 · 2025-02-18T13:59:54Z

Codecov Report

All modified and coverable lines are covered by tests ✅

Flag	Coverage Δ
3.10	`59.73% <100.00%> (+6.44%)`	⬆️
3.11	`58.69% <100.00%> (+4.01%)`	⬆️
3.12	`56.18% <100.00%> (+2.93%)`	⬆️
3.13	`59.33% <100.00%> (+6.36%)`	⬆️
3.9	`66.60% <100.00%> (+13.54%)`	⬆️
anthropic	`36.99% <100.00%> (?)`
autobuild	`16.79% <100.00%> (?)`
bedrock	`15.96% <60.00%> (?)`
browser-use	`33.94% <100.00%> (+0.29%)`	⬆️
cerebras	`15.69% <60.00%> (?)`
cohere	`15.73% <60.00%> (?)`
commsagent-discord	`29.98% <60.00%> (?)`
commsagent-slack	`29.96% <60.00%> (?)`
commsagent-telegram	`29.91% <60.00%> (?)`
crawl4ai	`33.26% <100.00%> (+0.45%)`	⬆️
deepseek	`32.50% <100.00%> (?)`
docs	`28.71% <60.00%> (?)`
gemini	`39.34% <100.00%> (?)`
graph-rag-falkor-db	`28.19% <60.00%> (?)`
groq	`15.71% <60.00%> (?)`
interop	`33.49% <100.00%> (+0.51%)`	⬆️
interop-crewai	`28.94% <60.00%> (?)`
interop-langchain	`28.77% <60.00%> (?)`
interop-pydantic-ai	`29.29% <60.00%> (?)`
jupyter-executor	`30.76% <60.00%> (?)`
lmm	`18.86% <60.00%> (?)`
long-context	`14.89% <60.00%> (?)`
macos-latest	`60.19% <100.00%> (+6.86%)`	⬆️
mistral	`15.43% <60.00%> (?)`
ollama	`16.08% <60.00%> (?)`
openai	`44.01% <100.00%> (?)`
retrievechat	`29.69% <60.00%> (?)`
retrievechat-couchbase	`17.20% <60.00%> (?)`
retrievechat-mongodb	`29.68% <60.00%> (?)`
retrievechat-pgvector	`29.68% <60.00%> (?)`
retrievechat-qdrant	`29.63% <60.00%> (?)`
teachable	`17.88% <100.00%> (?)`
together	`15.75% <60.00%> (?)`
twilio	`28.41% <60.00%> (?)`
ubuntu-latest	`68.35% <100.00%> (+13.56%)`	⬆️
websurfer	`17.85% <60.00%> (?)`
windows-latest	`57.98% <100.00%> (+6.89%)`	⬆️

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

Files with missing lines	Coverage Δ
autogen/oai/client.py	`76.03% <100.00%> (+20.89%)`	⬆️

... and 38 files with indirect coverage changes

* Log cache value message only once using lru_cache * Log cache value per client * Modify param name * Add test for log_cache_seed_value function * Use mock credentials and monkeypatch cache.get * Remove mark * Move import to appropriate location

Log cache value message only once using lru_cache

3933a52

kumaranvpl had a problem deploying to openai1 February 18, 2025 13:54 — with GitHub Actions Failure