Re: [Freevo-devel] Gah, 1.7.3's record server still broken!

2007-08-21 Thread Duncan Webb
Andrew Flegg wrote:
> On 8/21/07, Duncan Webb <[EMAIL PROTECTED]> wrote:
>>> Was this the first time using conflict detection by chance?
> [snip]
>> I don't think so as this is really an old old bug. It happened to me a
>> couple of times when I started using freevo.
> 
> Indeed, I'd also seen it before enabling conflict resolution and this
> wasn't the first time adding/editing a new favourite since enabling
> it.
> 
>> My guess is that is most likely caused by two processes updating the
>> record_schedule.xml at the same time. For example, when the recordserver
>> is doing an update and the webserver is adding a favourite.
> 
> Interesting. Unfortunately, there was no web access happening at the
> time and the TV updates happen at 07:16.

No background process running freevo schedulefavorites, freevo cache,
etc. I still suspect a race condition in the saving of the
record_schedule.xml. A write lock could be added around any code that
tries to read or write the record_schedule.xml, hopefully this is only
done in a couple of functions.

> In other words, the only thing accessing the record server at the time
> was the main Freevo UI on the TV.
> 
>> From Andrews bug report #126 you can see:
>> 2007-08-20 16:19:40,579 WARNING  recordserver.py (268) :
>> "/var/cache/freevo/record_schedule.xml" is invalid, removed
>> which means that the record_schedule.xml was corrupt and had to be
>> deleted. As the record_schedule.xml also contains the favourites this
>> means that they where also deleted. Ouch.
> 
> :-(
> 
>> Question for Andrew, is was there something recording at the same time as
>> he was using web interface? 16:18 is a good time to record children's
>> programmes.
> 
> Nope, nothing recording at the time either.
> 
> I've now got a little cronjob running as the PVR user:
> 
> 8<
> 0 * *   * *  perl -e '$a =
> "/var/cache/freevo/record_schedule.xml"; system("cp $a ~/") if -s $a >
> 10240'
> >8
> 
> So I feel safer doing some more tests ;-)
> 
> The strangest thing is why the web UI would work and the main UI
> wouldn't. Aren't they both talking to the recordserver using the same
> protocol and very similar code?

Before you update freevo please read the release notes in
freevo_config.py as I moved files around and you will need to move the
record_schedule.xml to it's new location. Since you are running freevo
as the user PVR it will create a ~/.freevo/static directory.

Duncan


-
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
___
Freevo-devel mailing list
Freevo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-devel


Re: [Freevo-devel] Panorama skin WAS: Re: Interesting failure

2007-08-21 Thread mike lewis
On 8/21/07, Gorka Olaizola <[EMAIL PROTECTED]> wrote:
> On Mon, Aug 20, 2007 at 09:12:58PM +0200, Duncan Webb wrote:
> >
> > MPlayer is a bit poor with videos that are 16x9 it seems to shrink them
> > vertically so the video has nice black borders.
> >
> I have a lot of videos that display as you say but they are corrected if
> I force its aspect ratio with "-aspect 16:9", "-aspect 1.85:1" etc...
> My "monitoraspect" variable is set to 16:9 always.
>
> I made a plugin that adds a submenu entry for changing it before playing
> the movie but didn't submit it because I suppose a FXD will work too and
> other people binds the aspect ratio to some buttons on the remote.
>
Ooh Ooh. Please submit. I've tried really hard to do this in the
pastand failed..  This would be a *great* feature.  Yes, users can do
this via remote, but to beable to hard code it would also be great..

Mick

-
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
___
Freevo-devel mailing list
Freevo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-devel


Re: [Freevo-devel] freevo 1.4 svn r9831 : doesn't work at all on my testbed

2007-08-21 Thread adam
> [EMAIL PROTECTED] schrieb:
>> Tanja, I wonder whether a better approach would be to factor the
>> scrollable full description stuff off into a separate file/class for use
>> by all (tvguide/recordings manager/video items/music etc) components or
>> perhaps add a new layout for video items that includes cover art?
>
[snip]
>
> So, if you can do the refactoring, that would be great!
> When I was thinking about that, I saw two problems.
> 1.) The eventhandler might be special to the item, e.g. as the programitem
> also
> needs to answer to REC.

I think we can get round this by either subclass the base class that will
handle the scrolling text or by having a callback function. Which one do
people prefer?

> 2.) The information one want to display might be different, e.g. plot
> description
> in the VideoItem and ProgramItem and Lyrics in the AudioItem and who knows
> what else.
> This might be solved, by putting the text constuction in the charge of the
> Item that
> wants to use the scrollable description.

I was just thinking that after having a quick browse of how its currently
used, this does seem the simplest/most sensible solution.

> Moreover I wondered, if the skin.py file is really the right place for
> this scrollable stuff,

I agree, but it was a case of 'hmm, where shall I put it' and I was
expecting people to say 'not there, but how about here...' :-)
I wondering whether to move the skin.py file into skin/init.py and the
data models into skin/models.py (notice thats skin not skins)

> but as I said, I never understood that completly and therefor decide to
> touch it as less
> as possible ;-)
>
> Again, I really like this scrollable area, it is really a great
> improvement
> and probably with time there will be more and more new additions which
> will use it.
> Refactoring would be great!

I'm also open for suggestion on the Text Entry component, I think this has
a great deal of potential for things like Video/Audio item search,
renaming files/favourites, dialling phone numbers, anything that requires
the user enter more than a simple selection. What would be really good is
for someone who has knowledge of right-to-left text input to improve the
code so that was supported as well. And finally ideas for how best to make
the keyboard internationalised via possible the translations

Cheers

Adam


-
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
___
Freevo-devel mailing list
Freevo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-devel


Re: [Freevo-devel] freevo 1.4 svn r9831 : doesn't work at all on my testbed

2007-08-21 Thread Tanja
[EMAIL PROTECTED] schrieb:
> Tanja, I wonder whether a better approach would be to factor the
> scrollable full description stuff off into a separate file/class for use
> by all (tvguide/recordings manager/video items/music etc) components or
> perhaps add a new layout for video items that includes cover art?

Hi Adam,

I also thought of that, but I did not understand enough of this skin business.
You are the skin expert :-)

At the moment my plan is to add a scrollable item
for all kinds of items that might need it.
I already have the recodings manager ready, but I do want to wait with the 
submit
until all other patches, that I already sent, are savely in the svn.
To much at a time brings confusion, as I realized ;-)

I also have the AudioItem on my to do list, for the lyrics plugin.

So, if you can do the refactoring, that would be great!
When I was thinking about that, I saw two problems.
1.) The eventhandler might be special to the item, e.g. as the programitem also
needs to answer to REC.
2.) The information one want to display might be different, e.g. plot 
description
in the VideoItem and ProgramItem and Lyrics in the AudioItem and who knows what 
else.
This might be solved, by putting the text constuction in the charge of the Item 
that
wants to use the scrollable description.
Moreover I wondered, if the skin.py file is really the right place for this 
scrollable stuff,
but as I said, I never understood that completly and therefor decide to touch 
it as less
as possible ;-)

Again, I really like this scrollable area, it is really a great improvement
and probably with time there will be more and more new additions which will use 
it.
Refactoring would be great!

Regards
Tanja

-
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
___
Freevo-devel mailing list
Freevo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-devel


Re: [Freevo-devel] Panorama skin WAS: Re: Interesting failure

2007-08-21 Thread Gorka Olaizola
On Mon, Aug 20, 2007 at 09:12:58PM +0200, Duncan Webb wrote:
> 
> MPlayer is a bit poor with videos that are 16x9 it seems to shrink them
> vertically so the video has nice black borders.
>
I have a lot of videos that display as you say but they are corrected if
I force its aspect ratio with "-aspect 16:9", "-aspect 1.85:1" etc...
My "monitoraspect" variable is set to 16:9 always.

I made a plugin that adds a submenu entry for changing it before playing
the movie but didn't submit it because I suppose a FXD will work too and
other people binds the aspect ratio to some buttons on the remote.


signature.asc
Description: Digital signature
-
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/___
Freevo-devel mailing list
Freevo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-devel


Re: [Freevo-devel] freevo 1.4 svn r9831 : doesn't work at all on my testbed

2007-08-21 Thread adam
> Pascal Schirrmann schrieb:
>> once, starting the record server (and one other service) :
>> 2007-08-20 22:29:08,767 INFO new process watcher instance
>> Traceback (most recent call last):
>>   File
>> "/usr/lib/python2.5/site-packages/freevo/helpers/recordserver.py", line
>> 60, in 
>> from video.commdetectclient import initCommDetectJob
>>   File "/usr/lib/python2.5/site-packages/freevo/video/__init__.py", line
>> 42, in 
>> from videoitem import VideoItem, FileInformation
>>   File "/usr/lib/python2.5/site-packages/freevo/video/videoitem.py",
>> line 720, in 
>> skin_object.register('tvguideinfo', ('screen', 'info',
>> 'scrollabletext', 'plugin'))
>>
>
> Should be fixed with this patch:
> [ freevo-Bugs-1778130 ]
[snip]

Tanja, I wonder whether a better approach would be to factor the
scrollable full description stuff off into a separate file/class for use
by all (tvguide/recordings manager/video items/music etc) components or
perhaps add a new layout for video items that includes cover art?

Cheers

Adam



-
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
___
Freevo-devel mailing list
Freevo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-devel


Re: [Freevo-devel] Gah, 1.7.3's record server still broken!

2007-08-21 Thread Andrew Flegg
On 8/21/07, Duncan Webb <[EMAIL PROTECTED]> wrote:
> > Was this the first time using conflict detection by chance?
[snip]
>
> I don't think so as this is really an old old bug. It happened to me a
> couple of times when I started using freevo.

Indeed, I'd also seen it before enabling conflict resolution and this
wasn't the first time adding/editing a new favourite since enabling
it.

> My guess is that is most likely caused by two processes updating the
> record_schedule.xml at the same time. For example, when the recordserver
> is doing an update and the webserver is adding a favourite.

Interesting. Unfortunately, there was no web access happening at the
time and the TV updates happen at 07:16.

In other words, the only thing accessing the record server at the time
was the main Freevo UI on the TV.

> From Andrews bug report #126 you can see:
> 2007-08-20 16:19:40,579 WARNING  recordserver.py (268) :
> "/var/cache/freevo/record_schedule.xml" is invalid, removed
> which means that the record_schedule.xml was corrupt and had to be
> deleted. As the record_schedule.xml also contains the favourites this
> means that they where also deleted. Ouch.

:-(

> Question for Andrew, is was there something recording at the same time as
> he was using web interface? 16:18 is a good time to record children's
> programmes.

Nope, nothing recording at the time either.

I've now got a little cronjob running as the PVR user:

8<
0 * *   * *  perl -e '$a =
"/var/cache/freevo/record_schedule.xml"; system("cp $a ~/") if -s $a >
10240'
>8

So I feel safer doing some more tests ;-)

The strangest thing is why the web UI would work and the main UI
wouldn't. Aren't they both talking to the recordserver using the same
protocol and very similar code?

Cheers,

Andrew

-- 
Andrew Flegg -- mailto:[EMAIL PROTECTED]  |  http://www.bleb.org/

-
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >>  http://get.splunk.com/
___
Freevo-devel mailing list
Freevo-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/freevo-devel