Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

update loggers #810

Merged
merged 1 commit into from
Feb 6, 2025
Merged

update loggers #810

merged 1 commit into from
Feb 6, 2025

Conversation

lazToum
Copy link
Collaborator

@lazToum lazToum commented Feb 4, 2025

Why are these changes needed?

Adds an extra check before using "object.name"

Related issue number

Closes #809

Checks

Copy link

codecov bot commented Feb 4, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Flag Coverage Δ
3.10 58.42% <100.00%> (+2.44%) ⬆️
3.11 29.13% <0.00%> (?)
3.12 27.39% <0.00%> (?)
3.13 41.59% <0.00%> (+7.87%) ⬆️
3.9 40.04% <0.00%> (+6.83%) ⬆️
anthropic 16.39% <0.00%> (?)
bedrock 16.76% <0.00%> (?)
cerebras 16.48% <0.00%> (?)
cohere 16.52% <0.00%> (?)
docs 28.86% <0.00%> (ø)
gemini 16.47% <0.00%> (?)
graph-rag-falkor-db 28.30% <0.00%> (ø)
groq 16.49% <0.00%> (?)
interop 29.75% <0.00%> (ø)
jupyter-executor 31.02% <0.00%> (ø)
lmm 19.35% <0.00%> (?)
long-context 15.62% <0.00%> (?)
macos-latest 58.48% <100.00%> (+2.40%) ⬆️
mistral 16.20% <0.00%> (?)
ollama 16.88% <0.00%> (?)
retrievechat 29.87% <0.00%> (+0.04%) ⬆️
retrievechat-mongodb 29.86% <0.00%> (+1.28%) ⬆️
retrievechat-pgvector 29.86% <0.00%> (+1.28%) ⬆️
retrievechat-qdrant 29.80% <0.00%> (+1.30%) ⬆️
teachable 15.80% <0.00%> (?)
together 16.53% <0.00%> (?)
twilio 28.51% <0.00%> (ø)
ubuntu-latest 58.51% <100.00%> (+2.44%) ⬆️
websurfer 18.75% <0.00%> (?)
windows-latest 56.12% <50.00%> (+2.45%) ⬆️

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

Files with missing lines Coverage Δ
autogen/logger/file_logger.py 66.66% <100.00%> (ø)
autogen/logger/sqlite_logger.py 62.73% <100.00%> (ø)

... and 10 files with indirect coverage changes

@davorrunje davorrunje self-assigned this Feb 6, 2025
@davorrunje davorrunje added this pull request to the merge queue Feb 6, 2025
Merged via the queue into main with commit d35bea2 Feb 6, 2025
295 of 302 checks passed
@davorrunje davorrunje deleted the update_logger branch February 6, 2025 23:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: CaptainAgent with runtime_logging throws Error: 'NoneType' object has no attribute 'name'
2 participants