Skip to content

Conversation

@marcomelloni
Copy link
Owner

Summary

  • expand PerformanceMonitor key to include protocol, client ID and simulation type
  • maintain an index keyed only by protocol and request ID for subsequent events
  • document the new unique key fields in performance metrics guide

Testing

  • pytest simulation_bridge/test/unit/test_performance_monitor.py::test_start_operation_creates_metric -q (fails: ModuleNotFoundError: No module named 'yaml')

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

@codecov-commenter
Copy link

Codecov Report

Attention: Patch coverage is 66.66667% with 8 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
simulation_bridge/src/utils/performance_monitor.py 66.66% 7 Missing and 1 partial ⚠️

📢 Thoughts on this report? Let us know!

@marcomelloni marcomelloni deleted the codex/aggiungi-tracciamento-per-tutti-i-campi 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants