View Full Version : MeGUI development
Sharktooth
5th September 2007, 21:02
it should. however the user is warned to check if avisynth is installed AND if the path is set...
Kurtnoise
5th September 2007, 21:06
ok...great.
chickenmonger
5th September 2007, 22:38
Here's a tiny visual bug in the d2v creation window:
"Track1" and "Track 2"
http://www.imagehosting.com/out.php/t1106106_untitled.PNG (http://www.imagehosting.com/out.php/i1106106_untitled.PNG)
Here's what I hope is a patch to fix that single missing space. Let me know if I managed to do it correctly. It's my first time making a patch.
Index: VobinputWindow.cs
===================================================================
--- VobinputWindow.cs (revision 166)
+++ VobinputWindow.cs (working copy)
@@ -309,7 +309,7 @@
this.track1Label.Name = "track1Label";
this.track1Label.Size = new System.Drawing.Size(72, 23);
this.track1Label.TabIndex = 11;
- this.track1Label.Text = "Track1";
+ this.track1Label.Text = "Track 1";
//
// track2
//
Atak_Snajpera
5th September 2007, 22:53
0.2.6.1005
Commit by Sharx1976:
- Using Avisource() to open .avi files.
What If user has loseless HD avi larger than 2GB? I suggest OpenDMLSource instead of AVI...
squid_80
5th September 2007, 23:49
AVISource is just a wrapper for avifilesource and opendmlsource. It scans the avi file and uses whichever one is appropriate.
Sharktooth
6th September 2007, 02:25
Here's a tiny visual bug in the d2v creation window:
...
Done. Also added some logic to control the "Suggest resolution" control activation based on the "Resize" status.
Question for Berrinam: I see TIVTC 1.0.1 in the auto-update but it doesnt get updated (fixable)... but the point is, does megui already support version 1.0.1 (i have no interlaced content to test it and i dont wanna waste time in something you may have already did...)?
berrinam
6th September 2007, 08:55
I didn't put TIVTC 1.0.1 on the update servers, it was probably check, who kindly offered to manage and update them. I haven't done anything to change the interlacing, so that may have to be reverted to an older version which does still work with MeGUI.
Edit: Sometime, I plan to replace the auto-deinterlacing in MeGUI with my external program, bautodeint. It's a better analyser, and it will also be integrated into the job queue more nicely. (Being able to run small jobs like this without waiting for larger encodes to finish is one of my main motivations for having parallel job encoding)
check
6th September 2007, 10:59
I just updated TIVTC to the latest version a while ago. I have been using it without worries so far, but I dont use the MeGUI script creator much. Did I miss a problem with MeGUI & TIVTC 1.0.1?
Sharktooth
6th September 2007, 12:49
uhm... well, the problem may be the script creator. if TIVTC 1.0.1 has different parameters than the actual one (1.0RC6), the script created by MeGUI may not work.
Infact 1.0RC7 was reverted to 1.0RC6 for that reason until berrinam could have a look at the auto-deinterlacer and update it to support a newer TIVTC version.
Atak_Snajpera
7th September 2007, 00:14
Why MeGUI does not use Yadif deinterlacer? Yadif is faster, no artifacts and quality is comparable to TDeint
Sharktooth
7th September 2007, 02:06
well, probably it's not worth spending time on a function it is going to be replaced.
however that could be a suggestion for bautodeint... :)
Sharktooth
7th September 2007, 02:25
I just updated TIVTC to the latest version a while ago. I have been using it without worries so far, but I dont use the MeGUI script creator much. Did I miss a problem with MeGUI & TIVTC 1.0.1?
could you please try if MeGUI has problems with TIVTC 1.0.1 using the AVS creator?
Sharktooth
7th September 2007, 02:37
0.2.6.1006
Commit by Sharx1976:
- Fixed some visual interface glitches
- AVS Creator: Checking "Resize" enables "Suggest Resolution"
- AVS Creator: Checking "Suggest Resolution" automatically checks "Resize"
- (Kurtnoise) fix subtitles streams parsing when we have strings instead of numbers.
chickenmonger
7th September 2007, 05:29
The mewiki has changed slightly from what I've seen. What used to be:
http://mewiki.project357.com/wiki/Main_Page
is now
http://mewiki.project357.com/index.php/Main_Page
If this is a permanent change, the internal links in MeGUI should be updated.
Also, here's two more patches for minor visual errors:
Index: MuxProvider.cs
===================================================================
--- MuxProvider.cs (revision 170)
+++ MuxProvider.cs (working copy)
@@ -433,7 +433,7 @@
maxFilesOfType = new int[] { -1, -1, -1, 1 };
base.type = MuxerType.MKVMERGE;
generator = CommandLineGenerator.generateMkvmergeCommandline;
- name = "Mkv muxer";
+ name = "MKV Muxer";
// base.audioInputFilter = "All supported types (*.aac, *.ac3, *.dts, *.mp2, *.mp3, *.mp4, *.ogg)|*.aac;*.ac3;*.dts;*.mp2;*.mp3;*.mp4;*.ogg|RAW AAC Files (*.aac)|*.aac|AC3 Files (*.ac3)|*.ac3|DTS Files (*.dts)|*.dts" +
// "MP2 Files (*.mp2)|*.mp2|MP3 Files (*.mp3)|*.mp3|MP4 Audio Files (*.mp4)|*.mp4|Ogg Vorbis Files (*.ogg)|*.ogg";
// base.videoInputFilter = "All supported types (*.avi, *.mkv, *.mp4)|*.avi;*.mkv;*.mp4|AVI Files (*.avi)|*.avi|Matroska Files (*.mkv)|*.mkv|MP4 Files (*.mp4)|*.mp4";
Index: VobinputWindow.cs
===================================================================
--- VobinputWindow.cs (revision 170)
+++ VobinputWindow.cs (working copy)
@@ -382,7 +382,7 @@
this.projectNameLabel.Name = "projectNameLabel";
this.projectNameLabel.Size = new System.Drawing.Size(100, 13);
this.projectNameLabel.TabIndex = 3;
- this.projectNameLabel.Text = "d2v Project Ouput";
+ this.projectNameLabel.Text = "d2v Project Output";
//
// saveProjectDialog
//
Does anybody know of any current work being done on ContextHelp.xml? That's something even us non-programmers can handle. Also, the mewiki suggests that the preferred way to add a patch is through SF's patch tracker. Is this still the preferred way?
check
7th September 2007, 08:36
re: web site, please see my sig or http://forum.doom9.org/showthread.php?p=1042398#1328
Kurtnoise
7th September 2007, 09:30
@chickenmonger: done...:)
Sharktooth
7th September 2007, 13:58
0.2.6.1007
- (Sharx1976) New changelog format
- (Sharx1976) Fixed 1735676 (Video Output filename can't be changed directly)
- (Kurtnoise) fix wiki link
- (Kurtnoise) fix label typos
- (Kurtnoise) remove hint track option for NDAAC encoder + several tunings
- (Kurtnoise) few tunings for audio encoders
Atak_Snajpera
8th September 2007, 21:02
well, probably it's not worth spending time on a function it is going to be replaced.
By what? At the moment in terms of speed and quality Yadif is unbeaten.
chickenmonger
8th September 2007, 23:30
By what? At the moment in terms of speed and quality Yadif is unbeaten.
I think Sharktooth is talking about choosing to not integrate Yadif into MeGUI's current auto-deinterlacer, as there are tentative plans to replace the current auto-deinterlacer with Bautodeint instead. As Bautodeint is also AviSynth based, Yadif should be able to be integrated into it instead.
berrinam
9th September 2007, 00:16
I certainly want to add yadif to MeGUI. I already use it myself, using AVS templates in MeGUI. The automatic deinterlacing will still use the TIVTC package, because they provide filters such as TIsCombedTIVTC, but yadif should be offered as a possible filter if the source is detected as interlaced.
Sharktooth
9th September 2007, 01:49
I think Sharktooth is talking about choosing to not integrate Yadif into MeGUI's current auto-deinterlacer, as there are tentative plans to replace the current auto-deinterlacer with Bautodeint instead. As Bautodeint is also AviSynth based, Yadif should be able to be integrated into it instead.
exactly
@kurtnoise: thank you for your help with megui :)
Sharktooth
10th September 2007, 03:09
0.2.6.1009
- (Kurtnoise) fixed 1660566 (Audio Streams were discarded from command line in the Adaptive Muxer).
- (Kurtnoise) [AVS Creator] Add yadif.dll deinterlacer (also added to AutoUpdate)
- (Kurtnoise) [AVS Creator] Re-order deinterlacers, to prefer TDeint over TDeint+EEDI2, and yadif over TDeint
- (Kurtnoise) [MainForm] : some refactoring in the Help Menu.
- (Kurtnoise) added a TabControl for Program Paths.
- (Kurtnoise) updated MediaInfo.dll & MediaInfoWrapper.dll (0.7.4.4 to 0.7.5.3)
- (Kurtnoise) move Settings & Update items to Options Menu.
@kurtnoise: i saw you added yadif but yadif is an avisynth plugin... it is a nonsense to have a path for yadif.dll when we already have an avisynth plugins path.
it should be removed from the settings too since there is a special section in the upgrade.xml named:
<AviSynthFile type="tree" displayname="AviSynth plugins">
that includes ALL the avisynth plugins for the autoupdate. all files placed there will be automatically copied to the avisynth plugins directory stored in the settings.
the changes to to updatewindow.cs and settingsform.cs have be reverted and yadif.dll path removed from the settingsform.cs
Look at the upgrade.xml (http://megui.org/auto/upgrade.xml) file megui parses for the updates, it will make you understand better.
just adding:
<yadif type="file">
<filepath version="0.9">yadif09.zip</filepath>
</yadif>
in that section of the upgrade.xml file, will make megui download the yadif09.zip and unpack it in the avisynth plugins folder
so, i've built 0.2.6.1009 without fixing the yadif stuff since it's 5:00 AM and i need to sleep, but before fixing other bugs please have a look at the yadif stuff.
buzzqw
10th September 2007, 05:54
yadif should have a specific path for load since is a LoadCplugin..
in automkv (avaiable in next update) i added a cplugin folder for adding and loading all avisynth_C plugins. If cplugin folder is present and *.dll found i will load it with loadCplugin.. else skip
just my 0.02€
BHH
berrinam
10th September 2007, 06:41
Actually, I added yadif (and re-ordered the deinterlacers) -- you probably just misread the SVN log. Anyway, as buzzqw said, it's a C plugin, so it can't be put in the AVS plugins folder. This is why I put it in a MeGUI program files folder. If we will support more C plugins, then it might be worth doing what buzzqw does with automkv.
Sharktooth
10th September 2007, 12:46
Oops.... so where should yadif be added in the upgrade.xml?
EDIT: Nevermind, found it.
EDIT: There is a problem in the program paths browse buttons. they do not open the dialog.
Kurtnoise
10th September 2007, 13:17
EDIT: There is a problem in the program paths browse buttons. they do not open the dialog.
fixed...:o
bob0r
10th September 2007, 14:35
I compiled a fresh new megui, but when i run a clean install, updater keeps asking for this:
Trying server: http://megui.org/auto/stable/
Retrieving update file from server...
File downloaded successfully...
Loading update data...
Update data loaded successfully...
Finished parsing update file...
There are 1 files that can be updated.
Updating libs. File 1/1.
Error: libs could not be found on the server
Update completed.
0 files were completed successfully
1 files had problems.
Easy fix i assume...
Sharktooth
10th September 2007, 14:48
does it work now?
also where i can find the megui NSIS installer script you use?
bob0r
10th September 2007, 14:58
It works but uhm, aren't the libs i compiled newer and possibly needed??
http://x264.nl/megui.nsi
Edit:
Also why does the "new updates available" pop-up, pop up behind the main windows?
Sharktooth
10th September 2007, 15:02
Trying server: http://megui.org/auto/stable/
use the development auto-update server (look in the settings) :)
EDIT: is it ok if i add the installer script to megui svn? ... if so, who's the author?
bob0r
10th September 2007, 15:26
Uhm, you are the author?
former x264 installer script? :)
bob0r
10th September 2007, 15:30
Uhm, on the update server, isn't it a bad thing to use old LIBS for 0.2.6.1010+ ? Won't this create conflicts?
Sharktooth
10th September 2007, 15:38
LOL! ok.:)
however the libs are fine. there will be no conflict.
bob0r
10th September 2007, 15:44
Hmm, ok, good to know, 0.2.6.1010 added to x264.nl with new installer and some files removed that were no longer used!
Please fix the New Updates Available Pop-UP popping up behind the main window issue... it has annoyed me from day one :p
..... now all we need is a: MEGUI LOGO!
Sharktooth
10th September 2007, 15:55
eh...
Atak_Snajpera
10th September 2007, 16:52
AviSource won't open Avis with MJPEG compression, MPEG2 and so on. In case of error MeGUI should switch to DirectShowSource()
Sharktooth
10th September 2007, 16:55
That's tricky... however it will be able to do that if you have the respective VFW decoders installed and working (in ffdshow VFW config enable MPEG in AVI and MJPEG)
Atak_Snajpera
10th September 2007, 16:58
I've already fixed that in my GUI but I use Delphi so I cannot help you further
Sharktooth
10th September 2007, 16:59
it's not necessary. also it's tricky coz megui workflow codebase is more complex.
Sharktooth
10th September 2007, 18:01
0.2.6.1011
- (Sharktooth) [OneClick] Audio Track 2 is disabled by default unless "Show Advanced Options" is checked.
- (Sharktooth) [Autoupdate] Check if yadif.dll path was set before trying to update it...
also available here: http://sourceforge.net/project/showfiles.php?group_id=156112
Sharktooth
11th September 2007, 14:12
i've put the latest CD's build in the auto-update (dev) server. Please test if it works correctly.
bob0r
11th September 2007, 15:42
0.2.6.1011
1: you put start menu shortcut to point to: "C:\Program Files\trunk\dist\bigdist\megui.exe"
2: The Updates Available pop-up still pop ups behind the main window.
3: you cannot set yadif.dll path, because when you browse for it, you need the actual file.
Just set one default path for all none set program files?
%installdir%/tools would be a good place to dumb stuff :)
- Muxer/avimux_gui.exe also has no path set
- Audio/neroAacEnc.exe also has no path set
- Audio/besplit.exe also has no path set
Maybe just set default path to C:\Program Files\megui\tools\
Or C:\Program Files\megui\tools\<program name>\ for each .dll or .exe
icon for installer:
http://x264.nl/x264.ico (you already have it seems)
http://x264.nl/x264.bmp the logo inside the installer, use for dimensions to create your own! (now it looks ugly/weird)
Sharktooth
11th September 2007, 16:11
installer should be fixed.
i will put a new build up...
however i dont really know why the autoupdate pop-up just pops "under"...
nk
11th September 2007, 16:19
I can't see preview with new version of MeGUI)0.2.6.1011), becase of MediaInfoWrapper.dll.
If I use old version of MediaInfoWrapper.dll, I can see preview.
Sharktooth
11th September 2007, 16:34
0.2.6.1012
- (Sharktooth) [Autoupdate] Removed yadif.dll path check since it was impossible to set the path without already having yadif.
- (Sharktooth) Various Installer fixes.
@bobor: the installer stuff is located here: http://megui.svn.sourceforge.net/viewvc/megui/NSIS%20Installer%20script/
Sharktooth
11th September 2007, 16:34
I can't see preview with new version of MeGUI)0.2.6.1011), becase of MediaInfoWrapper.dll.
If I use old version of MediaInfoWrapper.dll, I can see preview.
are you sure? it works pretty well here.
are you on XP or Vista?
bob0r
11th September 2007, 18:06
@bobor: the installer stuff is located here: http://megui.svn.sourceforge.net/viewvc/megui/NSIS%20Installer%20script/
Hmm cool, maybe better to remove ANY spaces on SVN for any file or dir, for my auto scripts its very annoying :)
Just add _ or . or just remove spaces :cool:
I have added 0.2.6.1012 to x264.nl, if developement will be continuous again, ill have a script auto update once a day.
The more testers the better!
nk
11th September 2007, 22:54
are you sure? it works pretty well here.
are you on XP or Vista?
Vista. If it happens only on my environment, maybe something wrong with my envirnment.
This problem also happend at old version (I forgot version number, but not recent).
Additional Information:
MediaInfoWrapper(0.7.5.3 maybe latest version) does not properly work on my environment.
0,7.4.4 works fine.
If I open some avs file for video, an error dialog pops. It says wrong color space. But avs file's
color space is YV12.comverttoyv12() does not help me:mad:
MarcioAB
12th September 2007, 02:36
Is there any chance to see adaptive Q parms (--aq-strength and --aq-sensitivity) on MeGUI ?
Who knows why "adaptive Q" is not official in x264 (IMHO it should be) but anyway: it works great (at least for me).
It deserves a much better space in "Quantizers" Advanced tag than a "Custom Commandline Options" in Zones tag.
Thank you.
Sharktooth
12th September 2007, 14:03
@MarcioAB: NO. i already explained it in another thread and i also explained how to use those options with megui.
http://forum.doom9.org/showthread.php?t=129813
please dont crosspost.
@nk: thanks, i'll try to investigate the problem but i havent vista for testing...
vBulletin® v3.8.11, Copyright ©2000-2025, vBulletin Solutions Inc.