MeGUI development

[brogan]

buy a BD...:goodpost::thanks:


That's an adorable post, thanks! If something is AVAILABLE on BD, I buy it...if it isn't, I make it...thanks for your help, though...

:rolleyes::rolleyes::rolleyes:

[rack04]

That's an adorable post, thanks! If something is AVAILABLE on BD, I buy it...if it isn't, I make it...thanks for your help, though...

:rolleyes::rolleyes::rolleyes:

Well how about this reply? Re-encode without cropping.

[brogan]

Well how about this reply? Re-encode without cropping.

That's what I was trying to find out how to do. If I knew how to do that, I wouldn't have asked. I have several files that were either 1920x800 or 1920x808 & I would just make them 1920x1080, in Blu-ray Structure & burn them...but they looked totally stretched...my understanding (I could totally be wrong) was that even if I left the files at their native resolution, when I choose "Blu-ray" as output, they would automatically be resized to 1920x1080...that's why I wanted to know if there was a way in MeGUI to do the "AddBorders" command when creating the script so the video would still be 1920x800 & add the black borders so they would be letterboxed..AddBorders(0, 140, 0, 140) for example...if a movie is in a 2.35 AR, I would like to keep it that way so it doesn't get all mucked up when converting it...

[Sharktooth]

you dont get it. re-encode from the original without cropping.
if you dont have the original discs then your request is a forum rule 6 violation.
also your posts are off-topic. this is the megui DEVELOPMENT thread. for troubleshoting and other issues or any other help, there are other threads.

[LeXXuz]

there is a bug in the SAR detection. please feed it manualli to x264 with the --sar option (custom commandline options in the codec config window) until we find and fix the bug.

What about just cropping down the number of digits to 4 or 5 of the calculated sar values before passing them to the encoder? Still should be accurate enough and would solve the problem.

It's just a simple idea ofcourse. :D

[Kurtnoise]

I don't know why number digits would be the problem but I cannot reproduce here on my side.

So, could you try to describe more precisely what you do (+ what kind of source, pal or ntsc, etc...) ?

[Sharktooth]

i had such a source once... but it was a rented DVD i no longer have.

[Jawed]

I don't know why number digits would be the problem but I cannot reproduce here on my side.
x264 can't cope with a 12-digit numerator and 12-digit denominator - have you checked what the acceptable size is for x264?

When given these over-refined SARs, x264 produces a video file with a SAR of 1:1. The encode is fine, in my case, just the SAR is wrong.

If I progressively reduce the significant digits in the fraction sent to x264 (in order to find when the SAR is correct), I eventually get to 10 digits, and an encode with the correct SAR:

"C:\Program Files\megui\tools\x264\x264"
--crf 16 --ref 5 --mixed-refs --no-fast-pskip --bframes 3 --b-pyramid --weightb
--subme 6 --analyse p8x8,b8x8,i4x4,i8x8 --8x8dct --me umh --merange 32 --thread
s auto --thread-input --sar 1954337065:1374390942 --progress --no-dct-decimate -
-no-psnr --no-ssim --output "Test.mp4" "Test.avs"
avis [info]: 704x424 @ 25.00 fps (101 frames)
x264 [warning]: width or height not divisible by 16 (704x424), compression will
suffer.
x264 [info]: using SAR=59641/41943
x264 [info]: using cpu capabilities: MMX2 SSE2Slow
mp4 [info]: initial delay 2 (scale 25)
x264 [info]: slice I:1 Avg QP:10.71 size: 64070
x264 [info]: slice P:67 Avg QP:13.53 size: 41650
x264 [info]: slice B:33 Avg QP:16.23 size: 23646
x264 [info]: consecutive B-frames: 34.0% 66.0% 0.0% 0.0%
x264 [info]: mb I I16..4: 18.8% 43.4% 37.9%
x264 [info]: mb P I16..4: 3.9% 17.7% 11.9% P16..4: 21.9% 26.1% 18.5% 0.0% 0
.0% skip: 0.1%
x264 [info]: mb B I16..4: 32.5% 0.0% 0.0% B16..8: 26.1% 7.7% 11.1% direct:
20.5% skip: 2.1% L0:35.3% L1:20.9% BI:43.8%
x264 [info]: 8x8 transform intra:36.0% inter:20.1%
x264 [info]: ref P L0 56.8% 19.4% 11.8% 6.6% 5.4%
x264 [info]: ref B L0 67.0% 18.2% 8.9% 5.8%
x264 [info]: kb/s:7197.9

encoded 101 frames, 3.70 fps, 7199.06 kb/s

I did this test from a command line, I didn't use meGUI.

Notice the info line "using SAR="? This line does not show when the SAR fraction has too many digits. I guess x264 will only accept numerator and denominator smaller than 2147483648 (2^31). 9 significant digits seems like a safe maximum (i.e. 1e-8 in the code).

So, could you try to describe more precisely what you do (+ what kind of source, pal or ntsc, etc...) ?
25fps PAL in my case, the rest of the info is here:

http://forum.doom9.org/showthread.php?p=1193855#post1193855

I hand code the Avisynth (copy+paste is my friend), not using any of MeGUI's cropping/resizing features - I merely use meGUI as a task manager. I set MeGUI_darx by calculation with MeGUI_dary always set to 1000, unless it's a regular case such as 16:9, and use a simple calculation for X based upon the cropping, the original frame dimensions and the original DAR.

So that's why this particular clip has MeGUI_darx=2361 and MeGUI_dary=1000 and the SAR is 1.421965909090909090909090909...

Jawed

[Kurtnoise]

upload a sample of your source please...

[LeXXuz]

x264 can't cope with a 12-digit numerator and 12-digit denominator - have you checked what the acceptable size is for x264?

When given these over-refined SARs, x264 produces a video file with a SAR of 1:1. The encode is fine, in my case, just the SAR is wrong.


I can confirm this. At the moment I'm backing up most parts of my DVD-collection (PAL) to h264 for use with my Tvix box. Almost every third source ends up with those long numerators/denominators. Mostly 21:9 sources get this strange error.

At the moment I start the encode, watch the log file and If I see those long numbers, I simply write down the first 4 or 5 digits, cancel the encode and restart it with manual --sar setting from the numbers of the logfile.

[Kurtnoise]

http://forum.doom9.org/showthread.php?p=1196265#post1196265

[Sharktooth]

just forgot to post the latest changelogs:
0.3.0.3003
- (kurtnoise) [AviSynthWindow] fix a small issue with avs profiles (#2152972). Patch by ateeq.
- (kurtnoise) [FilmCutter] replace framerate value by AudioRate(). Fix issue #2152576
- (kurtnoise) [x264] remove fourCC component.
- (kurtnoise) ensure that "," are replaced by "." in the custom lines for x264 & xvid.
- (kurtnoise) [ProgressWindow] change the priority focus to avoid bad surprises...:>

0.3.0.3002
- (kurtnoise) [VobinputWindow] add ThreadSafeRun() to fix the problem of ASW dissapearing as soon as it appears on the system. Patch by ateeq (#2141406)
- (kurtnoise) [VobinputWindow] better rbtracks_CheckedChanged() event. Patch by ateeq (#2141406)
- (kurtnoise) [VideoUtil] remove getSourceInfo() entries to switch using Stream Information DVDDecrypter file.
- (kurtnoise) [OneClickWindow] some clean up.
- (kurtnoise) [VideoUtil] fix a if statement which has been omitted in rev 757.
- (kurtnoise) [MainForm] fix openOneClickFile(). Patch by ateeq (#2141406).
- (kurtnoise) [VideoEncodingComponent & AudioEncodingTab] fix instance Profiles to allow VS2008 designer to load the MainForm. Patch by ateeq (#2141406)
- (kurtnoise) [OneClickWindow] change accessibility for openVideo(). Patch by ateeq. (#2141406)
- (kurtnoise) [OneClickWindow] use DefaultOutputDir for matching the workingDirectory. Patch by ateeq (#2144486)
- (kurtnoise) [VobinputWindow] few tunings in the ifoFile creation. Patch by ateeq (#2144486).
- (kurtnoise) [VobinputWindow] move projectName.Text & add projectPath to the openVideo(). Patch by ateeq (#2144486)
- (kurtnoise) [VobinputWindow] add a checking for projectName. Patch by ateeq (#2144486)
- (kurtnoise) [PrettyFormatting] fix weird results when the source file has the letters vts or video anywhere in the file name. Patch by ateeq (#2144486)
- (kurtnoise) [AviSynthWindow] include an output FileBar. Patch by ateeq (#2144486)
- (kurtnoise) [VideoUtil] remove empty lines at the end.
- (kurtnoise) [MainForm] use null as file extension for openD2VCreatorFile(). Patch by ateeq (#2144486)
- (kurtnoise) [SettingsForm] add Default Output Directory components. Patch by ateeq (#2144486)
- (kurtnoise) [FileBar] add a checking for the Filename. Patch by ateeq (#2144486)
- (kurtnoise) [AudioEncodingTab] use DefaultOutputDir to test the projectPath. Patch by ateeq (#2144486)
- (kurtnoise) [VideoEncodingComponent] use now DefaultOutputDir to test the filePath. Patch by ateeq (#2144486)
- (kurtnoise) [MeGUISettings] add defaultOutputDir as string and its getter/setter. Patch by ateeq (#2144486)
- (kurtnoise) [UpdateWindow] add Proxy support. Patch by gigibop (#2142498)
- (kurtnoise) [MeGUISettings] add getters/setters & default values for the proxy support. Patch by gigibop (#2142498)
- (kurtnoise) [SettingsForm] add autoUpdate Proxy components. Patch by gigibop (#2142498)
- (kurtnoise) [ProgressWindow] fix a priority setting on Vista. Patch by griffore (#2145806)
- (kurtnoise) [x264ConfigurationPanel] few refactoring.

0.3.0.3001
- (kurtnoise) [VideoUtil] remove one iterator.
- (kurtnoise) [x264] update subME items.
- (kurtnoise) [x264] remove bRDO code.
- (kurtnoise) [x264] remove biME code.
- (kurtnoise) [OnClickWindow.Designer] add openOnQueue checkbox. Patch by ateeq.
- (kurtnoise) [OneClickWindow] update to the last changes. Mainly from the patch provided by ateeq.
- (kurtnoise) [FileBar] add performClick() function. Patch by ateeq.
- (kurtnoise) [VobinputWindow] add selectStat as boolean to reflect changes in the radioboxes CheckedChanged events. Patch by ateeq.
- (kurtnoise) [VobinputWindow] use "]" as delimiter for the 2nd argument. Patch by ateeq.
- (kurtnoise) [VobinputWindow] add getSourceMediaInfo() from vUtil & reconstruct ifofile creation into openVideo(). Patch by ateeq.
- (kurtnoise) [VobinputWindow] clear AudioTracks items for all new files, not only VOBs. Patch by ateeq.
- (kurtnoise) [VobinputWindow] add *.m2p as file extension. Patch by ateeq.
- (kurtnoise) [d2vReader] remove duplicated code. Patch by ateeq.
- (kurtnoise) [AviSynthWindow] suggestResolution checked by default. Patch by ateeq.
- (kurtnoise) [AviSynthWindow] return a DialogResult now for gotoD2vCreator() instead of a void & update openVideoSource(). Patch by ateeq.
- (kurtnoise) propagate the last changes to getAllDemuxedAudio() used elsewhere in the code.
- (kurtnoise) [VideoUtil] add audioTrackIDs as List in getAllDemuxAudio(). Patch by ateeq.
- (kurtnoise) [VideoUtil] add getSourceMediaInfo() to the openVideoSource(). Patch by ateeq.
- (kurtnoise) [VideoUtil] remove unused code.
- (kurtnoise) [VideoUtil] modify trackID & Type in the getDGindexLogInfo() function. Patch by ateeq.
- (kurtnoise) [VideoUtil] add SamplingRate as getter/setter & getSourceMediaInfo() function to retrieve some infos via the MediaInfo library. Patch by ateeq.
- (kurtnoise) [PrettyFormatting] use IsNullOrEmpty function & add '\' as delimiter. Patch by ateeq.
- (kurtnoise) [IFOparser] add a test to check whether or not the length of vobFiles is equal to 0. Patch by ateeq.
- (kurtnoise) [VideoPlayer] fix a typo in the VideoPlayer Class summary. Patch by ateeq.
- (kurtnoise) [MainForm] add openAudioFile() to the other video FileType. Patch by ateeq.
- (kurtnoise) [FileBar] add FilterIndex in the openButton_Click() - Patch by ateeq.
- (kurtnoise) [FileBar] add FilterIndex as getter/setter. Patch by ateeq.
- (kurtnoise) [AviSynthAudioEncoder] force the 2nd portion to be long. Patch by ateeq.
- (kurtnoise) [AviSynthAudioEncoder] move AVI sources in the non DShow part. Patch by ateeq.
- (kurtnoise) [ProfileExporter] added a tooltip to the list.
- (kurtnoise) [ProfileImporter] use the ContextHelp stuff for the tooltip like in the x264 config.
- (kurtnoise) [ProfileImporter] put SetToolTips method in the Shown Event instead of the Load one. Presets Importation failed otherwise...

0.3.0.2018
- (kurtnoise) added DTS MA & DTS HD as file extension for audio inputs. updated RaWavSource config according to the last library default settings. Both require NicAudio 2.02 or higher...
- (kurtnoise) [x264Encoder] cosmetics.
- (kurtnoise) added a toolTip in the Presets Importer to inform people that we can check all presets via the right click button on the list.
- (kurtnoise) [VobSubber + D2vCreator] Now, we use the ifo file selected; not necessary the one from the main movie.
- (kurtnoise) [x264Encoder] updated GetFrameString() to have the correct parsing for the last builds (#2132041).

[Jawed]

upload a sample of your source please...
Here you go:

global MeGUI_darx = 2361
global MeGUI_dary = 1000
blankclip(length=100,width=704,height=424,pixel_type="YV12",fps=25)

that's an Avisynth script that generates this problem with meGUI 3.0.3002. It encodes correctly in meGUI 2.6.1046.

Jawed

[hydro]

I couldn't find how to remove only completed jobs from the queue. If I haven't missed the obvious here's one way to do it if anyone else is interested:
http://pastebin.com/m71864880

NicMPG123Source creates .d2a files when processing .mp2 files, which get left behind (I originally thought it was DGIndex but it's not), here's a small fix to take care of the problem:
(I didn't apply it to all the mpg formats as it doesn't get created on .mp3 although a .mpa could contain an mp2 stream?)
http://pastebin.com/m112b3630

ps. please let me know if you want any of these posted to Sourceforge.

[Kurtnoise]

looks good to me...:) I'll apply them tomorrow if no one object.


If you (hydro) and Greif want to join us in the team, you have my vote ;) Give your nicks from SF website to Sharktooth...he'll give you the keys.

[Sharktooth]

Welcome on board Greif

[Greif]

Thanks

I'll do my best... :)

[Sharktooth]

i gave you SVN write access too.

[Sharktooth]

I couldn't find how to remove only completed jobs from the queue. If I haven't missed the obvious here's one way to do it if anyone else is interested:
http://pastebin.com/m71864880

NicMPG123Source creates .d2a files when processing .mp2 files, which get left behind (I originally thought it was DGIndex but it's not), here's a small fix to take care of the problem:
(I didn't apply it to all the mpg formats as it doesn't get created on .mp3 although a .mpa could contain an mp2 stream?)
http://pastebin.com/m112b3630

ps. please let me know if you want any of these posted to Sourceforge.
.mpa could contain an mp2 stream, but i dont think it's such as common. we'll see if anyone reports a bug or something.
patches seem good to me.

[tebasuna51]

NicMPG123Source creates .d2a files when processing .mp2 files, which get left behind (I originally thought it was DGIndex but it's not), here's a small fix to take care of the problem:
(I didn't apply it to all the mpg formats as it doesn't get created on .mp3 although a .mpa could contain an mp2 stream?)
http://pastebin.com/m112b3630

NicMPG123Source always create the .d2a file, then the patch must be for mpa, mpg, mp2 and mp3.

Maybe in next version I make optional create the file.
If the file exist AviSynth don't need scan all the audio file to know the sample count when open a second time the same file.

[Greif]

I'd like to define a MeGUI conditional compilation symbol. That way we can use preprocessor directives, such as:

#if MeGUI
// in the MeGUI project
#else
// outside the MeGUI project
#endif

I would to provide the HD-DVD/Blu-Ray tool as a MeGUI tool window but also as an external EXE.

Pretty much the only place I would need to use them would be on form load/close and on job queue.

Let me know what you think.

[Kurtnoise]

@Greif: welcome...:) and I asked you something about that on devs forum on SF.

@Hydro: I committed the first one (jobControl).

@Sharktooth: I think it's time to upgrade the package on SF. Now, megui is more stable than before...

@Check : I'm ready to move to the trac system now...:p

[ggf31416]

Is this intentional?

CommandlineJobProcessor.cs
public void changePriority(ProcessPriority priority)
{
[...]
case ProcessPriority.HIGH:
proc.PriorityClass = ProcessPriorityClass.RealTime;
break;


If it's intentional, could you add a confirmation dialog when the priority is set to HIGH (for OS other than Vista, as Vista already have one)

[Kurtnoise]

you mean high priority ?

well, no...it's not intentional but I can add the confirmation dialog as well.

[ggf31416]

you mean high priority ?


I mean if setting PriorityClass to RealTime when priority is HIGH is intentional or not.

[Sharktooth]

@Kurtnoise: done

[check]

@Check : I'm ready to move to the trac system now...:p
Bah! If you had mentioned this 15 days earlier :P But it's OK, assuming the other main devs (sharktooth) approves also I'll set it up either this weekend or next.

BTW, about priority, why even have realtime as an option? I think even High should not be there, if people want it they can use taskmgr.exe :p

[Sharktooth]

RealTime hangs other processes. IMHO it shouldnt be an option. High should be more than enough...

[hydro]

NicMPG123Source always create the .d2a file, then the patch must be for mpa, mpg, mp2 and mp3.

Maybe in next version I make optional create the file.
If the file exist AviSynth don't need scan all the audio file to know the sample count when open a second time the same file.

I thought I'd pointed it at an mp3 and didn't notice a .d2a but just tried it again and yes it's there.

Is it supposed to work with .mpg files as it just hangs on my system with no output, in MPC hangs at "opening..."?

I can see the usefulness in AviSynth to avoid scanning large audio files but do you think processing an audio file multiple times is a frequent occurence in MeGUI, I din't notice anything that does 2-pass audio encoding and AVS cutting doesn't open the audio file either?

Much simpler change for all Nic123MPG sources: http://pastebin.com/m22dff47

@Kurt: SF nick is ateeq, although I'm not a developer so once I fix the things that feel broken to me in MeGUI I'll probably lose interest...

[Sharktooth]

hydro, all megui devs code it in their free time. so if you feel you can lose interest, well... that's not a big problem. even occasional patches or discussion about features and bugs are welcome and i gladly accept your work even if you do it only occasionally or when you have time.
that said, the kurtnoise offer is still valid ;)

[Sharktooth]

welcome on board hydro :)

[Sharktooth]

0.3.0.3005
- (ateeq) Removed filename passed to new VobinputWindow as SetConfig passes it anyway. This way OpenVideo isn't called twice.
- (ateeq) clear queue without prompting if all jobs are done
- (ateeq) oneclick fixes
- (ateeq) mod16 undercrop for anamorphic encoding
- (ateeq) delay setting status to done till after postprocessing
- (kurtnoise) [ProgressWindow] trigger the high priority warning for all Windows OS, not only Vista.

0.3.0.3004
- (kurtnoise) [FilmCutter] AudioRate(__film) instead of AudioRate(__just_audio) for else case.
- (kurtnoise) [AMGMuxer] Splitting feature works fine now. Fix issue #2145143.
- (kurtnoise) [JobControl] allow to remove only completed jobs from the queue. Patch by ateeq.
- (kurtnoise) [DAR] use rounding value for SAR calculation. Should fixed issues #2010131 & #2013287.
- (kurtnoise) allow name for Video Tracks in the AdaptiveMuxer & fix several cosmetics spaces. Fix #2063188
- (kurtnoise) [PrettyFormatting] fix issue #2164193 about the wrong length for output filename in the oneClick encoder. Patch by ateeq.

[Greif]

0.3.0.3005
- (ateeq) mod16 undercrop for anamorphic encoding

Can someone explain this one?

[Nightshiver]

What's to explain? There was a mod16 overcrop, now there's a mod16 undercrop. What do you think it does?

[Greif]

I wasn't aware of this feature... had to look deeper to find it.

[Sharktooth]

from what i see there is now an option to Undercrop instead of overcrop or resize.
Undercropping means the cropping function will not remove some lines to comply with mod16 but it will leave some small black borders around the picture.
over/undercrop features can be automated (using a 8 pixel bias to decide where to undercrop or undercrop)... one day ill do it...

[[ReX]]

mkvmerge 2.4.0 is out for some time (11 Oct 2008). :)

[check]

@Check : I'm ready to move to the trac system now...:p

I'll be setting this up before Sunday (once I convince the trac install to coexist with my existing python environment).

[Sharktooth]

there is a problem... where do we host the megui builds after switching to trac?

[check]

there is a problem... where do we host the megui builds after switching to trac?
We already have two autoupdate mirrors I'm sure can help out here :) I reckon if you poke jarod you could convince him to run another autoupdate mirror too.

[Sharktooth]

0.3.0.3006
- (ateeq) [DAR] Alternative SAR fraction calculations for slightly better accuracy
- (ateeq) (AdaptiveMuxer) Allow splitting to same container with Adaptive Muxer
- (ateeq) [AdaptiveMuxWindow] Fix to work with previous baseMuxWindow changes
- (ateeq) [AMGMuxer.cs] File Splitting fix and don't number unless splitting
- (ateeq) [baseMuxWindow.cs] Add DefaultOutputDir and mainForm visibility change
- (ateeq) [MuxWindow.cs] Remove duplication of mainForm between MuxWindow and baseMuxWindow

[hajj_3]

feature request: Ability to show % error of aspect ratio on the "avisynth script creator" window. Gordian knot has this and i miss that when doing my x264 encodes.

I've done a screen what it would look like, highlighted in blue is the new bits:

http://i38.tinypic.com/eu0t35.jpg

[RyaNJ]

Question for any devs out there. Any plans to add a Spline64 resize option? My understanding is that AviSynth already has the Spline64Resize filter correct?

[Avenger007]

@ hajj_3 and RyaNJ
Submit it on the feature requests tracker (http://sourceforge.net/tracker/?group_id=156112&atid=798479).

@Sharktooth, update MeGUI to use DGMPGDec 1.5.3 Final (http://forum.doom9.org/showthread.php?t=129510).

[RyaNJ]

@ hajj_3 and RyaNJ
Submit it on the feature requests tracker (http://sourceforge.net/tracker/?group_id=156112&atid=798479)

Already done. Just wanted to know how simple it would be to do here :)

[hajj_3]

added request to sourceforge, thanks.

[Sharktooth]

@Sharktooth, update MeGUI to use DGMPGDec 1.5.3 Final (http://forum.doom9.org/showthread.php?t=129510).
when my stupid DSL stops acting like a 14.4kbps line...
Pinging forum.doom9.org [213.113.149.94] with 32 bytes of data:

Reply from 213.113.149.94: bytes=32 time=520ms TTL=46
Reply from 213.113.149.94: bytes=32 time=674ms TTL=46
Reply from 213.113.149.94: bytes=32 time=619ms TTL=46
Reply from 213.113.149.94: bytes=32 time=567ms TTL=46
...
... up/down speed is also ridiculous ...

[Avenger007]

Make sure the modem isn't near any EMI. Also, make sure it's not getting too hot.

[Sharktooth]

it's the ISP that's having problems with the aggregator.

[check]

I'd like to move the autoupgrade url for my megui binary mirror to a new hostname. Will the autoupdater follow a HTTP 301/302 redirect without breaking in current versions of MeGUI?