Skip to content
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.

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

yosys-witness concat yw trace files #3902

Merged
merged 2 commits into from
Aug 25, 2023
Merged

yosys-witness concat yw trace files #3902

merged 2 commits into from
Aug 25, 2023

Conversation

jix
Copy link
Member

@jix jix commented Aug 25, 2023

Intended for use with SCY to combine sequential cover traces.
Arbitrary number of inputs, will load all and attempt to join them. Each trace will be replayed one after the other, in the same order as the files are provided.
Mismatch in init_only fields seems to work fine, with values in subsequent traces being assigned in the initial only if they weren't previously defined.
Uncertain if a mismatch in non init_only fields will cause problems.
Fixes WitnessSig.eq().
Adds helper functions to WitnessValues and ReadWitness classes.

Intended for use with SCY to combine sequential cover traces.
Arbitrary number of inputs, will load all and attempt to join them.  Each trace will be replayed one after the other, in the same order as the files are provided.
Mismatch in init_only fields seems to work fine, with values in subsequent traces being assigned in the initial only if they weren't previously defined.
Uncertain if a mismatch in non init_only fields will cause problems.
Fixes WitnessSig.__eq__().
Adds helper functions to WitnessValues and ReadWitness classes.
Can now append a user defined number of steps to input traces when joining.
If the number of steps is +ve, inputs are all set to 0.
If -ve then steps are skipped.
If all of steps are skipped (including init step) then the input trace will not be copied.
If more than one input trace is provided, the append option will need to be provided the same number of times as there are input traces.
@jix jix added the check-sby Run sby tests for this PR label Aug 25, 2023
@mmicko mmicko merged commit c6caadf into master Aug 25, 2023
15 checks passed
@mmicko mmicko deleted the krys/yw_join branch August 25, 2023 13:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
check-sby Run sby tests for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants