Detecting complete sync of books from device

[chaley]

Originally Posted by kovidgoyal

Use

self.gui.device_signals.signal_name.connect(...

while making sure you are running the most recent source from trunk.

[meme]

Got the most recent source, but it didn't work as self.gui. I noticed in the code that device_signals was a global so I used:

from calibre.gui2.device import device_signals

device_signals.device_metadata_available.connect(s elf.testconnect1)
device_signals.device_connection_changed.connect(s elf.testconnect)


And this works. Is this what you intended?

It doesn't seem to matter if its in genesis or initialization complete. I didn't even know about initialization complete - should I be using it instead of genesis.

[kovidgoyal]

Yeah, that's fine, I misremembered. Using genesis is fine. You typically only use initialization_complete if something your plugin does depends on initialization being completed.

[chaley]

Originally Posted by meme

Got the most recent source, but it didn't work as self.gui. I noticed in the code that device_signals was a global so I used:

from calibre.gui2.device import device_signals

device_signals.device_metadata_available.connect(s elf.testconnect1)
device_signals.device_connection_changed.connect(s elf.testconnect)


And this works. Is this what you intended?

Yes.

FYI: the problem before is that signals cannot be declared in mixin classes. Well, they can be declared, but emitting one of them corrupts memory and the application falls over dead. Moving them to a global fixes it. Kovid mentioned that I also could have put them into the outer-level gui class, which would be the only solution if we had multiple gui instances.

[meme]

Its working well, and quite fun to test Maybe I should go back to 7.55 to give me more time to check before metadata is read

Many thanks.