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

Make included benchmark pluggable for different dataset #9

Open
raganhan opened this issue Oct 31, 2018 · 1 comment
Open

Make included benchmark pluggable for different dataset #9

raganhan opened this issue Oct 31, 2018 · 1 comment

Comments

@raganhan
Copy link
Contributor

It would be nice to make this pluggable for different data sets as well, since the performance of the path extractor is highly dependent on the characteristics of the data. It would be good to eventually provide results for several data sets along with a description of the data and what was skipped. This doesn't block the initial release.

Originally posted by @tgregg in #8
@raganhan raganhan mentioned this issue Oct 31, 2018
Merged
@raganhan
Copy link
Contributor Author

It's probably good to not force the text version in other benchmarks as it was included mostly to demonstrate you get equivalent performance to the DOM as there is no skip scanning

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@raganhan