Feat/ivan/pure modules

[leshy]

No description provided.

[codecov]

❌ 2 Tests Failed:

Tests completed	Failed	Passed	Skipped
1922	2	1920	155

View the top 2 failed test(s) by shortest run time

dimos.project.test_get_logger::test_no_get_logger

Stack Traces | 0.181s run time

def test_no_get_logger():
        """
        Fail if any file uses `= logging.getLogger` outside the whitelist.
        """
        violations = find_get_logger_usages()
        if violations:
            report_lines = [
                f"Found {len(violations)} forbidden use(s) of `logging.getLogger`. "
                "Use `setup_logger` instead:",
                "",
                "    from dimos.utils.logging_config import setup_logger",
                "",
                "    logger = setup_logger()",
                "",
                "If the usage is legitimate (e.g. standalone script, logging "
                "infrastructure, or third-party logger suppression), add it to the "
                "WHITELIST in dimos/project/test_get_logger.py.",
                "",
            ]
            for path, lineno, text in violations:
                report_lines.append(f"  {path}:{lineno}: {text.strip()}")
>           raise AssertionError("\n".join(report_lines))
E           AssertionError: Found 1 forbidden use(s) of `logging.getLogger`. Use `setup_logger` instead:
E           
E               from dimos.utils.logging_config import setup_logger
E           
E               logger = setup_logger()
E           
E           If the usage is legitimate (e.g. standalone script, logging infrastructure, or third-party logger suppression), add it to the WHITELIST in dimos/project/test_get_logger.py.
E           
E             dimos/memory2/tick.py:66: logger = logging.getLogger(__name__)

lineno     = 66
path       = 'dimos/memory2/tick.py'
report_lines = ['Found 1 forbidden use(s) of `logging.getLogger`. Use `setup_logger` instead:', '', '    from dimos.utils.logging_config import setup_logger', '', '    logger = setup_logger()', '', ...]
text       = 'logger = logging.getLogger(__name__)'
violations = [('dimos/memory2/tick.py', 66, 'logger = logging.getLogger(__name__)')]

dimos/project/test_get_logger.py:124: AssertionError

dimos.robot.test_all_blueprints_generation::test_all_blueprints_is_current

Stack Traces | 2.55s run time

def test_all_blueprints_is_current() -> None:
        root = DIMOS_PROJECT_ROOT / "dimos"
        all_blueprints, all_modules = _scan_for_blueprints(root)
    
        common = set(all_blueprints.keys()) & set(all_modules.keys())
        assert not common, (
            f"Names must be unique across blueprints and modules, "
            f"but these appear in both: {sorted(common)}"
        )
    
        generated_content = _generate_all_blueprints_content(all_blueprints, all_modules)
    
        file_path = root / "robot" / "all_blueprints.py"
    
        if "CI" in os.environ:
            if not file_path.exists():
                pytest.fail(f"all_blueprints.py does not exist at {file_path}")
    
            current_content = file_path.read_text()
            if current_content != generated_content:
                diff = difflib.unified_diff(
                    current_content.splitlines(keepends=True),
                    generated_content.splitlines(keepends=True),
                    fromfile="all_blueprints.py (current)",
                    tofile="all_blueprints.py (generated)",
                )
                diff_str = "".join(diff)
>               pytest.fail(
                    f"all_blueprints.py is out of date. Run "
                    f"`pytest dimos/robot/test_all_blueprints_generation.py` locally to update.\n\n"
                    f"Diff:\n{diff_str}"
                )
E               Failed: all_blueprints.py is out of date. Run `pytest dimos/robot/test_all_blueprints_generation.py` locally to update.
E               
E               Diff:
E               --- all_blueprints.py (current)
E               +++ all_blueprints.py (generated)
E               @@ -180,6 +180,7 @@
E                    "local-planner": "dimos.navigation.nav_stack.modules.local_planner.local_planner.LocalPlanner",
E                    "manipulation-module": "dimos.manipulation.manipulation_module.ManipulationModule",
E                    "map": "dimos.robot.unitree.type.map.Map",
E               +    "marker-detection-pure-module": "dimos.perception.fiducial.marker_detection_pure_module.MarkerDetectionPureModule",
E                    "marker-detection-stream-module": "dimos.perception.fiducial.marker_detection_stream_module.MarkerDetectionStreamModule",
E                    "marker-tf-module": "dimos.perception.fiducial.marker_tf_module.MarkerTfModule",
E                    "mcp-client": "dimos.agents.mcp.mcp_client.McpClient",
E               @@ -207,6 +208,7 @@
E                    "pgo": "dimos.navigation.nav_stack.modules.pgo.pgo.PGO",
E                    "phone-teleop-module": "dimos.teleop.phone.phone_teleop_module.PhoneTeleopModule",
E                    "pick-and-place-module": "dimos.manipulation.pick_and_place_module.PickAndPlaceModule",
E               +    "pure-module": "dimos.memory2.puremodule.PureModule",
E                    "quest-teleop-module": "dimos.teleop.quest.quest_teleop_module.QuestTeleopModule",
E                    "ray-tracing-voxel-map": "dimos.mapping.ray_tracing.module.RayTracingVoxelMap",
E                    "real-sense-camera": "dimos.hardware.sensors.camera.realsense.camera.RealSenseCamera",

all_blueprints = {'alfred-nav': 'dimos.robot.diy.alfred.blueprints.alfred_nav:alfred_nav', 'coordinator-basic': 'dimos.control.blueprin...sian_ik_mock', 'coordinator-cartesian-ik-piper': 'dimos.control.blueprints.teleop:coordinator_cartesian_ik_piper', ...}
all_modules = {'alfred-high-level': 'dimos.robot.diy.alfred.effector_high_level.AlfredHighLevel', 'arm-teleop-module': 'dimos.teleop..._navigation.BBoxNavigationModule', 'b1-connection-module': 'dimos.robot.unitree.b1.connection.B1ConnectionModule', ...}
common     = set()
current_content = '# Copyright 2025-2026 Dimensional Inc.\n#\n# Licensed under the Apache License, Version 2.0 (the "License");\n# you m...ebsocket_vis_module.WebsocketVisModule",\n    "zed-camera": "dimos.hardware.sensors.camera.zed.camera.ZEDCamera",\n}\n'
diff       = <generator object unified_diff at 0xff67e2879f20>
diff_str   = '--- all_blueprints.py (current)\n+++ all_blueprints.py (generated)\n@@ -180,6 +180,7 @@\n     "local-planner": "dimos...le.RayTracingVoxelMap",\n     "real-sense-camera": "dimos.hardware.sensors.camera.realsense.camera.RealSenseCamera",\n'
file_path  = PosixPath('.../dimos/robot/all_blueprints.py')
generated_content = '# Copyright 2025-2026 Dimensional Inc.\n#\n# Licensed under the Apache License, Version 2.0 (the "License");\n# you m...ebsocket_vis_module.WebsocketVisModule",\n    "zed-camera": "dimos.hardware.sensors.camera.zed.camera.ZEDCamera",\n}\n'
root       = PosixPath('.../dimos/dimos/dimos')

dimos/robot/test_all_blueprints_generation.py:66: Failed

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