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

Consolidate benchmark generating programs #617

Closed
xrchz opened this issue Feb 11, 2019 · 1 comment
Closed

Consolidate benchmark generating programs #617

xrchz opened this issue Feb 11, 2019 · 1 comment
Labels
tooling Things like building, linters and LSP

Comments

@xrchz
Copy link
Member

xrchz commented Feb 11, 2019

Currently there are two similar (but not identical) Python programs compiler/benchmarks/cakeml_benchmarks/benchmark.py and .../mlton_benchmarks/benchmark.py. This issue is to refactor these programs so that only 1 or fewer Python programs are required.

For example, this might involve moving the differences between the files above into a configuration file or command line arguments.

Ideally, the Python functionality would be removed altogether (e.g., replaced by a CakeML program), or reduced to a program with a clear function and interface (e.g., just for creating a plot given a data file and configuration arguments).

@dnezam dnezam added the tooling Things like building, linters and LSP label Dec 14, 2024
@ordinarymath
Copy link
Contributor

Note that currently the benchmark is currently broken in multiple ways. Subsumed by #1126

@ordinarymath ordinarymath closed this as not planned Won't fix, can't repro, duplicate, stale Jan 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
tooling Things like building, linters and LSP
Projects
None yet
Development

No branches or pull requests

3 participants