Skip to content

Conversation

@mkorbel1
Copy link
Contributor

Description & Motivation

When None is passed for dart_command in order to perform interactive debug of the Dart process in cosimulation with the SystemVerilog simulator, there was a bug where the timeout variable was still used even if it was set to None as well. This caused a crash in python.

This PR

  • fixes the bug, so that the combination of dart_command = None and dart_connect_timeout = None works properly with the rohd_port_connector.py
  • adds tests to cover this bug fix
  • adds tests to cover that python crashing shows up as a failure in the output logs
  • improves python error logging to print the stack trace that led to the error when it occurs within the rohd-cosim python infrastructure
  • exposed the extractSocketPort function on CosimConnection so that it can be reused to extract port information from the stdout of the cosimulation process

Related Issue(s)

N/A

Testing

Added new tests, existing tests cover things

Backwards-compatibility

Is this a breaking change that will not be backwards-compatible? If yes, how so?

No, just bug fixes, API improvements, and error logging enhancements

Documentation

Does the change require any updates to documentation? If so, where? Are they included?

No, API docs are minorly updated

@mkorbel1 mkorbel1 merged commit f3d6451 into intel:main Jun 17, 2025
2 checks passed
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