Skip to content

[src] Integrate DVPlan tool commands#142

Open
martin-velay wants to merge 1 commit intolowRISC:masterfrom
martin-velay:dvplan_integ
Open

[src] Integrate DVPlan tool commands#142
martin-velay wants to merge 1 commit intolowRISC:masterfrom
martin-velay:dvplan_integ

Conversation

@martin-velay
Copy link
Copy Markdown

  • Integrate DVPlan subcommands. These commands will be run only if:
    • DVPlan tool is available
    • Coverage is enabled
    • A "".vplan" file is found in the cuurent sim_cfg.hjson file
  • When the condition are met, at the end the regression/simulation, the DVPlan tool will:
    • Check the coverage of the specification
    • Generate a derived vPlan in the simulation directory
    • Perform back-annotation of the vPlan with the coverage data
    • Compute the coverage of the vPlan
    • Generate a report based on obtained annotated vPlan

@martin-velay martin-velay force-pushed the dvplan_integ branch 2 times, most recently from fd95bec to 570f42a Compare April 2, 2026 13:27
@martin-velay martin-velay force-pushed the dvplan_integ branch 3 times, most recently from bc457b5 to 3fbb934 Compare April 2, 2026 14:53
@machshev
Copy link
Copy Markdown
Collaborator

machshev commented Apr 6, 2026

Looks like there is a Ruff lint issue?

- Integrate DVPlan subcommands. These commands will be run only if:
    - DVPlan tool is available
    - Coverage is enabled
    - A "".vplan" file is found in the cuurent sim_cfg.hjson file
- When the condition are met, at the end the regression/simulation, the
  DVPlan tool will:
    - Check the coverage of the specification
    - Generate a derived vPlan in the simulation directory
    - Perform back-annotation of the vPlan with the coverage data
    - Compute the coverage of the vPlan
    - Generate a report based on obtained annotated vPlan

Signed-off-by: martin-velay <[email protected]>
@martin-velay
Copy link
Copy Markdown
Author

Hi, I have rebased on the master. Can you review my PR?

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