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.

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

Improve performance of Pipeline #52

Closed
bertfrees opened this issue Nov 17, 2016 · 3 comments
Closed

Improve performance of Pipeline #52

bertfrees opened this issue Nov 17, 2016 · 3 comments
Assignees

Comments

@bertfrees
Copy link
Contributor

bertfrees commented Nov 17, 2016

This is just a reference to snaekobbi/issues#28. This way I'm adding the issue to SBS's to-do list. I have assigned it to me but note that I will be working on it on behalf of DAISY as well.

This also includes shortening the time it takes to run the test suite.

+1 from:

@bertfrees
Copy link
Contributor Author

bertfrees commented Dec 6, 2016

I updated the sbs branch to the latest xprocspec which has logging to file disabled. This had a really good effect on the dotify-formatter tests (on both the master branch and sbs branch) but unfortunately the sbs tests are still a problem (which suggests that for the sbs tests the logging wasn't the main issue).

@bertfrees
Copy link
Contributor Author

bertfrees commented Mar 21, 2017

This table lists the conversion times of test books provided by Mischa for different revisions:

777d5f4 9654ef2 css:split in Java (dirty)
short.xml 337 12s 12s
fiction_engel.xml 103K 30s (ERROR) 27s
nonfiction_natur.xml 670K 730s (ERROR) 281s (ERROR) 65s
fiction_haus.xml 865K 820s 83s 74s
nonfiction_vegeta.xml 866K 783s (ERROR) (ERROR)
fiction_kuss.xml 969K 674s (ERROR)
fiction_grey.xml
fiction_arena.xml 2.8M 2842s 166s

∞ = aborted process because took too long

@bertfrees
Copy link
Contributor Author

After I fixed a memory leak in Calabash (daisy/xmlcalabash1@92a3460), the test suite finished again.

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

No branches or pull requests

2 participants