Skip to content

Runtime benchmarks#1126

Merged
Perryvw merged 8 commits into
TypeScriptToLua:masterfrom
GlassBricks:runtime-benchmark
Sep 9, 2021
Merged

Runtime benchmarks#1126
Perryvw merged 8 commits into
TypeScriptToLua:masterfrom
GlassBricks:runtime-benchmark

Conversation

@GlassBricks

Copy link
Copy Markdown
Contributor

Adds basic runtime benchmarks, modeled off of existing memory benchmarks.

At least can be used for local benchmarks; Still need to test with github workflows.

@Perryvw

Perryvw commented Sep 7, 2021

Copy link
Copy Markdown
Member

Could you try uncommenting the benchmarks in: https://github.com/TypeScriptToLua/TypeScriptToLua/blob/master/.github/workflows/ci.yml? They are meant to be uncommented but I forgot it, and it would also be good to see what happens with your new benchmarks.

@GlassBricks

GlassBricks commented Sep 8, 2021

Copy link
Copy Markdown
Contributor Author

Are the benchmark output (markdown) supposed to be in a particular format for benchviz to work? In the CI, it displays a blank page.
Right now, I changed the formatting a bit on the benchmarks so my IDE views it better

EDIT: it seems to be an issue with the browser, not having access to Buffer...

Comment thread benchmark/src/run.ts
@GlassBricks GlassBricks requested a review from Perryvw September 9, 2021 18:17
@Perryvw Perryvw merged commit 3a3465b into TypeScriptToLua:master Sep 9, 2021
sanikoyes pushed a commit to sanikoyes/TypeScriptToLua that referenced this pull request Sep 24, 2021
* Basic runtime benchmarks

* Add basic doc for runtime benchmarks

* Fix runtime benchmark text formatting

* Add /benchmark/dist to .prettierignore

* Sort benchmarks by name

* Remove benchmark/dist/json.lua from gitignore

there's already benchmark/src/json.lua

* Enable benchmark in ci

* Refactor comparison Markdown formatting
@GlassBricks GlassBricks deleted the runtime-benchmark branch March 16, 2022 16:26
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