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

Performance refactorings #58

Merged
merged 17 commits into from Mar 26, 2017

Conversation

Projects
None yet
2 participants
@thibaudcolas
Collaborator

thibaudcolas commented Mar 25, 2017

Not too big of a speed bump but good to take nonetheless. I would expect real-world content (where text is way more common than entities and styles) to see a higher speed up.

# Numbers for the perf test suite, and the example (notice how the difference is inverted).
Before: 23420 23688
After: 16631 16344
@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Mar 26, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 4c8c4ad on chore/perf-update into ed56f4d on master.

coveralls commented Mar 26, 2017

Coverage Status

Coverage remained the same at 100.0% when pulling 4c8c4ad on chore/perf-update into ed56f4d on master.

@thibaudcolas thibaudcolas merged commit 3dbd597 into master Mar 26, 2017

3 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 100.0%
Details

@thibaudcolas thibaudcolas deleted the chore/perf-update branch Mar 26, 2017

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