Skip to content

Support for badges generation#117

Merged
machshev merged 2 commits intolowRISC:masterfrom
machshev:badges
Mar 17, 2026
Merged

Support for badges generation#117
machshev merged 2 commits intolowRISC:masterfrom
machshev:badges

Conversation

@machshev
Copy link
Collaborator

We use badges within the OpenTitan documentation to report on regression status. It's helpful if DVSim is capable of generating these badges. We can then update the regression runner to make these available with the nightly/weekly reports as required.

This is a cut down summary JSON that provides the data that the
dashboard HTML report provides.

Signed-off-by: James McCorrie <james.mccorrie@lowrisc.org>
Signed-off-by: James McCorrie <james.mccorrie@lowrisc.org>
@martin-velay
Copy link

I am not sure I understand this PR, but have you considered the formal flow? Or isn't it something we care about here? And FYI later on we'll have to add the vPlan coverage (and maybe spec cov too).

@machshev
Copy link
Collaborator Author

I am not sure I understand this PR, but have you considered the formal flow? Or isn't it something we care about here? And FYI later on we'll have to add the vPlan coverage (and maybe spec cov too).

@martin-velay this is a step towards being able to generate the badges such as the ones at the top of the documentation pages. Which are currently outdated and need generating with the new regression runs. If we bake this into DVSim then we are not having to couple other systems with knowledge of JSON results schemas, we just use DVSim to generate the svg files and the regression runner publishes them to a new dashboard bucket.

Once we have this working with Sim flow it would be useful to have a conversation about how this should look for formal flow.

Copy link

@martin-velay martin-velay left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I haven't tested it, but LGTM

@machshev machshev added this pull request to the merge queue Mar 17, 2026
Merged via the queue into lowRISC:master with commit 92b8f44 Mar 17, 2026
6 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.

4 participants