Skip to content
This repository has been archived by the owner on Sep 30, 2023. It is now read-only.

new benchmarker/benchmarks; keeps old benchmarks #329

Closed
wants to merge 1 commit into from

Conversation

tabcat
Copy link
Contributor

@tabcat tabcat commented Apr 26, 2021

adds/updates benchmarks and an npm script to use the new benchmarker.
i decided not the do anything with the old benchmark files yet but we can add that as part of the pr if decided.

related: orbitdb-archive/benchmark-runner#32

@@ -51,7 +51,7 @@
"mocha": "^8.4.0",
"mocha-headless-chrome": "^3.1.0",
"nyc": "^15.0.1",
"orbit-db-benchmark-runner": "^1.0.3",
"orbit-db-benchmark-runner": "github:tabcat/benchmark-runner#benchmarker",
Copy link
Contributor

Choose a reason for hiding this comment

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

I suppose it's ok to publish a new version of the bechmark runner at this point?

Copy link
Contributor

Choose a reason for hiding this comment

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

I published [email protected] so you can use that here.

Copy link
Contributor

@aphelionz aphelionz left a comment

Choose a reason for hiding this comment

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

Whoops, meant to request changes

@aphelionz
Copy link
Contributor

We should probably remove the call to npm run benchmark in CI until we have a way to reliably compare performance across builds

@tabcat tabcat closed this Oct 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants