Skip to content

Add rohd waveform package#681

Open
desmonddak wants to merge 7 commits into
intel:mainfrom
desmonddak:add-rohd-waveform-package
Open

Add rohd waveform package#681
desmonddak wants to merge 7 commits into
intel:mainfrom
desmonddak:add-rohd-waveform-package

Conversation

@desmonddak

Copy link
Copy Markdown
Contributor

Description & Motivation

This is a package used to communicate from ROHD debugging to the ROHD waveform viewer. It can supply waveform services to other tools that query for incremental waveform updates as the simulator is running.

Related Issue(s)

None.

Testing

Local standalone tests are included. It has also been tested in a full path through a remote copy of the ROHD waveform viewer (upcoming release).

Backwards-compatibility

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

No. This is a new capability.

Documentation

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

None. This is an internal protocol which we will document when stable.

Comment thread packages/rohd_waveform/README.md Outdated
Comment thread packages/rohd_waveform/pubspec.yaml Outdated
Comment thread rohd_devtools_extension/packages/rohd_devtools_widgets/lib/src/regex_utils.dart Outdated
Comment thread test/benchmark_test.dart Outdated
Comment thread packages/rohd_waveform/lib/src/models/metadata.dart
Comment thread packages/rohd_waveform/lib/src/models/signal_waveform.dart Outdated
Comment thread packages/rohd_waveform/lib/src/models/signal_waveform.dart
Comment thread packages/rohd_waveform/lib/src/waveform_api.dart Outdated
Comment thread packages/rohd_waveform/test/metadata_test.dart Outdated
Comment thread packages/rohd_waveform/test/signal_waveform_test.dart
@desmonddak

Copy link
Copy Markdown
Contributor Author

I added flutter test coverage and a set of durable tests that should survive the downstream merges as a core set of tests for the rohd_devtools_extension as well.

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.

2 participants