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.

Sign up for GitHub

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

Jump to bottom

Add execution time testing #19

Open
lGrom opened this issue Jul 29, 2024 · 1 comment
Open

Add execution time testing #19

lGrom opened this issue Jul 29, 2024 · 1 comment
Assignees
@lGrom

Comments

@lGrom
Copy link
Owner

lGrom commented Jul 29, 2024

Add execution time testing

Run the functions multiple times, take the average time to execute, and check if it's below a certain threshold.
@lGrom lGrom added the todo label Jul 29, 2024
@lGrom lGrom self-assigned this Jul 29, 2024
@lGrom lGrom linked a pull request Jul 30, 2024 that will close this issue
rename bi/unidirectional to cardinal/orthogonal #22
Merged
@lGrom lGrom closed this as completed in #22 Jul 30, 2024
@lGrom lGrom reopened this Jul 30, 2024
@lGrom lGrom removed the todo label Jul 30, 2024
@lGrom lGrom removed a link to a pull request Jul 30, 2024
rename bi/unidirectional to cardinal/orthogonal #22
Merged
@lGrom
Copy link
Owner Author

lGrom commented Jul 30, 2024

Try adding benchmark.js as a dev dependency. We'll also need to add that that as a github action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@lGrom lGrom

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@lGrom