google-code-export / tovid

Automatically exported from code.google.com/p/tovid
1 stars 0 forks source link

problem creating chapters #160

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Hi

I've been trying out tovid svn-r3412 on 64-bit linux and have come across a 
problem with creation of chapters when using the command line.

If I run this from the command line...
tovid disc -pal -textmenu -menu-title "title" -titles "title" -files 
filename.mpg -menu-fontsize 48 -titles-fontsize 32 -out outputdisk

then the output says...
Chapters for filename.mpg are: 
00:00:00.000,00:18:01.650,00:36:03.290,00:54:04.940,01:12:06.580,01:30:08.230

but the chapters are not actually created.

whereas with this command, the chapters are created perfectly...
tovid disc -pal -no-menu -menu-title "title" -chapters 5 -titles "title" -files 
filename.mpg -menu-fontsize 48 -titles-fontsize 32 -out outputdisk

Also reading the man page, it seems to me that there is no way to specify 
chapters say every 5 minutes when using the "-textmenu" option - it would be 
great if it could as I've used this to good effect in the released versions of 
tovid.

Keep up the great work.
Cheers
Paul.

Original issue reported on code.google.com by pabloker...@gmail.com on 27 Jan 2012 at 12:46

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
I don't seem to be getting emails from google code anymore re issues, so sorry 
I missed this for so long.

I am not seeing this issue with SVN.  When I run the same command as you with 
-keep-files and look at the dvdauthor.xml, the chapters are there.  As well the 
output from dvdauthor:
[dvdauthor]:  CHAPTERS: VTS[1/1] 0.000
[dvdauthor]:  CHAPTERS: VTS[1/1] 24.120
[dvdauthor]:  CHAPTERS: VTS[1/1] 48.240
[dvdauthor]:  CHAPTERS: VTS[1/1] 72.360
[dvdauthor]:  CHAPTERS: VTS[1/1] 96.480
[dvdauthor]:  CHAPTERS: VTS[1/1] 120.600

(which matches the xml file)
for the short test file I used.

I'm not sure what the problem is but you can debug with -keep-files and look at 
the dvdauthor.xml (and the todisc.log) and let me know if you find anything.

The only way to specify fixed chapter points at present is to pass them in for 
the -chapters option as outlined in the man page.
example:
-chapters 00:00:00,00:05:00,00:10:00,00:15:00,00:20:00 etc

all the best,
grepper

Original comment by grepper@gmail.com on 9 Mar 2012 at 5:39

GoogleCodeExporter commented 9 years ago
a quick way to make the timecodes rather than typing them out:
in a terminal type:
(. /usr/local/lib/tovid/tovid-init && make_chapters 7200 5)

The output:
00:00:00,00:05:00,00:10:00,00:15:00,00:20:00,00:25:00,00:30:00,00:35:00,00:40:00
,00:45:00,00:50:00,00:55:00,01:00:00,01:05:00,01:10:00,01:15:00,01:20:00,01:25:0
0,01:30:00,01:35:00,01:40:00,01:45:00,01:50:00,01:55:00

So you could do:
timecodes==$(. /usr/local/lib/tovid/tovid-init && make_chapters 7200 5)
tovid disc -files 1.mpg -chapters $timecodes ... etc

Original comment by grepper@gmail.com on 9 Mar 2012 at 6:12

GoogleCodeExporter commented 9 years ago
Hi

I've now updated to svn-3416 - when I run this command I still don't get any 
chapters created...

/usr/local/tovid-svn-3416/bin/tovid disc -pal -textmenu -chapters 12 
-keep-files -menu-title "Apollo" -titles "Apollo 11" -files apollo.mpg 
-menu-fontsize 48 -titles-fontsize 32 -out apollo_disk

I'm attaching the todisc.log file - it looks to me as those the 12 chapters I 
specified are recognised in the "[todisc]:" section, but are later ignored in 
the "[dvdauthor]:" section.

There doesn't seem to be much in the dvdauthor.xml file, but I've attached it 
for you as you mentioned it above. Give me a shout if you other files to help 
with debugging...

Cheers
Paul.

Original comment by pabloker...@gmail.com on 22 Mar 2012 at 10:56

Attachments:

GoogleCodeExporter commented 9 years ago
I'll admit to being mystified as to why this is happening, I don't see any 
errors in your log.
When I run the same command as you here (different video of course) I get the 
attached dvdauthor.xml, which as you can see has all 12 chapters.

Does it work if you do:
-chapters \
00:00:00,00:01:15,00:02:30,00:03:45,00:05:00,00:06:15,00:07:30,00:08:45,00:10:00
,00:11:15,00:12:30,00:13:45
?

I'll prepare a todisc script that has some debugging built in so we can see 
what is happening to the variable that contains the chapters between the time 
it is echoed to screen and the time the xml is created.  I don't see where it 
could be unset looking at the code, or why the 'heredoc' where the xml is 
created doesn't expand it.

grepper

Original comment by grepper@gmail.com on 23 Mar 2012 at 5:26

Attachments:

GoogleCodeExporter commented 9 years ago
Sorry, that doesn't work either.

If I do a similar commmand using...

-nomenu -chapters 00:00:00,00:01:15,00:02:30,etc,etc

then that works perfectly. Weird!

Cheers
Paul.

Original comment by pabloker...@gmail.com on 23 Mar 2012 at 10:06

GoogleCodeExporter commented 9 years ago
Well, not so weird that -no-menu works as it doesn't use a dvdauthor.xml, it 
calls dvdauthor from the command line from a different block of code.  But it 
is weird that I can't reproduce your error and no one else has reported it.  
I'd like to find out why.

Please replace your todisc script (usually /usr/local/lib/tovid/todisc) with 
the attached todisc. (You can rename the original script.)  It will output a 
log:  /tmp/todisc.debug.

Then run tovid adding the -noask option to your commands and saving stdout to a 
file, like so:
tovid disc -no-ask ... rest of your command  > apollo.log 2>&1
Then attach /tmp/todisc.debug and apollo.log to your next comment

thanks for your patience,
grepper
tovid disc

Original comment by grepper@gmail.com on 24 Mar 2012 at 6:35

Attachments:

GoogleCodeExporter commented 9 years ago
done - files are attached.
For info, I ran the command with -chapters 12

Cheers
Paul.

Original comment by pabloker...@gmail.com on 27 Mar 2012 at 9:47

Attachments:

GoogleCodeExporter commented 9 years ago
Okay Paul, thanks.  Looks like you have an environment variable called GROUP 
which is set to "star".  The GROUP variable is tested in the code for making 
the xml, and because it is set, it branches to the section dealing with 
'grouped' videos, rather than where it should have branched to.

A quick test would be to add "unset GROUP" (no quotes) to the todisc script 
(/usr/local/lib/tovid ) or wherever you have it installed.  Add it at the top 
of the script where variables are defined, at about line 100 in the modified 
script I gave you.  The actual fix will be forthcoming, as this qualifies as a 
bug, rare as this env variable is.  It points to a deeper inadequacy in the 
script, namely that it should NOT be using upper case variables at all for this 
reason.

Hopefully that will run to completion.

On another note, there is a glaring error in the logs: "tovid, command not 
found".  I don't think this would cause any problems like we are dealing with, 
but I'm curious as to why this is happening.  Your install --prefix should have 
a $PREFIX/bin directory that is is in your PATH variable.

thanks again for testing,
grepper

Original comment by grepper@gmail.com on 28 Mar 2012 at 5:12

GoogleCodeExporter commented 9 years ago

you've fixed it :-)

adding the "unset GROUP" to line 100 of the todisc file I downloaded now works 
perfectly.

Regarding the commmand not found error, it was my mistake that I'd not changed 
the newer $PREFIX/bin to my path - I've done that now and all is fine.

Cheers and thanks once again.
Paul.

Original comment by pabloker...@gmail.com on 28 Mar 2012 at 1:50

GoogleCodeExporter commented 9 years ago
[deleted comment]
GoogleCodeExporter commented 9 years ago
Great, thanks Paul.  I've commited a fix in SVN r3417, which changes the GROUP 
array name to GROUP_ARR and will avoid conflict.  Is the GROUP var a variable 
you set yourself or does a program set it for you, and if so which program ?

I'll wait until you test the new todisc before closing this issue.

all the best,
Robert aka grepper

Original comment by grepper@gmail.com on 29 Mar 2012 at 1:28

GoogleCodeExporter commented 9 years ago
r3417 works perfectly thanks.

I've not set the GROUP variable myself per se - it comes from part of the login 
via details held in /etc/group. The system I'm on is also a nis client and 
there is also an additional group setting taking place via details in 
/etc/security/group.conf

Thinking about this, I'm surprised that you've not come across a problem before 
with using GROUP as a variable as I believe it's a global standard variable 
that is set at user login, along with USER, HOME, etc, etc.

Cheers
Paul.

Original comment by pabloker...@gmail.com on 29 Mar 2012 at 8:29