Fix PerformanceMonitor operation tracking #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Closed

marcomelloni wants to merge 1 commit into simulation-bridge from codex/fix-performance_monitor-to-support-tuples

Owner

marcomelloni commented Jul 14, 2025

Summary

use (operation_id, protocol) tuple as PerformanceMonitor key
propagate protocol argument throughout event recording calls
adjust tests and docs for new key logic

Testing

pytest -q (fails: ModuleNotFoundError: No module named 'cryptography')

https://chatgpt.com/codex/tasks/task_e_6874cad42c64833385a1535a2174daf8


          Use protocol-operation tuple for PerformanceMonitor

de7cddf

marcomelloni added the codex label

— with

ChatGPT Codex Connector

marcomelloni deleted the branch simulation-bridge

July 15, 2025 14:13

marcomelloni closed this

marcomelloni deleted the codex/fix-performance_monitor-to-support-tuples branch

July 15, 2025 14:13

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels