commented out Mem Transport test case

[leshy]

since it's so fast it ruins the graph

[greptile-apps]

Greptile Summary

Commented out the Memory transport benchmark test case to prevent it from skewing graph visualizations due to its extremely fast performance compared to other transport implementations.

• Memory transport uses direct in-process function calls with no serialization overhead, making it orders of magnitude faster than network-based transports (LCM, UDP, Redis, ROS) and shared memory transports
• The extreme performance difference makes it difficult to visualize and compare the performance characteristics of real-world transports on benchmark graphs
• The Memory transport implementation and message generator remain in the codebase for potential future use

Confidence Score: 5/5

• Safe to merge - simple comment-out change that improves benchmark visualization without affecting functionality
• The change only comments out a single test case registration without modifying any logic or removing the underlying implementation. The Memory transport code remains available for future use, and all other transport benchmarks continue to function normally. This is a non-breaking change that addresses a legitimate UX issue with benchmark graph readability.
• No files require special attention

Important Files Changed

Filename	Overview
dimos/protocol/pubsub/benchmark/testdata.py	Commented out Memory transport test case to avoid skewing benchmark results due to its extremely fast performance

Sequence Diagram

sequenceDiagram
    participant Test as test_benchmark.py
    participant TestData as testdata.py
    participant Memory as Memory Transport
    participant Other as Other Transports
    
    Note over TestData: Before Change
    TestData->>TestData: testcases.append(Memory)
    Test->>TestData: Import testcases
    TestData-->>Test: [LCM, UDP, Memory, SHM...]
    Test->>Memory: Run benchmark
    Memory-->>Test: Extremely fast results
    Note over Test: Memory skews graph scale
    
    Note over TestData: After Change
    TestData->>TestData: # testcases.append(Memory)
    Test->>TestData: Import testcases
    TestData-->>Test: [LCM, UDP, SHM, Redis, ROS...]
    Test->>Other: Run benchmarks
    Other-->>Test: Comparable results
    Note over Test: Better graph visualization

Loading