Beta testers for calibre 9 needed

[DNSB]

Originally Posted by jackie_w

I am unable to see the bookshelf at all in calibre portable v8.99.6. I do have an error in bookshelf_view.py as soon as I open calibre.

Spoiler:

DEBUG:    0.5 No Kobo Touch, Glo or Mini appears to be connected
DEBUG:    0.5 rebuild_menus - self.supports_ratings=None, self.supports_tiles=None
DEBUG:    0.5 KoboUtilities:set_toolbar_button_tooltip - start: text='None'
DEBUG:    0.5 KoboUtilities:set_toolbar_button_tooltip - setting to text='Utilities to use with Kobo ereaders

Driver: KTExtra'
[0.75] main UI initialized...

Out of a perhaps morbid curiosity, how did you get the KTExtra driver to show? On my 8.99.6 *, I can not install the KTE driver never mind have it show as the installed driver.

[jackie_w]

Originally Posted by JSWolf

You do have a problem with KTExtra. It looks like the same error KoboTouch has.

The debug log I posted shows no errors in KTExtra (a personal plugin), so I've no idea what you're talking about.

Apart from one of those "future warnings" from the Extract ISBN plugin I see only one error and it's clearly related to Bookshelf code, namely bookshelf_view.py (i.e. nothing to do with KoboTouch) and you tell me this is already fixed. I'm not running from source so can't confirm the fix until there's a new release of calibre portable.

[jackie_w]

Originally Posted by DNSB

Out of a perhaps morbid curiosity, how did you get the KTExtra driver to show? On my 8.99.6 *, I can not install the KTE driver never mind have it show as the installed driver.

KTExtra is a personal plugin which started out as KoboTouchExtended, many years ago, then morphed into a kind of Kobo driver sandpit for adding some whim features (which seemed like a good idea at the time ) and removing some original features I thought were pointless and/or unhelpful.

When KoboTouchExtended got blacklisted I was initially horrified because I'd grown attached to some of those whims, so I had to re-work everything under a new, not-blacklisted name, hence KTExtra. Took a while but I got there in the end.

[PeterT]

Originally Posted by DNSB

That is a Python error. As the error message says, the updated Python 3.14.2 in the calibre beta does not support the \x string format. This will need to be fixed in the plugins.

The error only affects command line / testing usage of the plugin.

@jswolf the thing to remember is that python "processes" all the source in the plugin file when it's loaded into calibre, so errors like this are not necessarily in code that will be executed.

[kovidgoyal]

The prototype version use it, but Kovid drop it when he reworked the shelf to what is this now. The main reason I suspect is because the shelf is become pretty heavy rendering process, but I'd rather not go into speculation about that, it's Kovid domain. Personaly I'm okay with that.

The shelves and case back are textured and shadowed, to give more of a feel of physical shelves. Trying to allow arbitrary colors/textures with that is waaay too much work.

[DNSB]

Originally Posted by jackie_w

KTExtra is a personal plugin which started out as KoboTouchExtended, many years ago, then morphed into a kind of Kobo driver sandpit for adding some whim features (which seemed like a good idea at the time ) and removing some original features I thought were pointless and/or unhelpful.

When KoboTouchExtended got blacklisted I was initially horrified because I'd grown attached to some of those whims, so I had to re-work everything under a new, not-blacklisted name, hence KTExtra. Took a while but I got there in the end.

Thanks for the explanation. I was wondering where KTExtra came from.

[DNSB]

Originally Posted by DNSB

That is a Python error. As the error message says, the updated Python 3.14.2 in the calibre beta does not support the \x string format. This will need to be fixed in the plugins.

Minor correction. Python 3.14.2 still supports that syntax but it is not going to be supported in some future version. This does give time to fix the plugins that use that form.

[JSWolf]

Originally Posted by jackie_w

The debug log I posted shows no errors in KTExtra (a personal plugin), so I've no idea what you're talking about.

Apart from one of those "future warnings" from the Extract ISBN plugin I see only one error and it's clearly related to Bookshelf code, namely bookshelf_view.py (i.e. nothing to do with KoboTouch) and you tell me this is already fixed. I'm not running from source so can't confirm the fix until there's a new release of calibre portable.

Here is the fix...

Originally Posted by kovidgoyal

@JSWolf: This fixes your issue: https://github.com/kovidgoyal/calibr...17c05caba2d41d

[DisabledReader]

Originally Posted by kovidgoyal

Hmm well only way forward I can see is to disable the plugins one byone till we find the conflicting one.

I have spent this morning disabling and deleting plugins in my beta test installation and still haven't been able to pinpoint the problem.
But, if you can add a restart option after changes are made to Shelf height in the bookshelf view, then the freeze/crash doesn't happen.
I hope this helps.

[kovidgoyal]

Originally Posted by DisabledReader

I have spent this morning disabling and deleting plugins in my beta test installation and still haven't been able to pinpoint the problem.
But, if you can add a restart option after changes are made to Shelf height in the bookshelf view, then the freeze/crash doesn't happen.
I hope this helps.

Thanks for trying. I did find one deadlock but no crashes, let's see if the deadlock fix helps your crash as well.

[JSWolf]

Originally Posted by DNSB

Minor correction. Python 3.14.2 still supports that syntax but it is not going to be supported in some future version. This does give time to fix the plugins that use that form.

calibre_plugins.count_pages.statistics:415: SyntaxWarning: "\D" is an invalid escape sequence. Such sequences will not work in the future. Did you mean "\\D"? A raw string is also an option.
calibre_plugins.count_pages.statistics:416: SyntaxWarning: "\D" is an invalid escape sequence. Such sequences will not work in the future. Did you mean "\\D"? A raw string is also an option.

    get_cbz_page_count('''C:\Dev\Tools\eclipse\workspace\_Misc\misery-depot.zip''')
    get_cbr_page_count('''C:\Dev\Tools\eclipse\workspace\_Misc\misery-depot.cbr''')

How would this be fixed so it no longer brings up the warning?

[un_pogaz]

Originally Posted by JSWolf

calibre_plugins.count_pages.statistics:415: SyntaxWarning: "\D" is an invalid escape sequence. Such sequences will not work in the future. Did you mean "\\D"? A raw string is also an option.
calibre_plugins.count_pages.statistics:416: SyntaxWarning: "\D" is an invalid escape sequence. Such sequences will not work in the future. Did you mean "\\D"? A raw string is also an option.

    get_cbz_page_count('''C:\Dev\Tools\eclipse\workspace\_Misc\misery-depot.zip''')
    get_cbr_page_count('''C:\Dev\Tools\eclipse\workspace\_Misc\misery-depot.cbr''')

How would this be fixed so it no longer brings up the warning?

Use / instead of \
Or r'''C:\Dev\Tools\eclipse\workspace\_Misc\misery-depot.zip''' (don't forget r prefix at the start at the string, it mean is a raw string and escape sequence ar not take acount)

[JSWolf]

Originally Posted by un_pogaz

Use / instead of \
Or r'''C:\Dev\Tools\eclipse\workspace\_Misc\misery-depot.zip''' (don't forget r prefix at the start at the string, it mean is a raw string and escape sequence ar not take acount)

Would this be correct?

    get_cbz_page_count(r'''C:\Dev\Tools\eclipse\workspace\_Misc\misery-depot.zip''')
    get_cbr_page_count(r'''C:\Dev\Tools\eclipse\workspace\_Misc\misery-depot.cbr''')

[un_pogaz]

Originally Posted by JSWolf

Would this be correct?

    get_cbz_page_count(r'''C:\Dev\Tools\eclipse\workspace\_Misc\misery-depot.zip''')
    get_cbr_page_count(r'''C:\Dev\Tools\eclipse\workspace\_Misc\misery-depot.cbr''')

Yes, that it.

[JSWolf]

Originally Posted by un_pogaz

Yes, that it.

Thanks. It works. I fixed Count Page.