-
Notifications
You must be signed in to change notification settings - Fork 38
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
Renaming suggestions for *run_config.json files #302
Comments
@Sudha247 can I rename these as per the suggestion? |
Hi @moazzammoriani, thanks for the interest. I'd suggest taking this up after finishing #310, if no one has claimed it by then. I must also admit #310 is a bit more challenging than this one, but I hope it's also more rewarding. :) |
Hi Sudha, can i work on this? |
Hi @codeamic, sure you may work on it. |
Thank you @Sudha247, i am on it |
@Sudha247 If this issue hasn't been resolved yet, I would love to work on it as a part of my outreach application. |
Currently the names of the json files are not illustrative and one has to look at the description in https://github.com/ocaml-bench/sandmark#config-files to know what they mean. Here is a renaming suggestion:
run_config.json
->sequential.json
multicore_parallel_run_config.json
->parallel_turing.json
multicore_parallel_navajo_run_config.json
->parallel_navajo.json
micro_multicore.json
->micro.json
The text was updated successfully, but these errors were encountered: