Skip to content

Ivan/feat/scene - #3279

Open
leshy wants to merge 8 commits into
mainfrom
ivan/feat/scene
Open

Ivan/feat/scene#3279
leshy wants to merge 8 commits into
mainfrom
ivan/feat/scene

Conversation

@leshy

@leshy leshy commented Jul 29, 2026

Copy link
Copy Markdown
Member

dimos/perception/memory/python tool_localize.py "coke can" --from 30 --duration 120 && dimos-viewer localize.rrd

leshy and others added 6 commits July 29, 2026 20:43
This reverts commit ce2c71a, except for
dimos/models/embedding/dino.py, which is kept: DINOModel is a standalone
EmbeddingModel whose only dependencies (dimos.models.base,
dimos.models.embedding.base, dimos.msgs.sensor_msgs.Image) predate the
reverted commit.

Conflict in dimos/robot/all_blueprints.py resolved by keeping the
wrist-camera entry added after the merge; only the two world-belief
registry entries are removed.
@mintlify

mintlify Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
dimensional 🟢 Ready View Preview Jul 29, 2026, 10:30 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

@codecov

codecov Bot commented Jul 29, 2026

Copy link
Copy Markdown

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
3245 1 3244 178
View the top 1 failed test(s) by shortest run time
dimos.codebase_checks.test_no_sections::test_no_section_markers
Stack Traces | 0.692s run time
def test_no_section_markers():
        """
        Fail if any file contains section-style comment markers.
    
        If a file is too complicated to be understood without sections, then the
        sections should be files. We don't need "subfiles".
        """
        violations = find_section_markers()
        if violations:
            report_lines = [
                f"Found {len(violations)} section marker(s). "
                "If a file is too complicated to be understood without sections, "
                'then the sections should be files. We don\'t need "subfiles".',
                "",
            ]
            for path, lineno, text in violations:
                report_lines.append(f"  {path}:{lineno}: {text.strip()}")
>           raise AssertionError("\n".join(report_lines))
E           AssertionError: Found 1 section marker(s). If a file is too complicated to be understood without sections, then the sections should be files. We don't need "subfiles".
E           
E             .../perception/memory/tool_localize.py:190: # --- rerun output: direct logging, everything on the "ts" timeline ---

lineno     = 190
path       = '.../perception/memory/tool_localize.py'
report_lines = ['Found 1 section marker(s). If a file is too complicated to be understood without sections, then the sections should ....../perception/memory/tool_localize.py:190: # --- rerun output: direct logging, everything on the "ts" timeline ---']
text       = '# --- rerun output: direct logging, everything on the "ts" timeline ---'
violations = [('.../perception/memory/tool_localize.py', 190, '# --- rerun output: direct logging, everything on the "ts" timeline ---')]

dimos/codebase_checks/test_no_sections.py:145: AssertionError

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

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.

2 participants