Skip to content

chore: remove annoying logging messages - #3612

Open
paul-nechifor wants to merge 1 commit into
mainfrom
paul/fix/ignore-some-warnings
Open

chore: remove annoying logging messages#3612
paul-nechifor wants to merge 1 commit into
mainfrom
paul/fix/ignore-some-warnings

Conversation

@paul-nechifor

Copy link
Copy Markdown
Contributor

No description provided.

@codecov

codecov Bot commented Aug 22, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 85.71429% with 1 line in your changes missing coverage. Please review.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
...imos/perception/experimental/spatial_perception.py 0.00% 1 Missing ⚠️
@@           Coverage Diff           @@
##             main    #3612   +/-   ##
=======================================
  Coverage   77.19%   77.20%           
=======================================
  Files        1260     1260           
  Lines      119881   119850   -31     
  Branches    10581    10580    -1     
=======================================
- Hits        92539    92527   -12     
+ Misses      24276    24260   -16     
+ Partials     3066     3063    -3     
Flag Coverage Δ
OS-ubuntu-24.04-arm 72.08% <85.71%> (+0.01%) ⬆️
OS-ubuntu-latest 74.00% <85.71%> (+0.02%) ⬆️
Py-3.10 73.99% <85.71%> (+0.02%) ⬆️
Py-3.11 73.99% <85.71%> (+0.02%) ⬆️
Py-3.12 73.99% <85.71%> (+0.01%) ⬆️
Py-3.13 73.99% <85.71%> (+0.02%) ⬆️
Py-3.14 73.99% <85.71%> (+0.01%) ⬆️
Py-3.14t 73.99% <85.71%> (+0.01%) ⬆️
SelfHosted-Large 30.04% <71.42%> (+0.01%) ⬆️
SelfHosted-Linux 35.18% <57.14%> (-0.01%) ⬇️

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

Files with missing lines Coverage Δ
dimos/agents/mcp/mcp_client.py 83.72% <100.00%> (+0.23%) ⬆️
dimos/cli/commands/lifecycle.py 60.84% <100.00%> (ø)
dimos/core/coordination/module_coordinator.py 86.56% <100.00%> (ø)
dimos/memory/tf.py 97.91% <ø> (ø)
dimos/perception/experimental/image_embedding.py 53.43% <ø> (-1.72%) ⬇️
dimos/perception/experimental/spatial_vector_db.py 63.15% <ø> (+7.31%) ⬆️
dimos/protocol/tf/tf.py 97.04% <100.00%> (ø)
dimos/stream/audio/node_output.py 51.85% <ø> (+0.04%) ⬆️
...imos/perception/experimental/spatial_perception.py 54.86% <0.00%> (+0.36%) ⬆️

... and 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@greptile-apps

greptile-apps Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This update adjusts application diagnostic behavior and transformation lookup interfaces.

Confidence Score: 5/5

No blocking failure remains.

No accepted blocking failure remains.

T-Rex T-Rex Logs

What T-Rex did

  • Reviewed the general contract validation proof across TF components: the base TFLookup.get signature lacks a keyword-only warn parameter, while the live MultiTBuffer.get and replay StreamTF.get definitions include forward_tolerance and warn as keyword-only arguments, and the replay path executed with exit code 0 reporting VERIFIED.

View all artifacts

T-Rex Ran code and verified through T-Rex

Reviews (2): Last reviewed commit: "chore: remove annoying logging messages" | Re-trigger Greptile

Comment thread dimos/agents/mcp/mcp_client.py
Comment thread dimos/protocol/tf/tf.py
@paul-nechifor
paul-nechifor force-pushed the paul/fix/ignore-some-warnings branch from a0e27aa to 8f6ebd9 Compare August 22, 2026 02:00
@github-actions github-actions Bot added the ready-to-merge Required CI checks have passed on this PR label Aug 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-merge Required CI checks have passed on this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant