Log in

View Full Version : PgcEdit 0.6.3.1


Pages : 1 [2] 3 4 5 6 7 8 9

r0lZ
11th October 2005, 23:05
No, I don't think so. Seems this one is a bug in my code. Will verify.
Thanks

blutach
12th October 2005, 07:53
No, thank you for being so diligent in answering all our bug reports while still attending to feature requests :)

Regards

CoNS
14th October 2005, 12:55
r0lZ, in the new version, the warning about ImgBurn at startup is a bit annoying.

It's ok that it pops up the very first time, or every time you'd enter the ISO creation part of PgcEdit without having specified the path for ImgBurn, but for users like me who doesn't use PgcEdit for the ISO part at all, and thus naturally does not use either DVDDecrypter or ImgBurn in connection with PgcEdit, it's annoying that it pops up every time.

Also, I've edited two DVDs with the new version, ripped straight off the original disc, and with both of them PgcEdit has given me the warning upon load that the number of subpics streams in 20 of the menu domains didn't match the number of streams actually used. Weird, that I'd get that warning on two discs in a row, whereas I've never seen that warning before? In both cases I clicked "Fix", and then confirmed 20 times that PgcEdit could fix it in each VTSM domain. Could you add a "Fix All" button to that dialog, and do I need to worry about getting the warning?

r0lZ
14th October 2005, 13:15
You can stop the warning about ImgBurn simply by saving the burn preferences. Just go to "Options -> Input/Output -> Burn/Create ISO Setup" and click the Save button.

No, I can't add a Yes to All button in the Fix Streams dialog, because it's a standard Yes/No/Cancel button that cannot be modified, at least with Tcl/Tk.
If you are annoyed by these requests each time you load a DVD, you may turn this feature OFF in "Options -> Input/Output".
Tech note: in some DVDs, there is a subpic stream defined in the table for each menu, even if the menu is made of dummies only. It doesn't hurt, but it's cleaner to remove these unnecessary definitions. I have added the option to launch automatically this check when a new DVD is loaded at the request of several users. As I said, you can turn it off.

CoNS
14th October 2005, 15:08
Thanks for your help! I got rid of the ImgBurn warning, but I kept the automatic launch of the "fix number of streams" macro for now. Is it a new thing that it automatically launches by default at startup? That could explain why I haven't noticed that warning until now.

zacoz
14th October 2005, 15:14
Added to the betas for 0.6.1 (not sure which one it initally appeared in however)

From the 0.6.1 final change history

The "Fix number of streams" macro is now launched automatically when a DVD is opened (but not when the DVD is reopened). If some discrepancies are found, the user is prompted to fix them. An option to disable this feature has been added in the Options menu.

blutach
14th October 2005, 15:16
Options - Input/Output in fact (naming is a bit strange for this set of Options, IMHO).

Regards

Z-95
20th October 2005, 04:15
Hey, wanted to say what an excellent program you have here; it is extremely powerful and simple to use IMO. I do have one feature request: is it possible to have a way to blank multiple cells at once? It can get tiring to have around 10 cells that need blanking that require you to open, click, blank, click ok 4 times, repeat. Or maybe I'm just missing the simple way to do it. If you could that would be great!

Video Dude
20th October 2005, 05:39
I have had this problem several times. I don't understand why. :confused: But if you save the DVD and quit PgcEdit, then open it again, it will probably work.I have also experienced this many times.

There are some instances where PGCEdit will remove the cell, but not reflect the changes in the GUI. For example, on a DVD project I wanted to remove 1 cell out of 10. It did not work, so I clicked the button 4 more times, still nothing. I closed (and saved) and when I reopened I found that five cells had been removed. So it did work, but did not show the changes in the GUI. (Thanks for automatically creating a backup. :D )

I've noticed a pattern for when it occurs for me. It seems only to happen (but not every time) after I cut and paste commands and also edit the "current domain streams attributes". Everything is fine if I remove the cells first.

r0lZ
20th October 2005, 09:27
Hey, wanted to say what an excellent program you have here; it is extremely powerful and simple to use IMO. I do have one feature request: is it possible to have a way to blank multiple cells at once? It can get tiring to have around 10 cells that need blanking that require you to open, click, blank, click ok 4 times, repeat. Or maybe I'm just missing the simple way to do it. If you could that would be great!Thanks, and welcome to the forum, Z-95.

You can delete several cells at once if they are contiguous. Just open the PGC Editor -> Remove Cells, and select Remove Cell Range. This option is relatively new, so be sure to use the latest v0.6.1.
I have also the intention to revamp completely the PGC Editor, to allow moltiple selection of cells. But it's a big work, and I have also many other things to do. Maybe this winter...

r0lZ
20th October 2005, 09:31
I have also experienced this many times.

There are some instances where PGCEdit will remove the cell, but not reflect the changes in the GUI. For example, on a DVD project I wanted to remove 1 cell out of 10. It did not work, so I clicked the button 4 more times, still nothing. I closed (and saved) and when I reopened I found that five cells had been removed. So it did work, but did not show the changes in the GUI. (Thanks for automatically creating a backup. :D )

I've noticed a pattern for when it occurs for me. It seems only to happen (but not every time) after I cut and paste commands and also edit the "current domain streams attributes". Everything is fine if I remove the cells first.Thanks for the tip. Will try to find the bug, though it's difficult when it occur only from time to time.

blutach
20th October 2005, 11:55
Yeah, it is infuriating. Right now, I am in the habit of doing Incremental backup before cell operations - especially add blank cells.

(WooHoo - 1,500 totally unhelpful and useless posts!)

Regards

zacoz
20th October 2005, 13:24
Nah, think you still have a few to go Blu, as I seem to recall at least half a dozen that were actually helpful to me.

Sorry for OT.

Z-95
21st October 2005, 02:51
You can delete several cells at once if they are contiguous. Just open the PGC Editor -> Remove Cells, and select Remove Cell Range. This option is relatively new, so be sure to use the latest v0.6.1.
I have also the intention to revamp completely the PGC Editor, to allow moltiple selection of cells. But it's a big work, and I have also many other things to do. Maybe this winter...
By using the Remove Cells function, it leaves the actual video content on the DVD correct? When I first used PCGedit I used remove cell and it left the unreferenced content in the vobs and I had to run it through VobBlanker to get rid of it. I use blank cell to get rid of the video and save space. Cool, no real rush, its fine now as long as I don't run into another 10+ cell PCG :p .

r0lZ
21st October 2005, 08:40
Right. You need VobBlanker to recover the space used by the VOB cells.

However, it is possible to blank a whole domain at once in PgcEdit. See Domain -> Blank Out all PGCs in Domain. It patches the IFOs, and replace the VOB(s) by one single black frame. In the menu domain, it is even possible to remove completely the VOB, by converting all PGCs to dummies.
If you don't need to keep anything in a domain, I recommand to use this option. It's faster, and it recovers even more space than with VobBlanker.

r0lZ
21st October 2005, 12:53
I noted that a log was displayed after the Create ISO process which also had the LB sector number included and seemed to use the same place as the create ISO when saved. Could you have the default log name set to be same as the ISO name with a suffix _log.txt to link them together for later burning reference?
Done. Thanks for the suggestion.

r0lZ
21st October 2005, 13:41
I will check the ~folder problem. Maybe it works only for files.
Well, I forgot I have removed the automatic exclusion of ~* files when creating the ISO. It causes problems under Linux, because ~ is expanded as the current user's home path. Anyway, your trick to move your work folders in the PgcEdit backup folder works to exclude them. So, I will not implement another solution.

r0lZ
21st October 2005, 13:58
Similar problem adding cells if you say no to updating the VOB in the backup folder. It appends 10k to the VOB but does not update the cell table. Next time you do it, you get an error of course, since the VTS is too big :(
I've tried to reproduce this bug, but I can't. Seems everything is normal.
When you don't backup the VOBs, the procedure continues as expected. If you use the Cancel button when requested to backup the VOBs, or in the overwrite confirmation dialog, the procedure is not executed at all. In both cases, it's normal.
Could you please explain what you did exactly?

Z-95
21st October 2005, 15:31
I have never seen this bug in my usage of the program, either in removing the checkmark to backup the VOB when blanking a cell or when leaving it marked and then saying no to overwriting the already backed up VOB. Each time it correctly updated the cells for me. Note that I have not tried leaving it checked and click no to overwrite in 0.6.1, only in 0.6.

r0lZ
21st October 2005, 16:00
Thanks for the confirmation.
However, blutack is talking about the Create New Blank Cell feature available in the PGC Editor. There is no checkmark to backup the VOBs. It's a dialog window that is opened when needed. I suppose you are speaking of the Blank All PGCs in Domain function.
But you're right. Since both methods are similar, the same bug should occur in both functions, or never. (But the add cell function is more complex...)

Z-95
21st October 2005, 20:31
Whoops, my bad, assumed when he said "say no to updating the VOB in the backup folder" to be him referring to blanking cells and the backing up of original VOBs.

One thing I've noticed, when the fix streams macro is run when opening the DVD, it doesn't seem to fix the problems it finds. When I reopen that DVD, it comes up with the same "there are 2 discrepancies..." no matter how many times I run the macro.

r0lZ
22nd October 2005, 00:36
The fix streams macro fixes the discrepancies on the data stored in memory, but doesn't save the DVD automatically. You have to save it manually.

Z-95
22nd October 2005, 01:59
Whoa, just tried it and it works if you save it immediately, but I have run the macro and then did editing on the DVD and then saved it manually and it didn't save what the macro fixed.

blutach
22nd October 2005, 06:10
@r0lZ - all I did was try to add a few blank cells. Unofrtunately you have to do this one at a time, but so be it (maybe easier to use AddChapter for bulk adds). So, I add the forst one no probs. Saves the DVD autmatically (I think). Add another blank cell. Asks if I want to overwrite the backup VOB (I don't). Then the little 10k is added to the VOB but the cell table is not updated, so the third time around, I'm in trouble.

Regards

r0lZ
22nd October 2005, 10:27
Thanks. Will try this procedure...

r0lZ
22nd October 2005, 13:18
I have updated my homepage with links to new guides (including 2 brand new german PDFs by Tobi), and to the newest plugins (including the anti-ARccOS plugin by the Mad Monk.)

See the hemepage for details...

Rippraff
22nd October 2005, 13:51
Thanks r0lZ and a special thank you to Tobi as well! :)

Cu Rippraff

blutach
23rd October 2005, 11:32
A note on the Anti-ARccOS plugin - it is still under development and does not work for everything out there (Episodic DVDs still require a little development). But give it a try - it works on the vast majority of ARccOS disks in circulation.

Regards

E-Male
25th October 2005, 04:03
r0lZ, could you add a cli-option (or make a seperate cli-tool) that allows just patching a VIDEO_TS folder to have the 32k padding between video_ts.ifo and .bup, so it can then be used with mkisofs?
would be great to have for batch-files and front-ends
also: could you please post the correct mkisofs parameters to use after the patching?

r0lZ
25th October 2005, 06:58
Well, PgcEdit is a front-end for MkISOFS. And doing a little cli-tool is not easy. I will have to extract many things from the main program...

But there is a trick. When PgcEdit starts (with a DVD folder name passed as argument), it loads the DVD, and then loads the plugins. So, if you want, copy PgcEdit somewhere in a new directory, and create a little plugin to save automatically the DVD just after it has been loaded. Be sure to enable the 32K gap option, and enable expert mode to suppress the confirmation dialogs. If you call this copy of PgcEdit, it will load the DVD, and save it with the 32K gaps, then exit.
However, note that all setup values are saved in the same folder, regardless of which copy of PgcEdit is used. This means that you will have to leave the options above when you work with the 'normal' copy of PgcEdit. Also, PgcEdit may display some warnings, even when the expert mode is enabled.

The plugin should contain those lines:
set rc [::main::Write_DVD false] ;# save DVD, and remember error code
cleanexit $rc ;# and quit PgcEdit with error code
I have not tested the method, but it should work.

[EDIT:]
It is even possible to hardcode the setups you need in the plugin. Insert the following lines before the lines above:
set ::config(save_setups_on_exit) false
set ::config(save_BUP_with_gaps) true
set ::config(expert_mode) true
set ::config(auto_create_backup) false
set ::config(auto_fix_streams) false
set ::config(auto_launch_menubuttons_parser) false
Thanks to the first line, the setups will not be saved when the version with the plugin is used.
(I'm not sure if the last 3 lines are useful, since the DVD will be already loaded when they are executed.)

2COOL
28th October 2005, 08:06
Bug report.

Select a Title and go all the way to last sub dialog (PUO value by clicking on Input Value button) in image and just click on OK to set and exit.

http://img457.imageshack.us/img457/9961/screenhunter0302ij.gif

When doing so, the Prohibited User Operations dialog is on top and you then can successfully exit that by clicking OK. After this, you can also click on OK to exit PgcEditor dialog. Business as usual.

The problem lies if you try to do it again. Now, when you repeat the same procedure going all the way to PUO value dialog and clicking OK, the Prohibited User Operations dialog disappears too. What happens next is that you cannot close out the Pgc Editor dialog and therefore PgcEdit is locked. The only way to exit is using task manager.

r0lZ
28th October 2005, 09:16
Thanks. It's strange indeed. I don't understand why it works the first time but not after...
Note that when you experience this problem, you can bring the PUO window on top again by clicking somewhere on the PGC Editor window. No need to kill PgcEdit.

Tobii
1st November 2005, 17:02
It is very easily possible to produce an applications error with the macro "import title closing clip",if the PGC doesn't have a title number.

Example (last VTS):

VTSM 12 , LU 1 (en) , 1 (dummy) RootM
VTSM 12 , LU 1 (en) , 2 (0:01) 1b.
VTST 12 , 1 TTN 1 (3:29) Title 11
VTST 12 , 2 TTN 2 (13:47)

If one selects the last PGC and then calls the macro,it comes to the applications error.

Error: can't read ":: pgcs(12,2,Title)": no such element in array


One can use only the macro, if the PGC has a title number? r0lZ?

spyhawk
1st November 2005, 17:16
VMG Title must be defined for "VTST 12,2 TTN2". The macro just needs another check before importing.

r0lZ
1st November 2005, 17:21
Yes. It is theorically possible to have something like that:

VTST 12 , 1 TTN 1 (3:29) Title 11
VTST 12 , 2 TTN 1 (13:47)
(Note that both PGCs are related to TTN 1)

But it is not legal to have a TTN without a title number at all.

Anyway, I will fix this issue.

Thanks, Tobi and spyhawk.

radius
3rd November 2005, 01:30
Hi :)
i found a small issue with the "PGC Playback Time" fixing,
I've build a dvd with maestro then when editing the title pgc i get this Warning message:
http://r4dius.free.fr/warning.png
If i click yes then ok to leave the pgc editor window then reopen the pgc editor the "remove cells" doesn't work as it should, cells are not directly deleted (and there's no "Building cell tab..." in background), if i remove a cell then again ok to leave and reopen editor the previously "removed" cell is gone and i get another warning.
Btw i've been using the menu editor to redirect some things and i've done a lot of illegal commands before figuring it was illegal ^^, it would be great if the big button with the command was in red as in commands window when creating an illegal thing :)
finally ^^, are you thinking in adding vobs PUOps editing ?
Thanks

r0lZ
3rd November 2005, 10:13
Thanks for the bug report.

I know the Remove Cell bug, but it is somewhat occasional, and I have many difficulties to fix it. Maybe with your report I will be able to find it more easily...

For the menu editor, what did you exactly? It is theorically impossible to encode an illegal command in the command editor, but it is possible to use a command that cannot be present in a menu button. Is it what you did?
Note that it's not easy to highlight an illegal command in the menu editor, because I don't have the routine to check for the illegal commands in menu buttons. Will see what I can do...

No, I have no plans to implement VOB PUOs editing. In the PUO plugin, there is a function to clear all PUOs in the VOBs. My advice: use it to get rid of any VOB PUOs, and set the PUOs in the IFOs only. It's easier and faster. (However, with this method it is impossible to set the GoUp PUO, which, for an obscure reason, cannot be used in the PGCs.)

blutach
3rd November 2005, 11:10
Be careful in getting rid of VOB PUOps where there are angles. In many players, the little angle icon comes on (if these have been removed) and can not be turned off.

I only turn off VOB PUOps if there are no angles.

Regards

r0lZ
3rd November 2005, 12:10
But you can set the Angle PUO in the IFO, and remove it in the VOBs, no?

Maybe I have to modify the PUO plugin to allow the user to selectively remove some PUOs in the VOBs, but not all of them...
But IMO, it's the player fault. What a stupid idea to display the Angle symbol on screen! You should have at least a way to disable it. (My Sony is smart enough and don't display it!)

radius
3rd November 2005, 14:33
yeah as you said i used legal commands but not allowed on menus :)
for the puos i'm actually using maestro to mark the vobs

Sir Didymus
12th November 2005, 23:11
Big sorry if the question has been already discussed or in case I am missing something fundamental...

I think the Kill Pgc Playback Macro is one of the most appreciated by PgcEdit Users.

Noticed in a couple of titles (really... just on two examples) that it did not killed the pgc playback. The reason was that into these specific situations the navigation among pgcs, after the playback of the pgc, was not performed through post commands (linkpgcn or other flow control instructions) but by means of the "NextPgc" link.

Do you think it should be possible, in the macro, to check if the last post command in the Pgc is an instruction of unconditional branch, and in case it is not, to insert as a last instruction of the pre command section a new branch instruction (LinkPgcn) pointing to the NextPgc ?

All the best,
SD

r0lZ
12th November 2005, 23:38
It should be possible to append anyway a LinkPGCN command when the NextPGC link is not zero, and doesn't point to itself. If the PGC is exited by a post command, the LinkPGCN will never be reached. Right?

[EDIT] Just finished modifying the code. Seems adding a LinkPGCN anyway works fine.
Thanks for your bug report, Sir Didymus. :goodpost:

blutach
13th November 2005, 13:44
I'm pretty sure 2COOL reported on this and made a little guide on how to fix it for multi PGCN titles.

EDIT: Here it is - http://forum.doom9.org/showthread.php?t=90773

Regards

r0lZ
13th November 2005, 14:43
Hum, maybe it's another thing 2COOL has encountered. He doesn't say if the playback jumps from PGC 1 to PGC 2 by means of a NextPGC link, or if it jumps directly to the next PGC in the list because there are no post-commands nor NextPGC link. I don't know if it's something possible, but I'll test this scenario.

If there is a NextPGC link in PGC 1 pointing to PGC 2, my method will work: a new LinkPGCN to PGC 2 command will be added after the pre-commands of PGC 1, therefore the playback will be bypassed, and the pre-commands of PGC 2 will be executed normally.

Sir Didymus
13th November 2005, 17:21
It should be possible to append anyway a LinkPGCN command when the NextPGC link is not zero, and doesn't point to itself...
Mhhh. Very smart!!!

And what about, in case the link is 0, to add instead a LinkPGCN to the immediately following PGC ?

This should cover the situation described by 2COOL also in case no NextPGC link is present in PGC 1...

r0lZ
13th November 2005, 17:59
Yes, but I need to be sure it's legal. IMO, if there are no post commands nor NextPGC link, the DVD must simply exit. This is how it's simulated in the trace.

r0lZ
13th November 2005, 18:17
I have tried all software players I have. All are stopping after the first PGC. So, I suppose that there is a NextPGC link in 2COOL's example, and that my current method should work in all cases.
Anyway, I prefer that, or I will add false references to some PGCs. This may confuse the Cross References and Go to Calling Command functions.

Sir Didymus
13th November 2005, 18:39
Yes, but I need to be sure it's legal.
Agreed. I will check the matter on some documents...


I have tried all software players I have. All are stopping after the first PGC. So, I suppose that there is a NextPGC link in 2COOL's example, and that my current method should work in all cases...

That's a good point... anyway feeling unsafe in totally thrusting on sw players, even in case all show the same behaviour... :D


Anyway, I prefer that, or I will add false references to some PGCs. This may confuse the Cross References and Go to Calling Command functions.

Fully understood and agreed. A BIG :thanks: for all of the work you are putting into your beautiful application.

frank
17th November 2005, 14:54
Time for a new thread to PgcEdit 0.6.2, isn't it?

r0lZ
17th November 2005, 15:11
Well, if you want so. But 0.6.2 is still in beta phase. I have added a workaround for blutach's problem with the backups when burning with PgcEdit. This workaround still needs to be tested. After that test, I will release 0.6.2 final.

Anyway, if you want to test it, you may download 0.6.2 beta 1 here (http://www.videohelp.com/~r0lZ/pgcedit/beta/PgcEdit_winexe_0.6.2beta1.zip).