Enabling BBC plugin dramatically slows totem initialisation

[Chris Jones]

Binary package hint: totem

If I enable the BBC plugin (which is a great idea), totem seems to take much longer to start even if I am playing local content.
Going by the console output it's initialising the BBC plugin that takes the time. Is it doing something at that point which it should be doing when I choose it in the UI? (e.g. the Youtube plugin initialises quickly and only does hard work when I explicitly search with it)

[Jono Bacon (jonobacon)]

Some output when running totem with the BBC plug-in loaded (I also get a horribly slow init time for totem):

** (totem:11679): DEBUG: Creating object of type BBCViewer+TotemPythonPlugin
** (totem:11679): DEBUG: Creating Python plugin instance
Traceback (most recent call last):
  File "/usr/lib/totem/plugins/bbc/bbc.py", line 48, in activate
    self.view = ContentView()
  File "/usr/lib/totem/plugins/bbc/contentview.py", line 638, in __init__
    if not self.pool.load():
  File "/usr/lib/totem/plugins/bbc/contentview.py", line 590, in load
    cache_fn = self.ensureCache()
  File "/usr/lib/totem/plugins/bbc/contentview.py", line 571, in ensureCache
    cache_file.delete()

[Colin Watson (cjwatson)]

This should be fixed by Collabora's top priority at the moment, namely converting all the network access to use asynchronous I/O. The delay is because it's downloading the feed from the BBC's site.

[Colin Watson (cjwatson)]

Let's track Jono's output as bug 276959.

[Doru Horișco (doruhushhush)]

This is the output, when I run totem from command line:

** (totem:6713): DEBUG: Init of Python module
** (totem:6713): DEBUG: Registering Python plugin instance: BBCViewer+TotemPythonPlugin
** (totem:6713): DEBUG: Creating object of type BBCViewer+TotemPythonPlugin
** (totem:6713): DEBUG: Creating Python plugin instance
cache file name /home/doru/.cache/totem/plugins/bbc/content-000000-00000-00000000.cache
loading available content failed
** (totem:6713): DEBUG: Init of Python module
** (totem:6713): DEBUG: Registering Python plugin instance: YouTube+TotemPythonPlugin
** (totem:6713): DEBUG: Creating object of type YouTube+TotemPythonPlugin
** (totem:6713): DEBUG: Creating Python plugin instance

After the plugin is loaded it still not working. In alpha4 and alpha5 it worked well.

[Dan Andreșan (danyer)]

Same here. Is the BBC Content available only to people in the UK?

[Colin Watson (cjwatson)]

Being in the UK or not has nothing to do with this bug. (My understanding is that the BBC are going to provide a UK feed and an international feed, determined server-side by GeoIP. Some content will only be in the UK feed, but the international feed will not be empty by any means.)

[Launchpad Janitor (janitor)]

This bug was fixed in the package totem - 2.24.1-0ubuntu3

---------------
totem (2.24.1-0ubuntu3) intrepid; urgency=low

  * debian/patches/65_bbc-plugin.patch:
    - Update from Collabora, including:
      + Switch to production feed URL (LP: #274740).
      + Update format mappings, including Ogg, ASF, MP3, Matroska, WMA, WMV,
        Flash video, and Dirac.
      + Take configured connection speed into account when selecting the
        best encoding for an episode.
  * debian/control.in:
    - Depend/build-depend on python-gtk2(-dev) (>= 2.13.0) for BBC plugin.

 -- Colin Watson <email address hidden> Mon, 06 Oct 2008 19:22:41 +0100

[Colin Watson (cjwatson)]

Sorry, I shouldn't have closed this bug. You can at least now fetch BBC content again, but network access is still synchronous, so this bug is still open.

[Dan Andreșan (danyer)]

I'll test once totem 2.24.1-0ubuntu3 appears in my updates list. At the moment I am at
Installed: 2.24.1-0ubuntu2
Candidate: 2.24.1-0ubuntu2

[Dan Andreșan (danyer)]

It is really impressive. Great thing. I liked the Dirac Test. Although, one problem: a lot of items are displayed but when I clicked on them it said "An error occured|Could not open location; you might not have permission to open the file." Why is it displayed then?

Colin, I realise that this is not the proper place to raise this issue, could you give me the bug number for the issue I'm seeing or tell me if I need to create a new bug report.

Thanks again,
Dan.

[Tim Müller (t-i-m-zen)]

I'm still working on the async loading, it should be ready soon.

[Dan Andreșan (danyer)]

Thanks Tim, the async loading would be great. As it stands know, I can wait if I want to play BBC content, but not if I just started Totem for song or something.

[Ron B. (flybye882001)]

Bug seems to have been resolved with the updates that arrived this morning - or?

[Colin Watson (cjwatson)]

Ron: not as such, no, but I just got an update from Collabora in my mailbox that should deal with it. I'll upload it once I've tested it a bit.

[Ron B. (flybye882001)]

well I am no expert - but all that has been done here that seems at all
relevant is the big (166 files - ) partial update that was done this
morning. The fact that I have upgraded the cpu also from amd x24200 to a
5000 doesnt make things slower either heheh.

ron b.

Colin Watson schrieb:
> Ron: not as such, no, but I just got an update from Collabora in my
> mailbox that should deal with it. I'll upload it once I've tested it a
> bit.
>
> ** Changed in: totem (Ubuntu Intrepid)
> Status: Triaged => In Progress
>
>

[Ron B. (flybye882001)]

further to this theme.. please don't break it again...! ;=)

ron b.

Colin Watson schrieb:
> Ron: not as such, no, but I just got an update from Collabora in my
> mailbox that should deal with it. I'll upload it once I've tested it a
> bit.
>
> ** Changed in: totem (Ubuntu Intrepid)
> Status: Triaged => In Progress
>
>

[Launchpad Janitor (janitor)]

This bug was fixed in the package totem - 2.24.2-0ubuntu2

---------------
totem (2.24.2-0ubuntu2) intrepid; urgency=low

  * debian/patches/65_bbc-plugin.patch:
    - Update from Collabora, including:
      + Do network I/O, rdf parsing and apt.Cache() loading asynchronously
        or in a dedicated thread (LP: #274740).
      + Show loading progress and error messages in case of failure.
      + Fix installable codec check when the first codec in the list is
        installed but a subsequent one is not.
    - Note upstream bug in patch header.

 -- Colin Watson <email address hidden> Fri, 10 Oct 2008 21:53:06 +0100