Update json formatting to latest of current-bench

ocaml-bench / sandmark

A benchmark suite for the OCaml compiler

The Unlicense

82 stars 40 forks source link

Update json formatting to latest of current-bench #419

Closed ElectreAAS closed 1 year ago

ElectreAAS commented 1 year ago

As per the (incomplete) code in the readme of current-bench, the metrics object shouldn't have anything in it that isn't plottable, so no booleans nor nulls nor strings. This PR puts everything of that sorts in the context object so it's not lost.

punchagan commented 1 year ago

Thanks for working on this, @ElectreAAS! Merged it after minor adjustments to the JSON - removed an extra , and the unnecessary [ ] around the metrics value. Also, added a commit to run make json in CI to catch any broken json generation.