-
Notifications
You must be signed in to change notification settings - Fork 8
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
Send to Device fails with memory error #79
Comments
Forking issue to IOSRA as requested |
Attempt to send 10K+ resulted in lockup. CMD traceback: C:\Users\Rob>calibre-debug -g > debug_log.txt C++ exception thrown in slot: bad allocation |
I've built a version of iOSRA that should make some progress towards supporting very large downloads. Please download and install from this link. The change I made is to break up large 'Send to device' requests into chunks. By default, I've set the chunk size to 100 books. The chunk size shouldn't impact performance significantly, but if this works we can experiment with making it bigger. I'd like to ask that you test this in steps:
If this works, let's move on to the next test:
It's entirely possible that these tests will still fail, so it would be helpful to run in debug mode and capture the debug log. You can email it to me at the address I previously provided. I'm going to close the other issue you opened in the MXD issue tracker. The rule of thumb for deciding where to open an issue is "If the bug occurs while the MXD window is open, create the issue in the MXD bug tracker. Otherwise, create the issue in the iOSRA tracker". |
100 book test - success 500 book test - success 2000 book test - success 10930 book test - failed Debug log being sent via secondary address Additional details: Traceback (most recent call last): C:\Users\Rob>calibre-debug -g > debug_log_10930.txt |
Please download and install an updated version from this link. This should get through the full set of books. It still may crash later in the process when trying to report the results. |
I've created a new issue for iOSRA beta tester #80. Please go to that issue, then click the Subscribe button in the Notifications area. That will make it easier for me to notify all of the beta testers when a new version is available. |
@Dhardrian, I've found another bug related to multiple file transfers. This will require another code change within calibre proper, not the plugin. Unfortunately, that means waiting another release cycle of calibre to get the patch in place. Sorry. |
Let me know if/when I can help Sent from my iPad On May 10, 2014, at 3:14 PM, "GRiker" <notifications@github.commailto:notifications@github.com> wrote: @Dhardrianhttps://github.com/Dhardrian, I've found another bug related to multiple file transfers. This will require another code change within calibre proper, not the plugin. Unfortunately, that means waiting another release cycle of calibre to get the patch in place. Sorry. — |
Thanks, and sorry for the delay. I've already submitted the patch to Kovid for the next release, which should be next Friday. I will ping you when it's available. Are you by any chance running calibre from source? If so, you could test the change immediately. But if you're not, it's probably easier to just wait for the next release. |
Greg, I am not running from source. Timing is good, I am headed to Corpus Crisiti tomorrow, but will be back before Friday. Please send information about running from source if you have it. I could setup to do that later this week. Rob Sent from my iPad On May 11, 2014, at 11:05 AM, "GRiker" <notifications@github.commailto:notifications@github.com> wrote: Thanks, and sorry for the delay. I've already submitted the patch to Kovid for the next release, which should be next Friday. I will ping you when it's available. Are you by any chance running calibre from source? If so, you could test the change immediately. But if you're not, it's probably easier to just wait for the next release. — |
You can read about setting up a calibre development environment at this link. If you decide to do so, let me know after you get it running, and then I'll tell you what to do next. |
Re-opening issue. Problems have recurred in newer versions of Calibre and plugin |
The "Send to device" command is intermittently failing. I am updating meta data on existing books on the iDevice (i.e. Marvin configured to update rather than do nothing on existing). I have been able to trigger this error by sending as few as 901 books. Sending an entire library of 10930 books fails on every attempt.
Marvin version: 2.7
iDevice: iPad Model ME392LL/A
Calibre version: 1.34.0
Marvin XD plugin version:
iOS reading applications plugin version:
Operating system: Windows 7, 64-bit
iTunes version: 11.1.5.5
The specific steps to recreate the problem:
Attach iPad: Run Marvin: Select Get Books: Select Calibre
In Calibre, select "Send to Device"
Description of the problem:
Send to device fails with memory error.
The text was updated successfully, but these errors were encountered: