Skip to content

WIP: feat: Keyboard Teleop - #3328

Draft
paul-nechifor wants to merge 1 commit into
paul/feat/t7from
paul/feat/t6
Draft

WIP: feat: Keyboard Teleop#3328
paul-nechifor wants to merge 1 commit into
paul/feat/t7from
paul/feat/t6

Conversation

@paul-nechifor

@paul-nechifor paul-nechifor commented Aug 2, 2026

Copy link
Copy Markdown
Contributor

Closes DIM-1168

@codecov

codecov Bot commented Aug 2, 2026

Copy link
Copy Markdown

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
3552 1 3551 174
View the top 1 failed test(s) by shortest run time
dimos.codebase_checks.test_no_sections::test_no_section_markers
Stack Traces | 0.725s 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 2 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             .../web/relay_bridge/test_relay_bridge_e2e.py:403: # --- Teleop e2e: viewer datagrams -> relay lease gate -> bridge publishes ---
E             .../web/relay_bridge/test_relay_bridge_module.py:1174: # --- Teleop (the tele_cmd_vel tx channel) ---

lineno     = 1174
path       = '.../web/relay_bridge/test_relay_bridge_module.py'
report_lines = ['Found 2 section marker(s). If a file is too complicated to be understood without sections, then the sections should ...shes ---', '  .../web/relay_bridge/test_relay_bridge_module.py:1174: # --- Teleop (the tele_cmd_vel tx channel) ---']
text       = '# --- Teleop (the tele_cmd_vel tx channel) ---'
violations = [('.../web/relay_bridge/test_relay_bridge_e2e.py', 403, '# --- Teleop e2e: viewer datagrams -> relay lease gate -> b... ---'), ('.../web/relay_bridge/test_relay_bridge_module.py', 1174, '# --- Teleop (the tele_cmd_vel tx channel) ---')]

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.

@paul-nechifor paul-nechifor changed the title WIP: t6 WIP: feat: Keyboard Teleop Aug 2, 2026
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.

1 participant