Instapaper recipe issue & fix

mrabbitt · 10-04-2016, 05:16 PM

Hi. I recently experienced errors using the Instapaper recipe with some longer articles, and traced it down to this error:

Code:

SplitError: Could not find reasonable point at which to split: feed_0/article_9/index_u33.html Sub-tree size: 606 KB
http://www.w3.org/1999/xhtml}h3 /*/*[2]/*[4]/*/*[2]/*[6]/*[19]

I looked at the processed HTML of one of the articles, and found a huge section of hidden HTML in a <div id='speedRead'> element which replicated the text of the article with each word wrapped in individual <span> elements. I tried adding the "speedRead" div to the remove_tags attribute of the recipe, and that solved the problem.

I haven't contributed to Calibre recipes before, but would like to contribute this fix back to the community as it is probably affecting others. (It may be the cause of the issue reported in this thread.) What's the preferred way to do this? Should I submit a pull request to the github project?

Thank you!

kovidgoyal · 10-04-2016, 08:54 PM

Yes, a pull request is the preferred way. Or you can attach the updated recipe here or on a bug report. I'm not fussy

mrabbitt · 10-05-2016, 01:28 AM

Cool, thanks, kovidgoyal. I created a pull request: https://github.com/kovidgoyal/calibre/pull/559

Thanks for the quick response and for all your work developing Calibre!

10-04-2016, 05:16 PM	#1
mrabbitt Junior Member Posts: 2 Karma: 10 Join Date: Oct 2016 Device: Nook Simple Touch	Instapaper recipe issue & fix Hi. I recently experienced errors using the Instapaper recipe with some longer articles, and traced it down to this error: Code: SplitError: Could not find reasonable point at which to split: feed_0/article_9/index_u33.html Sub-tree size: 606 KB http://www.w3.org/1999/xhtml}h3 //[2]/[4]//[2]/[6]/*[19] I looked at the processed HTML of one of the articles, and found a huge section of hidden HTML in a <div id='speedRead'> element which replicated the text of the article with each word wrapped in individual <span> elements. I tried adding the "speedRead" div to the remove_tags attribute of the recipe, and that solved the problem. I haven't contributed to Calibre recipes before, but would like to contribute this fix back to the community as it is probably affecting others. (It may be the cause of the issue reported in this thread.) What's the preferred way to do this? Should I submit a pull request to the github project? Thank you!

Similar Threads
Thread	Thread Starter	Forum	Replies	Last Post
Custom Instapaper Recipe	haroldtreen	Recipes	9	05-27-2025 06:10 PM
Instapaper - Updated recipe	khromov	Recipes	78	01-23-2015 01:09 AM
Instapaper recipe - broken by site redesign?	adfadfsasdfafafd	Recipes	11	06-02-2014 08:31 AM
Instapaper recipe not working as of this week	largeboulder	Recipes	7	09-04-2013 02:59 AM
Kobo cannot fix my issue, can you?	misticaleclipse	Kobo Reader	12	10-09-2011 04:45 PM

10-04-2016, 08:54 PM	#2
kovidgoyal creator of calibre Posts: 45,353 Karma: 27182818 Join Date: Oct 2006 Location: Mumbai, India Device: Various	Yes, a pull request is the preferred way. Or you can attach the updated recipe here or on a bug report. I'm not fussy

10-05-2016, 01:28 AM	#3
mrabbitt Junior Member Posts: 2 Karma: 10 Join Date: Oct 2016 Device: Nook Simple Touch	Cool, thanks, kovidgoyal. I created a pull request: https://github.com/kovidgoyal/calibre/pull/559 Thanks for the quick response and for all your work developing Calibre!

Advert