MeGUI development

[Kurtnoise]

Sorry but...what is the relationship between MediaInfo(wrapper) and the avs preview ? because I'm pretty sure that the lib doesn't have an avs parser.

@nk : Vista 32 or 64 bits ?

[Sharktooth]

Hmm cool, maybe better to remove ANY spaces on SVN for any file or dir, for my auto scripts its very annoying :)
http://megui.svn.sourceforge.net/viewvc/megui/Installer/

[Sharktooth]

0.2.6.1013
- (Sharktooth) New MeGUI icons and installer fixes
- (Kurtnoise) [JobQueue] : scrollbar fix (#1793456)
- (Kurtnoise) [AudioEncoder] : fix mp2 bitrate command line for the new FFmpeg builds.
- (Kurtnoise) Better label names for some encoders.
- (Kurtnoise) Cosmetic : added m2ts extension.
- (Kurtnoise) Removed resize event.

[Kurtnoise]

you should wait before to submit new release dude...;)

coz, bugs tickets 1677527, 1749803, 1791912, 1793313 can be closed now.


btw, Vista issues suck...I need to reinstall this OS to debug the GUI. blahhhhh.

[Sharktooth]

get some virtualization software.
im also thinking to install vista to test megui issues... just for that...
however ill release a new build later...

[Sharktooth]

here we go:
0.2.6.1014
- (Kurtnoise) [MainForm] fixed minimize to tray (#1749803)
http://sourceforge.net/project/showfiles.php?group_id=156112&package_id=244469

[Kurtnoise]

I don't like very much virtualization tools to debug applications but this is an other story. :)

Questions for you:
About yadif : we should add it in the server and/or Update Window Form or not ?
If I add Aften, should I drop the FFmeg AC3 part/code or not ?

[Sharktooth]

yadif is already in the autoupdate and the updatewindow has an entry for it. are you having problems getting it from the autoupdate?
http://img265.imageshack.us/img265/8543/yadiflj2.th.png (http://img265.imageshack.us/my.php?image=yadiflj2.png)

I would keep both Aften and ffmpeg AC3. we have even 3 AAC encoders...

[Kurtnoise]

yadif is already in the autoupdate and the updatewindow has an entry for it. are you having problems getting it from the autoupdate?
well...it's not in the ListView for me. :confused:

http://img258.imageshack.us/img258/8492/meguiupdatewindow130920bd7.th.png (http://img258.imageshack.us/my.php?image=meguiupdatewindow130920bd7.png)


About Aften : it uses the same encoder engine. ;) That's why I ask...

[Sharktooth]

Its not in the listview coz its not in the avisynth filters section. since it's a C-Plugin its treated (and shown in the listview) as an external program.

about aften... i think ffmpeg is safer to "mantain"... i mean, aften is a small project and one day it may die, while ffmpeg is a solid project...

[Kurtnoise]

Its not in the listview coz its not in the avisynth filters section. since it's a C-Plugin its treated (and shown in the listview) as an external program.
well...the problem is outwhere I think. I can't see it when I select the 1st node in the TreeView (the complete list).

[Sharktooth]

uhm... try deleting the settings.xml file inside the megui installation folder
if it doesnt still show up delete also the autoupdate.xml file
or just reinstall megui in another dir

[Kurtnoise]

same problem...this link (http://mewiki.project357.com/auto/) is the correct one for the development ?

coz I can't select the web link...
http://img507.imageshack.us/img507/3312/meguiupdatewindow130920gj8.th.png (http://img507.imageshack.us/my.php?image=meguiupdatewindow130920gj8.png)

whereas they are listed in settings.xml. Weird...:s

next discussion about this issue here (http://forum.doom9.org/showthread.php?p=1044639#post1044639). :)

[Sharktooth]

no.. the autoupdate is located at http://megui.org/auto/ (dev) and http://megui.org/auto/stable/ (stable)
... however i cant see the servers too... but it updates regularly... damn bugs...

[saint-francis]

Yadif goes to the Avisynth plugin directory for me and then when there is a new update I get a message telling me that the path is incorrectly set for yadif.dll until I set it manually.

[Sharktooth]

yadif should not go to the avs plugin dir... nowhere in the code nor in the autoupdate there's something that tells megui to place it there...

[bob0r]

This is the dev server:
Trying server: http://mewiki.project357.com/auto/
Retrieving update file from server...

After restart:
Trying server: http://megui.org/auto/
Retrieving update file from server...

Nothing shows any update to yadif09.zip
Not even 0.2.6.1014

:)

[Sharktooth]

yeah... the mewiki is having troubles though...

[foxyshadis]

yadif should not go to the avs plugin dir...

Why not? It doesn't hurt anything, you just won't be able to autoload it. (Unless you also create an avsi to go alongside it that does nothing but load it.)

[saint-francis]

yadif should not go to the avs plugin dir... nowhere in the code nor in the autoupdate there's something that tells megui to place it there...

Then where should it go?

[Selur]

in a subfolder like e.g. avisynth/plugins/yadif

[Kurtnoise]

What about yadifmod instead ?

yadifmod v1.0 (http://bengal.missouri.edu/~kes25c/yadifmod_v1.zip). I've had this for a while, but never got it together for release. It is the same as Fizick's port, except that spatial predictions are taken from a user supplied clip. Also, it is not an Avisynth_C plugin. It works with YV12 and YUY2 input.

[Sharktooth]

yep, i think it will be much better just coz we can support it as an avs plugin instead of creating its own settings...

[Dark Shikari]

Just a reminder to the MeGUI devs that when my patches (http://forum.doom9.org/showthread.php?t=129842) are committed to the main x264 build, MeGUI will probably want to add options to support --me imh, --fpl-cmp satd, and --me-prepass (and tweak the more insane profiles to support them).

[Sharktooth]

np. post the request in the feature request tracker on SF along with both short and detailed descriptions of those options :)

[Kurtnoise]

well...the problem is outwhere I think. I can't see it when I select the 1st node in the TreeView (the complete list).
I think I found the problem...there is no yadif node in the upgrade.xml file. At least on this one (http://megui.org/auto/upgrade.xml)...:)

[Sharktooth]

??? what happened ???
i added it when berrinam added yadif to megui...

going to fix it...

EDIT: please check if it works now...

[Kurtnoise]

yes...works fine now.

10x

[Sharktooth]

i still cant understand why it wasnt there.
i clearly remember i've already added it...

[Kurtnoise]

@Sharktooth : about the #1797746 (http://sourceforge.net/tracker/index.php?func=detail&aid=1797746&group_id=156112&atid=798476):

Index: AviSynthAudioEncoder.cs
===================================================================
--- AviSynthAudioEncoder.cs (revision 215)
+++ AviSynthAudioEncoder.cs (working copy)
@@ -376,7 +376,58 @@
}
else
{
- _logBuilder.Append("Error:\n" + e.ToString());
+ // Better Errors Exception for Audio Encoders
+ int path_index = _encoderExecutablePath.LastIndexOf(@"\");
+ string audio_encoder = _encoderExecutablePath.Substring(path_index + 1).ToLower();
+
+ _logBuilder.Append("Error:\n");
+
+ if (audioJob.Settings is NeroAACSettings)
+ {
+ if ((audio_encoder != "neroaacenc.exe") || (audio_encoder != "neroaacencsse.exe"))
+ _logBuilder.Append(audio_encoder + " not found in the path...\n");
+ }
+
+ if (audioJob.Settings is OggVorbisSettings)
+ {
+ if (audio_encoder != "oggenc2.exe")
+ _logBuilder.Append(audio_encoder + " not found in the path...\n");
+ }
+
+ if ((audioJob.Settings is MP2Settings) || (audioJob.Settings is AC3Settings))
+ {
+ if (audio_encoder != "ffmpeg.exe")
+ _logBuilder.Append(audio_encoder + " not found in the path...\n");
+ }
+
+ if (audioJob.Settings is MP3Settings)
+ {
+ if (audio_encoder != "lame.exe")
+ _logBuilder.Append(audio_encoder + " not found in the path...\n");
+ }
+
+ if (audioJob.Settings is FaacSettings)
+ {
+ if (audio_encoder != "faac.exe")
+ _logBuilder.Append(audio_encoder + " not found in the path...\n");
+ }
+
+ if (audioJob.Settings is AudXSettings)
+ {
+ if (audio_encoder != "enc_audx_cli.exe.exe")
+ _logBuilder.Append(audio_encoder + " not found in the path...\n");
+ }
+
+ if (audioJob.Settings is WinAmpAACSettings)
+ {
+ if (audio_encoder != "encaacplus.exe")
+ _logBuilder.Append(audio_encoder + " not found in the path...\n");
+ if (File.Exists(encoder_path+"aac_encplus.dll") == false)
+ _logBuilder.Append("aac_encplus.dll not found in the path...\n");
+ if (File.Exists(Environment.SystemDirectory+@"\nscrt.dll") == false)
+ _logBuilder.Append("nscrt.dll must be in your Windows System directory...\n");
+ }
+
su.HasError = true;
raiseEvent();
}


ok to commit ?

[Sharktooth]

ok. keep in mind the audxsettings should be replaced by mp3surroundsettings once mp3surround gets fully integrated

[pimpMyHD]

Hi guys,

i use daily megui, and i must admit that if megui had been a commercial software, i 'd bought it.

As a .Net dev, i looked at the code. I can't help it ;-)
I must admit it is mostly well coded, but some .net/C# specifications or rules are not always respected.
for example, some abstract types have public constructors, raising some runtime reserved exceptions types , etc ... stuffs like that ...
There are no 'top of the todo list' stuff to change, but it could be fine to tweak some lines of code.

I know that managing a project do not make room to code quality, and time is mostly spend to optimizing features.



This is the only way i can help this incredible project, i do not have the video/audio processing knowledge to help more.

so ... if you agree ... if you are interested in it, i can correct what i think it should be.
I will post the sources if you will so.

regards.

[Sharktooth]

yes, any help is appreciated. if you have some experience with SVN, you can post unified diffs for latest code revision on the patches tracker (http://sourceforge.net/projects/megui) and we will gladly integrate them in the trunk. credits will be given in the megui changelog.

[pimpMyHD]

yes, any help is appreciated. if you have some experience with SVN, you can post unified diffs for latest code revision on the patches tracker (http://sourceforge.net/projects/megui) and we will gladly integrate them in the trunk. credits will be given in the megui changelog.
Ok, i will download the latest revision and start to look at the code. When i will ready to commit some code, i'll deal it with you ( just to be sure i'll commit the right way, in the right place).

[Sharktooth]

thanks

[pimpMyHD]

thanks
i tried to MP you, but it seems you need to make room in your mp mail box :D

[Sharktooth]

eh... my PM inbox is full on purpouse. however you can contact me by MSN email

[Fizick]

I do not like when my programs (Avisynth plugins) are distributed in stripped form (without documentation and source codes).

When you remove doc, you make user blind. He has no chance to learn how program work, what is additional options, who developed it, etc.

When you remove souce codes, you violate GPL license.


If you distribute GPL program binary, you must distribute its source too.

Here is part 3 of GPL license text:



3. You may copy and distribute the Program (or a work based on it,
under Section 2) in object code or executable form under the terms of
Sections 1 and 2 above provided that you also do one of the following:

a) Accompany it with the complete corresponding machine-readable
source code, which must be distributed under the terms of Sections
1 and 2 above on a medium customarily used for software interchange; or,

b) Accompany it with a written offer, valid for at least three
years, to give any third party, for a charge no more than your
cost of physically performing source distribution, a complete
machine-readable copy of the corresponding source code, to be
distributed under the terms of Sections 1 and 2 above on a medium
customarily used for software interchange; or,

c) Accompany it with the information you received as to the offer
to distribute corresponding source code. (This alternative is
allowed only for noncommercial distribution and only if you
received the program in object code or executable form with such
an offer, in accord with Subsection b above.)


Probably, if you downloaded some GPL binary WITHOUT source (for example Avisynth, which have separate source archive package), you may distribute it without a source too.

But if you downloaded some GPL binary WITH a source in one zip package, you must distribute whole package (with source),
or two packages (small without source codes and full with source code), from same place (server).

User must have info about license status of every "updated" programs and have option to get source codes for every distributed version of binary.

To be specific, I say here about Yadif.

[Sharktooth]

yeah, we know but we have limited traffic on our update server:(
however everything in the autoupdate is licensed under GPL.
since almost all the megui userbase is on this forum, they know yadif and other packages are somewhere here.
however the best thing we can do is to add a readme file with credits and links to the forum threads or websites containing the sources and docs, since distributing them would kill our traffic quota.

[Doom9]

But if you downloaded some GPL binary WITH a source in one zip package, you must distribute whole package (with source),Read the sections you have posted here again.. nowhere does it say that. As long as the offer or information on where to get the source code is there, it's perfectly okay.

Even as the author, you do not get to say which of the three mentioned methods is for you, and you cannot impose additional restrictions (such as which files must be kept from the archive).

While you are technically correct that there is a lack of license files for the individual applications that are being downloaded via autoupdate (plus the offer to get the source code or information on where to get the source code), I find it rather ridiculous to even suggest the authors of a software written from scratch and released under the GPL would not heed the obligations set forth by the license, when asked nicely. A "could it be that you forgot to include the license file and information on how to obtain the source code for software xy ;)", is imho a much more appropriate way to have this resolved - I consider throwing paragraphs around and trumped about GPL violations in public as a last resort and dare doubt that you had to go to such lengths.

[Fizick]

Sharktooth,
it will be first step in right direction.
Megui is quite mature project, and its distribution should be legal.
(And there are many users not from this forum.)

Doom9,
could it be that you forgot to include the license file and information on how to obtain the source code for software YADIF.
;)

(BTW, section C is not the case here.)

But what about removing docs?
It seems that I have got the most simple (stupid) questions from MeGUI users.
:)

[check]

The real problem with providing sources is keeping it up to date. Nobody will ever care about grabbing the files after this, and nobody will bother to update them. Better to simply link to the author's homepage on the MeWiki. I don't understand the GPL, is this an acceptable solution?

If not.... simple solution: provide a .zip file with all the sources and licences for all files in MeGUI. Then put it on all the update servers and put a link in the help menu to it (one that randomly selects an update server as per autoupdate). This is what the CCCP does after some similar drama to this.

[berrinam]

The problem with docs and extra files in general is that a lot of plugin writers name their doc/gpl files the same way: readme.txt and gpl.txt. This means that, since they all (except yadif, since it's an AviSynth_stdcall plugin) get extracted into the AVS plugins directory, they all overwrite each other and it's not clear what they refer to. Perhaps we should instead create a folder inside each zip and put the docs there, to avoid plugin "namespace" collisions.

I'm sorry about not including the license file and source code information in the autoupdate package. As Sharktooth said, we are trying not to make the downloads too big. Including the source code and license, the distribution is 6 times larger than with the plugin alone. However, I have now changed our distribution to include the following file, named "yadif_readme.txt":

The yadif.dll used in MeGUI is a port to AviSynth of MPlayer's YADIF (Yet
Another DeInterlacing Filter). The port is written by Fizick (Alexander
Balakhnin).

Yadif is licensed under GPL version 2; see
http://www.gnu.org/licenses/old-licenses/gpl-2.0.txt for the license.

Source code, documentation, and the newest version of the plugin, can be
found at the plugin's homepage: http://avisynth.org.ru/yadif/yadif.html



-- berrinam

I hope that the URL of the GPL licence is acceptable; I didn't want to include the licence itself as it is actually larger than the plugin itself.

berrinam

[Fizick]

Please consider it is not as collision with Yadif or me.
I simply try understand base for commom practice (links only, as doom9 pointed).

When we discussed Avisynth legal base with IanB, he pointed, that many projects are at public sourceforge servers, and all may be considered as "same place":


If distribution of executable or object code is made by offering
access to copy from a designated place, then offering equivalent
access to copy the source code from the same place counts as
distribution of the source code, even though third parties are not
compelled to copy the source along with the object code.


berrinam, thanks, but you updated readme must aslo include Michael Niedermayer (http://www.mplayerhq.hu) in copyrigt notice

There are no need in 20 copies of GPL License file. :)

[Doom9]

@berrinam: I think we should have one readme file in the main directory, listing all software that can be downloaded, mentions their license (we already have the GPL text so it can just be referenced) and where to get the source code.

Also, technically, we ought to keep the source code of every revision that is put on the update server, for three years - but to do that, I think it would be appropriate to make use of the clause where a user can get the source code upon request for a fee - and that fee should be high enough to make people twice (after all.. the time is cost associated with distribution, too) - after all that's just what the fee clause is designed to - prevent you from becoming bogged down with people requesting the source just because they can, and especially for a volunteer GPL software making use of GPL tools only, it would be simply unreasonable if every user requested the source when clearly every project has a homepage where that source can be gotten without any hassle (especially if we provide links to each project's home).

[Kurtnoise]

@devs: about this issue (http://forum.doom9.org/showthread.php?t=126407)...

in debug mode, I get this:

http://img233.imageshack.us/img233/2572/meguidebug2409200718102ix0.th.png (http://img233.imageshack.us/my.php?image=meguidebug2409200718102ix0.png)

So, any idea about the last line error (unable to load DLL 'AvisynthWrapper' : the specified module could not be found) ?



These vista things annoys me frankly...

[Sharktooth]

there was a similar problem in the past. i think it was berrinam to discover the cause and fixed it updating avisynth wrapper.

[berrinam]

There was a problem with MediaInfoLib and Vista, and I uploaded a new library with a fix given to me by (from memory) moadib2k. But I don't think I've done anything with AvisynthWrapper, and I don't know what would be causing the problems. Dimzon wrote it; maybe he can give us some advice.

[Sharktooth]

you know, my memory is faulty...

[berrinam]

0.2.6.1015
- (berrinam) Nicer CQM selection for XviD and x264 (#1651251 and #1717536)

Odd choice, I know, but I felt like doing a bit of GUI details.

[Sharktooth]

berrinam, check the changelog. there were some other commits before 1015 that were not included in any builds, so you miss some entries in the changelog.

[berrinam]

My mistake, fixed now:

0.2.6.1015
- (berrinam) Nicer CQM selection for XviD and x264 (#1651251 and #1717536)
- (Kurtnoise) better errors exception for Audio Encoders
- (Kurtnoise) [Job Queue] : added keydown event (for start, stop, up, down, delete)
- (Kurtnoise) [SettingsForm] : Overwrite Stats File & Keep 2nd Pass Output checkboxes were not saved in the Settings

[Bigmango]

Hi,

Megui 0.2.6.1014 can't update itself to 0.2.6.1015.

I am on vista x64, running megui with admin privileged for the core update to work (it crashes after the download when restarting if run as a user, so it seems it can't write some file).

So, with admin privileges, after the download it always restarts as the old 0.2.6.1014 version and asks to update again, and again... it's in a loop.

I remember updating with previous version worked fine (the only difference is that the currently installed version was installed with the installer package for version 0.2.6.1014).

So to sum up, the correct behavior should be:

1. after downloading the update megui should restart in the new version.

And:

2. Please fix the user privileges so that they behave correctly in vista. Megui should never need to be run as an admin. It should be run as a user, and if needed (i.ex for the update) it should popup the admin request window to temporarily use admin privileges, as any other properly behaving application does.

Thanks.

[check]

if megui is not updating, just remove and install the latest manually.

[berrinam]

0.2.6.1016
- (berrinam) [AutoUpdate] : add support for a local cache of update files (located via registry) (#1804656)
- (berrinam) [AutoUpdate] : Re-install function in autoupdater (#1790523)
- (berrinam) [Video Preview] : 'Go to frame' for video preview (#1746899)
- (berrinam) [OneClick Window] : Easier code for audio, better audio input selection
- (berrinam) [OneClick Window] : Fix order of jobs in OneClick (bitrate calculation there should work again)
- (berrinam) [Muxer Windows] : Automatically choose output filename in muxer (#1784648)
- (berrinam) [DGIndexer] : Read progress reports from DGIndex title

Note to the people organising the auto-update servers: the local caching feature added in 1016 works by filenames only. This means that every new version has to have a new filename, or the local copy will be used. This is of particular importance for the core, which is currently named core.zip. In future it will have to be named as something which includes the version number, like core_2_6_1014.zip.

[Kurtnoise]

Please fix the user privileges so that they behave correctly in vista. Megui should never need to be run as an admin. It should be run as a user, and if needed (i.ex for the update) it should popup the admin request window to temporarily use admin privileges, as any other properly behaving application does.
What ? We need higher privileges *to write* on the hard drives...Thanks to MS. Nothing to fix here except to embed a manifest file to rise admin rights.

[Sharktooth]

yeah... damn microsoft... they like screwing everything...
i hope Vista dies as fast as it got cracked (2 days after the release...)

[Sharktooth]

Note to the people organising the auto-update servers: the local caching feature added in 1016 works by filenames only. This means that every new version has to have a new filename, or the local copy will be used. This is of particular importance for the core, which is currently named core.zip. In future it will have to be named as something which includes the version number, like core_2_6_1014.zip.
Core and Data packages have version numbers in the filename now.

@Kurtnoise: Does dgindex 1.5b2 work well with MeGUI?

[berrinam]

What's the point of AviSynthScriptEnvironment? It doesn't seem to hold any special resources, so what's wrong with getting rid of the class entirely?

I'm PMing dimzon about this, since he wrote AvisynthWrapper. [[Well, dimzon's PM is full, so looks like I'll just hope he sees this]]

[Kurtnoise]

Does dgindex 1.5b2 work well with MeGUI?
yes...no problem for what I've done with it.

@Berrinam: try his email : dimzon541 at gmail dot com


btw, rev 202 breaks the download of xvid profiles...

[Sharktooth]

202?!?

[Kurtnoise]

222...sorry.

[Sharktooth]

0.2.6.1017
- (berrinam) [OneClick, AviSynth Creator, Video Preview, AutoEncode, and Bitrate Calculator] : nicer comboboxes, for AR, FPS, and target sizes
- (berrinam) Internal change from use of DARX/DARY pairs to a Dar struct
- (berrinam) Turn the IMediaFile info details from virtual calls into populating a MediaFileInfo class
- (Kurtnoise) [Auto Update Server Window] : put serverList on top (items were not visible)

[Sharktooth]

yes...no problem for what I've done with it.
have you tried it even with interlaced/telecined stuff?

[Kurtnoise]

yeah...works fine for me.

[dimzon]

Hi everyone!

[dimzon]

@devs: about this issue (http://forum.doom9.org/showthread.php?t=126407)...

in debug mode, I get this:

http://img233.imageshack.us/img233/2572/meguidebug2409200718102ix0.th.png (http://img233.imageshack.us/my.php?image=meguidebug2409200718102ix0.png)

So, any idea about the last line error (unable to load DLL 'AvisynthWrapper' : the specified module could not be found) ?



These vista things annoys me frankly...

actually I have no Vista installed so I can only guess... but I need at least detailed description first

[Kurtnoise]

Hey, nice to see you here...;)


About the problem: as soon as you load an avs script in the Video Encoding part, this message appears. Tried with different scripts (AVISource, MPEGSource, FFMpegSource, etc...), with different Avisynth versions (2.5.7 & 2.5.8 alpha) and still the same error. Note that a YV12 decoder is installed as well because the avs scripts are perfectly playable with some tools like VirtualDub or some dshow players (MPC...). To complicate the trick, it occurs only on Vista. :s


Hope this helps...you can find also some extra comments from the link pointed out with the picture. I don't think it's related to MediaInfo.

[berrinam]

Hey dimzon!

Can you answer this question too please?

What's the point of AviSynthScriptEnvironment? It doesn't seem to hold any special resources, so what's wrong with getting rid of the class entirely?

[foxyshadis]

What ? We need higher privileges *to write* on the hard drives...Thanks to MS. Nothing to fix here except to embed a manifest file to rise admin rights.

Well, it does crash hard if it can't write, I'd say that needs fixing at a minimum. If you want I'll write a patch to allow updates under limited user that only requires the updatecopier to be elevated to admin (by downloading into a user temp folder). I have a few vista systems I can terminal into at an office now.

Don't put it in the manifest though, it doesn't really belong there. (Especially not megui's manifest.) Better to check whether the folder is writable, if so proceed as normal, if not start it as an admin (accomplished by adding the Verb 'runas' to ProcessStartInfo).

I dealt with it by just making the whole audio/video/imagery swaths of my program files world-writable, but if you don't put them into special folder it's a pain.

[dimzon]

Hey dimzon!

Can you answer this question too please?
actually I do not remember... i need source code first. can anybody mirror it @ yousendit.com (i can't work with CSV/SVN from here)

[Sharktooth]

http://www.webalice.it/f.corriga/megui/MeGUI-sources.7z

[berrinam]

And I've also sent it to you with yousendit.com. Ah well, the more, the merrier. :p

[dimzon]

What's the point of AviSynthScriptEnvironment? It doesn't seem to hold any special resources, so what's wrong with getting rid of the class entirely?
Initially i planned to create special AviSynthScriptEnvironment one per thread. One AviSynthScriptEnvironment can serve multiple clips so I decide to use as little amount of AviSynthScriptEnvironment as possible to avoid resource leaks... Some times latter I asked Avisynth developers how about to create AviSynth ScriptEnviroment for each clip. They answered everything is ok. So I removed all such code from AviSynthScriptEnvironment class but don't remove completely bcz I do not want to modify ALL places where it was invoked...
If you want You can remove it entirely...

[Kurtnoise]

Well, it does crash hard if it can't write, I'd say that needs fixing at a minimum. If you want I'll write a patch to allow updates under limited user that only requires the updatecopier to be elevated to admin (by downloading into a user temp folder). I have a few vista systems I can terminal into at an office now.
yeah, feel free to post patches...:)

Anyway, updatecopier is elevated by itself (MS heuristic for update, etc...).

[Bigmango]

Reporting back:

Now trying to update version 0.2.6.1016 to 0.2.6.1017 with the updater. After downloading the update it asks to restart and yes, updatecopier.exe now asks for admin privileges :)

But it still restats as the old version 0.2.6.1016 and then asks to update again... and it's still always the same again and again... in a loop. It's not restarting with the new version, so I guess megui.exe was not replaced ?

Btw, this is on vista x64. Megui is installed in a directory that is writable to the current user (d:\temp\megui), so it is strange it doesn't work...

[berrinam]

0.2.6.1018
- (berrinam) [Internal] : Remove Commandline field from Job
- (berrinam) [Internal] : Remove commandline generation from CommandLineGenerator and put it in individual encoders
- (berrinam) [Internal] : Remove Framecount and FPS from VideoJob
- (berrinam) [Internal] : Separated Job into Job Queue info (now under TaggedJob) and job running info (under Job, as before). This may cause temporary Serialization bugs
- (berrinam) [Internal] : Reworked/simplified BitrateCalculator
- (berrinam) [Main window, Mux window, OneClick] : Can have arbitrarily many audio tracks
- (berrinam) [Internal] : Documentation for MuxPath finding
- (berrinam) [Internal] : Moved drag & drop code to FileBar to avoid repetition
- (berrinam) Load jobs now works for mux and audio jobs

[Sharktooth]

Bitrate calculator throws an exception if no input file is selected.
Commandline options are no longer shown in the codec config window.

[Warpman]

job list is gone....

if u have ~200jobs in the list you get 200errors saying that the job$number.xml is not readable. cause the .1017 update changed something i guess....

if a profil/job contains an error i get the same error but selecting "delete? yes" doesn't work it doesn't get deleted. (it ask again next time, annoying as hell)

And sometimes a random profile gets corrupted by megui(cause i don't think there is a way to damage a video profile if you didn't edit it for a month or so)

btw. got the same error as sharktooth as well...

Edi: ph almost forgot: the update window doesn't display the current version that is avaible though the upadter, only the installed version.

[Sharktooth]

yeah.. i already posted the update window thing in the bugtracker

[COOLBGT]

Somebody totally messed up the last update 2.6.1018. The MKV muxer where it says output there's an open button instead of save. When you leave the auto made name in the field it gives and error and closes. Peace

[Sharktooth]

eh... im not at home though... so hope some dev is around to fix it...

[mroz]

I can't do anything useful with 2.6.1018 - autoencode doesn't queue anything up & I can't use the bitrate calculator & manual enqueue as the bitrate calculator always throws an exception. Ouch.

Is there a way to rollback to an earlier version? If I manually edit autoupdate.xml I can get the updater to reoffer the current latest on the stable or dev servers, but I can't see a way to select a specific earlier build. Are they actually on the servers? Ah yes, a few are & in anycase I have a couple cached. I guess I'm reverting to 1017 for now.

Today's the first time I've used Megui for xvid encodes & I was getting worried it wasn't usable - it's a relief to discover it's all broken instead ;)

[Sharktooth]

well, it's a dev build... however there's an update_cache where you can get the .1017 binaries...
i think i will remove .1018 from the autoupdate as soon as im back home...

[saint-francis]

i think i will remove .1018 from the autoupdate as soon as im back home...

Thank you. I've had many headaches with the new build. Particularly with the gui.

[salehin]

you can also manually download core_0_2_6_1017.zip from http://megui.org/auto/ and put the contents in megui directory.

[Adub]

That's what I did and it works perfectly. At least, it starts up fine, I haven't had time to test any encodes.

[Sharktooth]

done.

[COOLBGT]

Oh yeah and more (actually two) things. As of update .1017 the update window doesn't show the latest version of the updates, only the existing one. It's not that big of a deal but it's a good thing to know what you are downloading. And why don't you include the nero aac audio encoder in the update window. We have to download it manually form the nero site to use it. Same thing with the avi-mux and another program called besplit(i don't know what's that used for). Peace

[Sharktooth]

i already said that "version not shown" bug was posted in the bug tracker AND we can't redisrtribute the nero AAC encoder coz the encoder license.

[berrinam]

0.2.6.1019
- (Kurtnoise) [One Click Encoder] : fix output filename according to the container selected in the Advanced Settings.
- (Kurtnoise) cosmetic - Move "cleanup" to EncodingMode instead of CodecString.
- (Kurtnoise) [Bitrate Calculator] : fix a runtime error.
- (berrinam) Fixed AutoEncode to queue jobs again
- (berrinam) Fixed #1805580 (Automatic output filename assumes .mp4 output in muxer)
- (berrinam) Fixed Bitrate Calculator to support multiple audio tracks; this fixes crash when opening it without input
- (berrinam) Fixed #1805603 (Automatic update not showing the Latest version number)

[foxyshadis]

Curious, did you forget to commit something called MultiTrackComponent.cs this last rev?

[berrinam]

0.2.6.1020
- (berrinam) Support backwards-compatible loading of jobs from 0.2.6.1017. (But not vice-versa!)

[berrinam]

I deliberately omitted it, since it wasn't needed. But I forgot to delete it from MeGUI.csproj. My last commit has the modified project file which doesn't reference MultiTrackComponent.cs

[Kurtnoise]

Hi,

Job.cs in core\details\0_2_6_1017_jobloader\ is still missing. Cannot compile...

[berrinam]

Added.

[berrinam]

Oh, I don't know if I mentioned this before, but the update_cache folder now makes it possible to distribute networked installs. It's not ideal, because megui will still access the web for the update.xml file, but it will get all the data files from the local cache.

To make such an installer, you simply need to package all of the zip files on a relevant update server (dev server if you're making dev releases; stable server if you're making stable releases) in the installer, and put them somewhere sensible during the install process. MeGUI checks the registry key, HKEY_LOCAL_MACHINE\SOFTWARE\MeGUI\update_cache, for the update cache's folder, so the installer should set that key in the registry to point to the folder the installer put the files.

[check]

Just thought everyone in here would like an update on how popular MeGUI is becoming.
Usage for July, Aug, Sept:
95gb, 110gb, 135gb
In particular, for september:

99.2% of the bandwidth was for autoupdate, which was around 260 000 files.
The other 0.8% of the bandwidth went to requests for PHP pages (ie, the wiki), with 23 000 hits.
About 4 000 requests for wiki pages were from links from other sites, the rest were direct links.
upgrade.xml was hit 78 000 times.
The split between IE and not-IE (mostly Firefox) for the wiki pages was around 50/50.
Someone in Victoria on Bigpond used the most bandwidth on the site -- 200mb.
The top five visiting countries by IP: US, EU, AU(!), DE, CN. The US accounted for just over 70% of all bandwidth.

Not bad eh (especially considering the changed link urls would have reduced traffic)? :)

[Sharktooth]

wow! i didnt expect that popularity...
more stats are available at the megui project tracker on SF: http://sourceforge.net/project/stats/detail.php?group_id=156112&ugn=megui&type=tracker
400-500 downloads/day...

[COOLBGT]

Why did you guys made the same mistake with .1020 . Again all the muxing windows has the open button instead of the save button

[Sharktooth]

it was simply not fixed... report the bug in the MeGUI bug tracker

EDIT: i fixed it though, next version will have the fix... so dont report it. thanx.

[Sharktooth]

0.2.6.1021
- (Sharktooth) [Muxers] Fixed the output file dialog (save mode)

small change but it was worth building since you couldnt select the output file in the muxers...
i also edited the .1019 changelog entries to reflect a minor change.
the new build is on the autoupdate.

[Sharktooth]

Has anyone a clue why shutdown doesnt work? Im getting mad...

@kurtnoise: i accepted you MSN invite...

@devs: further work must be aimed ad stabilizing the code, so forget about features and stuff (unless they're small changes, like exposing and adding new codec options or adding some exception handlers... however nothing that can screw the codebase and/or will need further bug fixing) and lets get all bugs squashed. after we reach a stable stage we'll release a stable package and we can concentrate on adding stuff or messing with the code.
Tomorrow ill create stable and unstable branches in the SVN repo, so we can port fixes to the stable version and continue with developing the unstable version until it reaches a stable status.
on the developers front, SF has a nice recruitment service. should i seek for new devs using the service?

@all: i need to know if dgindex 1.5 works correctly with MeGUI. I dont want to break things at this stage, so please extensively test it with ALL kind of sources (progressive, interlaced, telecined...) and report back. thanks.

[berrinam]

Some of the shutdown is simply not running: it was removed at one stage and never made it back in. I have a local version on my computer which fixes that.

EDIT: Committed to SVN.

[berrinam]

I notice that Sharktooth has added bug #1808055 (Encoder commandline options not being shown in the codec config window) to the SF tracker. Yes, it is a bug currently. However, it is not an accident -- I removed it in 1018 when I removed the CommandLineGenerator, as I moved some code around, thinking that command line generation is more appropriate in the individual encoders that it generates a commandline for.

There is of course no reason why it is infeasible to put the commandline back in the video codec config, but is it actually sensible to? My only use for it has been to check that all the buttons and checkboxes are being updated correctly and the commandline is being generated correctly: in short, I have only used it for diagnostic purposes, and not useful for anyone except a developer.

I think we should get rid of it. What do you think?

[mroz]

If you're asking all users, I personally find such information useful & would much prefer if devs feel the way you do, the information be hidden by default but still available via some sort of 'advanced info/features' toggle.

If you'd like to know why I find it useful, that's two fold. If I have a problem it reassures me I'm at fault rather than the gui I'm using, since I get feedback that the gui's doing what I asked. Secondly, it helps me to learn how the tool works & get to know the codec better, which I always find helpful.

If Megui ever turned into nothing more than a one click app with all internal functionality hidden I'd be very sad, however good the results. That said, I love to have such functionality available; I just prefer the freedom to choose how much automation I want, so that my usage patterns can grow as my understanding increases. I love apps that let me do that.

Sorry if I've gone off at a tangent or started ranting. Just wanted to voice my feelings. BTW you shouldn't be surprised how popular Megui's becoming. It's a very nice piece of kit :)

[foxyshadis]

It needs to be showable on that dialog, wherever the code actually is. One of MeGUI's primary target audiences is the sort of people who are likely to copy a configuration straight to a command-line (batch, etc), without running an encode first to get it out of the log. Or check it to quickly determine that they haven't made a mistake. Frankly, I think MeGUI has the best and most complete x264 option GUI out there.

[berrinam]

Hmmm... that comment reminds me of a feature I remember thinking about ages ago: exporting jobs to a batch file. I had thought someone had implemented it, but I can't find it in MeGUI or the changelog. Well, I better put it on the SF tracker.

[berrinam]

0.2.6.1022
- (berrinam) Fixed #1808324 (1 pass encode broken in autoencode)
- (berrinam) Fixed #1808055 (Encoder commandline options is not being shown in the codec config window)
- (berrinam) Fixed #1806911 (Clever anamorphic encoding not working)
- (berrinam) Shutdown when finished re-enabled (including countdown patch from imoon4)
- (berrinam) FPS chooser now says "choose your framerate" not "choose your AR"

[foxyshadis]

Do you guys think this is a good idea? Or better to just silently copy settings (jobs etc) to the user profile and work from there?
http://foxyshadis.slightlydark.com/random/meguiperm.png
Of course it only matters when people put it in programs, megui can happily run from documents if people want.

[berrinam]

By default, I think MeGUI should be installed to program files, and it should also work by default, even on Vista (without the user having to answer a question like you posted).

I can't see much point in having per-user settings (jobs/profiles), since it isn't like diary or email software where everyone has a personal account. However, I also don't see anything particularly wrong with it, and if it is the recommended way (or the clean way) of handling things on Vista, then I don't see anything particularly wrong with that. In that case, though, we should perhaps change the behavior for all versions of windows, though.

Of course, downloaded files such as x264, dgindex, should definitely go in program files.

[foxyshadis]

Having something in program files and having it work without user profiles or changing permissions is pretty much an impossibility in Vista (or an XP limited account). You can quietly change permissions, but a randomly appearing admin confirmation/runas box on startup will probably throw people.

Actually, on Vista the situation seems better, everything gets dumped into %Appdata%\Local\VirtualStore\Program Files\megui while you're using the program, I forgot that it has the virtalization. Even in the admin account. Since it would just work, it wouldn't even pop the box above up, the OS takes care of everything as long as it's not turned off. Update still has to be fixed, though, it just spins its wheels now.

The MS and unix standard is that anything that gets modified regularly is per-user. Jobs, logs, and profiles are kind of per-user things anyway, they're yours after all, although it could go either way.

I have a Vista system you can terminal into if you'd like, there's nothing private on it, it's just a terminal client. (I'm still getting used to it, first time I've really played with it in depth.)

[mroz]

2.6.10.21 Adaptive Mux Window bug - chapters can't be added. The Open file dialogue has file types set to:
"Chapter files (*.txt)
Notice the leading quote. This seems to result in no files being listed, consequently chapter txt files can't be selected.

[berrinam]

Please report bugs on the sourceforge bug tracker. There is a link in my signature.

[mroz]

Please report bugs on the sourceforge bug tracker. There is a link in my signature.

Done.

[Sharktooth]

@berrinam: knowing the commandline options will:
1 - let the user know what a megui options translates to the real encoder option
2 - be usefull to spot typos or other kind of errors in command line generation
3 - be usefull when implementing new encoder options

however thanks for re-adding it.

[berrinam]

@berrinam: knowing the commandline options will:
1 - let the user know what a megui options translates to the real encoder optionGood point, although it's not useful for me, because I trust MeGUI to do the entire job for me. Perhaps the "export to batch (https://sourceforge.net/tracker/index.php?func=detail&aid=1808438&group_id=156112&atid=798479)" feature request I added will help alleviate this need in the future?

2 - be usefull to spot typos or other kind of errors in command line generation
3 - be usefull when implementing new encoder optionsAs far as I can see, this is just diagnostics, mainly for the developers. We already have the log for that, don't we?

however thanks for re-adding it.No worries. It seems that, at the moment, it is a very useful thing to have.

[Sharktooth]

0.2.6.1023
- (Kurtnoise) [Muxer] : Chapters Filter fixed.
- (Kurtnoise) [Avisynth Window] : deinterlace checkbox unchecked when Analysis returns "Do Nothing" (to avoid interlaced=true with ColorMatrix())
- (Kurtnoise) [Help Button] : url fixed for the wiki.
- (berrinam) Catch all exceptions in ProgressWindow.UpdateStatus. (It is a non-essential function, so errors should be silently caught). Should probably fix #1760486

[Sharktooth]

0.2.6.1024
- (Sharktooth) Fixed main window (Form1.cs) size (some Audio controls were not shown)
- (Sharktooth) Fixed #1810120 (x264 video configuration dialog problem)
- (Sharktooth) [Update Window] Workaround for #1808878 (Maximize/Minimize states don't work). Disabled Minimize and Maximize buttons.

[Doom9]

c't asked to put megui on the DVD for one of their upcoming issues.
I took the liberty of approving the request.. I put "Doom9 community" as author and noted that users should come here for support and that avisynth is a prerequisite.

[Adub]

Who is c't?

[mroz]

My guess (http://en.wikipedia.org/wiki/C't).

[Sharktooth]

@Doom9: also tell them MeGUI is in beta stage too, so there could be some bugs.

[berrinam]

0.2.6.1025
- (berrinam) Fixed #1812162 (Up & down buttons in queue are mixed up)
- (berrinam) [Adaptive Mux Window] Fixed some incorrect mux-path-finding which caused a crash -- the window thought it could mux something that it couldn't.
- (berrinam) Filetypes are better determined, by also looking at the extension. This may fix #1808882.
- (berrinam) Fixed xvid AR signalling. (Use PAR not DAR)
- (berrinam) Fix #1808573 (Not able to import xvid profiles anymore)
- (berrinam) Fix #1812842 (ITU 4:3 is wrong)

[berrinam]

0.2.6.1026
- (berrinam) Fixed #1746055 (Video profile selection error)
- (berrinam) Fixed #1749810 (For each level have max bitrate for main and high profiles)
With this commit, there are no bugs in the SF bug tracker that I can reproduce other than Vista issues and the large font issue. If anyone has forgotten to post bugs, please do it now.

Also, I think that both the Vista and the large font problems are likely to require bigger changes than simple one-liners. Given that, do you think it would be a good idea to wait a week or so, and then release 0.2.6.1026 as a stable version?

@foxyshadis: Are you still working on the Vista stuff? What's the news? I think that we should move the jobs, profiles, and MeGUI settings to "my documents" in the future, but I'm not sure how that will work for people who want to run multiple instances of MeGUI at once, with different job queues (it's something I do all the time, as I have multiple copies of MeGUI for development, and one copy for encoding). Perhaps My Documents/megui should have a separate subdirectory for each version of megui installed on the computer, with either a registry key or an .ini file in each subdirectory to say which subdirectory belongs to which copy of megui. Thoughts?

[Bigmango]

If anyone has forgotten to post bugs, please do it now.

Posted "Core auto-update does not work".

[berrinam]

I tried debugging MeGUI on a Vista computer today, and I believe I solved the file-opening problem.

The old MediaInfoWrapper had a PInvoke call

[DllImport("MediaInfo.dll")]
internal static extern int MediaInfo_Count_Get(IntPtr Handle, [MarshalAs(UnmanagedType.U4)] MediaInfoStreamKind StreamKind, int StreamNumber);

// and later

private string GetSpecificMediaInfo(MediaInfoStreamKind KindOfStream, int trackindex, string NameOfParameter)
{
return MediaInfo.MediaInfo_Get(this.Handle, KindOfStream, Convert.ToUInt32(trackindex), NameOfParameter, MediaInfoInfoKind.Text, MediaInfoInfoKind.Name);
}

And when that was called, it occasionally (on the longer strings) crashed with an Access Violation. I replaced it with

[DllImport("MediaInfo.dll", CharSet = CharSet.Unicode)]
internal static extern IntPtr MediaInfo_Get(IntPtr Handle, [MarshalAs(UnmanagedType.U4)] MediaInfoStreamKind StreamKind, uint StreamNumber, string Parameter, [MarshalAs(UnmanagedType.U4)] MediaInfoInfoKind KindOfInfo, [MarshalAs(UnmanagedType.U4)] MediaInfoInfoKind KindOfSearch);

// and later

private unsafe string GetSpecificMediaInfo(MediaInfoStreamKind KindOfStream, int trackindex, string NameOfParameter)
{
IntPtr p = MediaInfo.MediaInfo_Get(this.Handle, KindOfStream, Convert.ToUInt32(trackindex), NameOfParameter, MediaInfoInfoKind.Text, MediaInfoInfoKind.Name);
char* p2 = (char*)p.ToPointer();
string s = "";
while (*p2 != '\0')
{
s += *p2;
++p2;
}
return s;
}

Which appears to fix it. The thing I don't get is, I thought the PInvoke Marshaller converted the string exactly as I did. Obviously not, though, as it crashed whereas this code works.

Does anyone know what's up with this?

--

Anyway, I have committed the updated library and I will upload it to AutoUpdate, so that this bug is hopefully fixed.

EDIT: It appears what I fixed was only part of the problem. Apparently there are also problems with DirectShow, which I haven't looked at yet.

[foxyshadis]

If you want I'll go ahead and do that tomorrow then, I wasn't sure what you were going to decide so I held off. I can set it to look for settings.xml in the MeGUI folder first, then the user's folder. Hm. Have it check for a <UseUserSettings>0</> tag and continue on to load/save to the user profile if not found? That way it can automatically propagate seamlessly unless you have the magic word.

I'm not expert in thinking about these issues, obviously.

[fields_g]

1026 not on the update server yet? Would love to try to find more bugs for you. Thanks for cleaning out the bug tracker!

[Sharktooth]

Posted "Core auto-update does not work".
coz the core was not updated on the autoupdate...
however it's coming ;)

[Sharktooth]

i updated the changelog as follows:

0.2.6.1026
- (berrinam) Fixed #1746055 (Video profile selection error)
- (berrinam) Fixed #1749810 (For each level have max bitrate for main and high profiles)
- (berrinam) Fixed #1715064 (Crash under Vista when loading avisynth script)
and uploaded the new build on the autoupdate server.

[Bigmango]

coz the core was not updated on the autoupdate...
however it's coming ;)

Nope, still not working. :(

I was in v0.2.6.1024. Autoupdate has now downloaded core 0.2.6.1025 + libs 0.2.6.1026. Then it asks to restart, and asks for admin priviledge (good !).

then it restarts.... but still with v.0.2.6.1024, and it asks me to update again.... same loop.

[fields_g]

I work at an evil corporation that has imposed Active Directory over us employees. Anyway... I don't have admin rights or install rights permanently. I installed MeGUI and have been able to do the updates until now. I believe that this must have something to do with the above talk about vista asking for admin.

Error: Access to registry key 'HKEY_LOCAL_MACHINE\Software\MeGUI' denied.

Any hope?

[Sharktooth]

Nope, still not working. :(

I was in v0.2.6.1024. Autoupdate has now downloaded core 0.2.6.1025 + libs 0.2.6.1026. Then it asks to restart, and asks for admin priviledge (good !).

then it restarts.... but still with v.0.2.6.1024, and it asks me to update again.... same loop.
Completely uninstall MeGUI AND delete the installation folder then reinstall it. I'll post a .1026 installer in few minutes on MeGUI's SF project page.

[berrinam]

In SVN revision 269, I added a test suite using NUnit (http://www.nunit.org/), the idea being that a number of bug reports can be expressed as tests, and this way we can hope to avoid more regressions as we accumulate more and more tests. I hope no-one minds the NUnit dependency; it is very light-weight, as simply being able to compile it only requires a single 76KB dll, nunit.framework.dll, which is included in the SVN.

Note that, in order to compile the test suite, you need to run compile-all.bat once first.

To run the tests, you need to install NUnit. After compiling the testsuite project, you can then run the tests either by double-clicking on the compiled testsuite.dll (NUnit registers with the filetype *.dll) or by running the NUnit GUI and opening the testsuite.dll file. All the tests should pass, which means that they should all end up green.

To find out how to write tests, see the documentation, and also the test I've already written as an example.

If, for some reason, you don't like NUnit, we can always revert.

[Maccara]

In SVN revision 269, I added a test suite using NUnit (http://www.nunit.org/), the idea being that a number of bug reports can be expressed as tests, and this way we can hope to avoid more regressions as we accumulate more and more tests. I hope no-one minds the NUnit dependency; it is very light-weight, as simply being able to compile it only requires a single 76KB dll, nunit.framework.dll, which is included in the SVN.


Excellent. I'm a fan of JUnit/NUnit myself. If you're using VS for the development, I can also recommend TestDriven.NET (http://www.testdriven.net/) (supports NUnit amongst others) which makes it easier to run/prepare NUnit tests from VS GUI directly. Free for non-commercial use.
(some might prefer this over the barebones NUnit framework, so the transition wouldn't be so painful :))

[Theliel]

umm, no more crash under vista :)
thank berrinam, work fine now.

[Sharktooth]

0.2.6.1027
- (Kurtnoise) [AviSynth Creator] : fixed Save Event for Vista OS.
- (Kurtnoise) change constant variables to shut down the PC correctly.
- (Kurtnoise) Cosmetic - fixed typos.
- (Kurtnoise) [AudioEncodingComponent]: remove the tab selected instead of the last one (#1814772)
- (berrinam) Fix #1813777 (Autocrop problem)
- (berrinam) Fixed #1814193 (AutoEncode with >1 audio track -> each audio track muxed twice)

[Kurtnoise]

#1715064,
#1728897,
#1808882,
#1814766,
#1813769

can be closed imo. To clean the bugs tracker...;)

[berrinam]

Closed 1808882, 1814766, 1728897. As far as I'm aware, fixing the Vista MediaInfo crash didn't fix the Vista DirectShow problems: can someone verify or refute this?

And 1813769 hasn't been fixed yet, has it? I know it's a duplicate, but is that a problem?

[Sharktooth]

what about #1816518. update works here...

[Kurtnoise]

As far as I'm aware, fixing the Vista MediaInfo crash didn't fix the Vista DirectShow problems: can someone verify or refute this?
I'll check this today...

And 1813769 hasn't been fixed yet, has it? I know it's a duplicate, but is that a problem?
it has been fixed...cannot reproduce the bug myself anymore.

what about #1816518. update works here...
That's strange indeed...I started to look at this yesterday but cannot find the bug/a fix.

[berrinam]

it has been fixed...cannot reproduce the bug myself anymore.Have you ever been able to reproduce it? I haven't, but I am pretty sure it was introduced in around 0.2.6.1018, and I'm not aware of any changes since then which would have fixed it, so I would expect it is still a problem.

[Bigmango]

Completely uninstall MeGUI AND delete the installation folder then reinstall it. I'll post a .1026 installer in few minutes on MeGUI's SF project page.

Ok, I have uninstalled megui and reinstalled a clean install of version 0.2.6.1026.

But now I can't test it anymore because the update versions are messed up. The latest versions are now older than the currently installed version:

- core: existing version: 0.2.6.1026 ; latest version: 0.2.5.107
- data: existing version: 0.2.4.1038 ; latest version: 0.2.4.1038

But in my previous install everything was at version 0.2.6.xxx.

So, bug #1813653 was closed; but what I see as a user is that the situation is even worse than it was before, because before the data was updating correctly to versions 0.2.6.xxx (or maybe it was 0.2.5, but definitely not 0.2.4) and the core was downloading new versions even though it failed to install them. But now it doesn't even see the new versions anymore, so I can't test if the core update works.

[Carpo]

http://mirror01.x264.nl/x264/megui-0.2.6.1022.exe - try that version and select the dev update server

[mini-moose]

hi

been playing around with megui recently and a few things I noticed/wanted to suggest in case they weren't brought up yet :

1) AVS script editor - I've loaded a .dv2 for an 50fps mpg and chose analyze.
the suggested solution was tdecimate(CycleR=2) which brings the encoded result to 30fps instead of 25fps as it should be from a pal source. tdecimate(cycle=2) would have achieved the correct result I believe.
On the main editor page I've selected PAL but I guess it doesn't influence the analyzing tool. I assume the analyzer is meant more for those who are not very experienced but thought I'd mention it anyway.

2) Bitrate Calculator/AutoEncode - one of the available sizes to choose from is 1/4 DVD / 1120mb. personally I liked the 1100mb on previous versions better as it leaves a little room for sizing errors.

3) Updates - on all the versions I've tried I always get an error when updating avc2avi : Error: Could not unzip avc2avi. Aborting...

4) MKV - something I thought might be useful for some who are used to virtualdub's "Save segmented avi" : maybe adding an option to split the encoded video file (before muxing i.e without audio) into parts. In other words have the video file output into smaller files while 2nd pass is ongoing.

5) splitting muxed file - I'm not sure if this is the place to suggest mkvmerge features or not but anyhow...I think it would be cool to have a splitting option that would not necessarily split the whole file into little bits but also an option to choose a certain segment to split from the full file. e.g 02:45 till 05:11. sometimes people might wanna save a certain clip from within a video.

I hope some of that makes sense and apologies if I have repeated issues discussed before or showed some lack of understanding of the tool's features. as I said I just recently started using it so I have not had the chance to explore it in full.

tnx

[Kurtnoise]

Have you ever been able to reproduce it? I haven't, but I am pretty sure it was introduced in around 0.2.6.1018, and I'm not aware of any changes since then which would have fixed it, so I would expect it is still a problem.
No, I'm not able to reproduce it. So...

1) AVS script editor - I've loaded a .dv2 for an 50fps mpg and chose analyze.
the suggested solution was tdecimate(CycleR=2) which brings the encoded result to 30fps instead of 25fps as it should be from a pal source. tdecimate(cycle=2) would have achieved the correct result I believe.
On the main editor page I've selected PAL but I guess it doesn't influence the analyzing tool. I assume the analyzer is meant more for those who are not very experienced but thought I'd mention it anyway.
Analysis tool is mainly dedicated to interlace stuff not framerate...

2) Bitrate Calculator/AutoEncode - one of the available sizes to choose from is 1/4 DVD / 1120mb. personally I liked the 1100mb on previous versions better as it leaves a little room for sizing errors.
Select Size...-> 1100 MB

3) Updates - on all the versions I've tried I always get an error when updating avc2avi : Error: Could not unzip avc2avi. Aborting...
yes, known issue.

4) MKV - something I thought might be useful for some who are used to virtualdub's "Save segmented avi" : maybe adding an option to split the encoded video file (before muxing i.e without audio) into parts. In other words have the video file output into smaller files while 2nd pass is ongoing.
what about splitting feature ?

5) I think it would be cool to have a splitting option that would not necessarily split the whole file into little bits but also an option to choose a certain segment to split from the full file. e.g 02:45 till 05:11. sometimes people might wanna save a certain clip from within a video.
ok...just put the request in SourceForge.

[Bigmango]

http://mirror01.x264.nl/x264/megui-0.2.6.1022.exe - try that version and select the dev update server


Arf, I forgot to select the dev update server after reinstalling :rolleyes:

The update works fine now :thanks:

[mini-moose]

Quote:
Originally Posted by "Kurtnoise13"
Analysis tool is mainly dedicated to interlace stuff not framerate...


just thought I would mention it cause it did suggest the wrong
command after analysing,

Quote:
Originally Posted by "Kurtnoise13"
what about splitting feature ?

the splitting feature seems to be only for the final muxed file unless I couldn't find a split feature for the vid only. what I mean is similar to the split feature in vdub. i.e one can set the output video file to be split while encoding and not while muxing.

[Sharktooth]

Ok, I have uninstalled megui and reinstalled a clean install of version 0.2.6.1026.

But now I can't test it anymore because the update versions are messed up. The latest versions are now older than the currently installed version:

- core: existing version: 0.2.6.1026 ; latest version: 0.2.5.107
- data: existing version: 0.2.4.1038 ; latest version: 0.2.4.1038

But in my previous install everything was at version 0.2.6.xxx.

So, bug #1813653 was closed; but what I see as a user is that the situation is even worse than it was before, because before the data was updating correctly to versions 0.2.6.xxx (or maybe it was 0.2.5, but definitely not 0.2.4) and the core was downloading new versions even though it failed to install them. But now it doesn't even see the new versions anymore, so I can't test if the core update works.

Data should be 0.2.4.1038. Everyone has "data" version 0.2.4.1038... coz it's the lastest. So everything is OK.

[Sharktooth]

3) Updates - on all the versions I've tried I always get an error when updating avc2avi : Error: Could not unzip avc2avi. Aborting...
Works for me.

[berrinam]

the splitting feature seems to be only for the final muxed file unless I couldn't find a split feature for the vid only. what I mean is similar to the split feature in vdub. i.e one can set the output video file to be split while encoding and not while muxing.Perhaps Tools->AVS Cutter is for you.

[Kurtnoise]

@Sharktooth: about update issues...I've made some tests from my own webpage with all packages from the main server and you know what ? it seems that renaming some packages, update reworks fine for me. By example, just renaming xvid_encraw-2007-07-31.zip to xvid_encraw.zip & modifying upgrade.xml file according to the new package and it works. Don't ask me why...:eek:


@all: an IFO parser could be great or not ? I started something from the Delphi IFO parser. Let me know if you're interested...

[Sharktooth]

@kurtnoise: does those problems happen only on Vista? have you tried reinstalling megui in a different folder?
an IFO parser could be good to "remove" the DVD Decrypter semi-dependancy in d2v creator.

[Kurtnoise]

does those problems happen only on Vista?
yes, I guess...

have you tried reinstalling megui in a different folder?
yes and same problems.


an IFO parser could be good to "remove" the DVD Decrypter semi-dependancy in d2v creator.
Right...In this case, only audio streams info is needed. In addition, could be useful also for Vobsubber tool but only subtitles streams info is required for that. So, coding only parts for Audios & Subtitles will be sufficient for the moment. However, a complete IFO parser will be more interesting in the future I think. ;) That's why I asked.

[Sharktooth]

@all: there's an update to sharpzlib: http://www.icsharpcode.net/OpenSource/SharpZipLib/Download.aspx
maybe it's worth doing the update. it may fix some problems... like the kurtnoise's ones.

[Sharktooth]

0.2.6.1028
- (Sharktooth) [Libs] Updated SharpZipLib (may fix some unzipping problems)
- (Kurtnoise) [Muxer] : output filename edition allowed.
- (Kurtnoise) [MP4 Muxer] : renable the tmp command correctly (#1795829)
- (Kurtnoise) [One Click] : few tunings
- (Kurtnoise) Cosmetics

[berrinam]

I hope I haven't caught you too late, but I've already done some work to support ifo parsing via vStrip. It calls vStrip.dll with PInvoke, and it seems to be working. I've used the current implementation to find out PGCs on a DVD, and I use this to automate running of DVDDecrypter. I don't know the relative advantages of different IFO parsers; I just used vStrip because I was able to link it to MeGUI most easily.

I've uploaded this code, updated to the latest version, and it is accessible as a branch here:

https://megui.svn.sourceforge.net/svnroot/megui/branches/vstrip

I have no objection to not using vStrip if you think it has problems, by the way.

[Sharktooth]

@devs: What about setting up a VPN (hamachi) between devs? We can chat, share info and sources, etc...

[berrinam]

I think it's about time that we branch megui so that we can start making some bigger changes again. What do you think?

Also, I think the branches are set up wrong. I, at least, have the entire repository as my working copy, so that I can play with all the other megui-related tools like MediaInfoWrapper, etc. I think we should rename 'trunk' to 'megui', and stick all the folders except 'branches' and 'tags' inside a newly-created 'trunk' folder, or something to a similar effect.

[Sharktooth]

i think we should rename trunk to Sources (or something) and move all folders except branches and tags into a new MeGUI folder.
something like:
root
|-branches
|-tags
|-MeGUI
|- bautodeint
|- MediaInfoWrapper
|- ...
|- Sources
however i would like to wait until all the bugs are fixed to make a sort of new start... once a stable version gets released we can mess with everything, including new stuff or the SVN tree structure.
releasing a stable version is the first priority so we can concentrate on design and new features implementation. we have just few bugs to fix...

[Kurtnoise]

I have no objection to not using vStrip if you think it has problems, by the way.
I don't think vStrip has issues and it has been tested a lot. So, why not after all. In fact, my main goal is to avoid using external dependancies. That's why I starded this...I'll look at this tonight if I've some time.

What about setting up a VPN (hamachi) between devs? We can chat, share info and sources, etc.
Well...I'll be much interested by an IRC chan or something coz I never tried/installed such tools.

I think the branches are set up wrong.
Indeed...I agree with Sharktooth to wait the last bugs fixing before to change something.

About SVN tree >> why not something like that :

MeGUI
|-branches
|-tags
|-trunk
|- bautodeint
|- MediaInfoWrapper
|- ...
|- Sources


That's the standard pattern iirc for the first nodes...

[Sharktooth]

Hamachi is a 1-click setup VPN... ;)
however if you prefer IRC we can register a MeGUI channel on freenode.

PS. I think the cause of #1816269 and #1813769 is the unnecessary "--split 0M" switch that gets in the commandline when splitting is not selected.

[berrinam]

I agree. But where does it come from? If you look at the MuxJob class, SplitSize is of type FileSize? -- a nullable type. It's either supposed to have a non-zero value or be null, and never supposed to have a value of zero. It's supposed to be impossible to even select a split size of zero, and when I try it, I can't -- if I enter 0, it just ignores me.

However, if you entered a split size of 1KB or even 1 byte, then it would result in --split 0M. I just played around with mkvmerge, and it only lets you enter split sizes greater than 1MB. I will put that restriction on MeGUI and hope it helps.

[Sharktooth]

ppl is weird an likes to experiment with stuff...

[Kurtnoise]

About update issues: now libs package is well unzipped but the problem remains with xvid_encraw and dgdecode. So, could you try to change/update your zip compressor and update those packages. Maybe, that should do the trick...

About IFO parsing: have a look at this (http://subtitlecreator.svn.sourceforge.net/viewvc/subtitlecreator/trunk/DVDinfo.cs?view=markup). Most of the code can be reused I think. Plus, that avoids some wasting time...:)

About VPN & IRC: well...I'll look at Hamachi but the channel IRC can be done first imo and if it's not enough, we can go to the VPN stuff. What do you think ?

[Sharktooth]

i used 7-zip to zip those files... and i doubt it has something to do with the zip format since here they got unzipped without any problems. however i'll replace those files...
about IFO parsing, yeah, that's almost what we need.
to register a IRC channel we need ppl...

EDIT: I replaced those files. Clear the cache directory and try again.

[Sharktooth]

0.2.6.1029
- (berrinam) Fix #1817211 (Bitrate calculator - MP4 container problem)
- (berrinam) Fix #1818125 (Unhandled exception on mux of x264 and neroaac into mkv)
- (berrinam) Fixed #1818131 (Error in computing resolution)
- (berrinam) Enforce minimum split size of 1MB; add assertions to help track down #1816269

[Kurtnoise]

EDIT: I replaced those files. Clear the cache directory and try again.
if you speak about the cache from my internet browser, the result is the same...:(



btw, I just added OS and Framework detection into the trunk. It's written in the log during the mainform loading. Could be useful for bugs reports I think...

[Sharktooth]

no.. i speak of the MeGUI cache directory. the "update_cache" folder into megui installation directory...

[dinolib]

For the update problem: I've solved the problem deleting the files of 0kb in the update_cache. The MeGUI downloaded again the files correctly.
There is any kind of problem with the update downloads?
Work-around: MeGUI should check the size of the files in the update_cache before unzip them (or use CRC).

[Taurus]

For the update problem: I've solved the problem deleting the files of 0kb in the update_cache. The MeGUI downloaded again the files correctly.
There is any kind of problem with the update downloads?
Work-around: MeGUI should check the size of the files in the update_cache before unzip them (or use CRC).
Thanks dinolib
This tip saves me hours of frustration.
Many of the updates downloaded correctly and show zero bytes. I guess the unpacker is behaving strange.
Sometimes it works, sometimes not:p

[berrinam]

MeGUI should delete files from the cache which update unsuccessfully.

[Taurus]

Some more observations:
There is no logic in the update failures.
Sometimes the very same file that gets corrupted at first try
is downloaded and unpacked nice at second attempt.
In case of mencoder it is about a 10 Mbytes repetition :(
The Xvid profiles got unpacked without the custom matrixes to "extras" folder, and so on.
So you can't be sure that everything is in the right place after updating and verifying each task after download is a pain in the a....
Yes, and my internet connection is very stable, no dropouts at downloads for a long time :confused:

[Sharktooth]

are you using Vista?

also the autoupdate server is getting overloaded... again... maybe that's the reason.

EDIT: it finally happened to me too. the 0 byte file doesnt get deleted. that means autoupdate doesnt delete the failed updates from the update_cache directory.
I submitted a bug report in the bug tracker.

[berrinam]

When I said "should," I really meant, "that would be a nice thing to have", not "I expect it to happen and I'm surprised it didn't"

Obviously unclear wording, sorry.

[Sharktooth]

i fixed the 0 byte thing by ignoring 0 byte files in update_cache directory but i just spotted another problem...
if the user hits the abort button while updating a file, an incomplete file gets cached... and that obvioulsy causes unzipping errors.
needs a fix...

[Taurus]

are you using Vista?

No, XP SP2

[Sharktooth]

oh... i forgot the 1030 changelog:
0.2.6.1030
- (berrinam) Forbid input and output filenames from being the same. (Should fix #1817324)
- (berrinam) Change AutoEncode default split size to "No splitting" and default target size to "1 CD." Hopefully fixes #1813769 and #1816269
- (Kurtnoise) Added OS and Framework detection.

[Kurtnoise]

hamachi installed...what's the next step now ? Create a new network I suppose ?

mail-me if you want: kurtnoise at free dot fr

[Sharktooth]

you got mail :)
however maybe it's better an IRC channel or just MSN. i just saw the linux version has no chat... what a shame...

[Kurtnoise]

\o/...I finally solved my update issues. Now, all seems to work flawlessly here on Vista. There were some conflicts between two update_cache folders content...OMG. :D I had 2 copies of megui svn trunk in my environment path with all content duplicated. That's why it didn't work. I removed the old one and...bingo. :stupid:

[berrinam]

Great! Now there are no bugs on the tracker that I can reproduce except the large fonts issue, which I don't know how to fix. Do you want to branch megui so we can start adding more features?

[Kurtnoise]

ok for me...but maybe, it's time to release a "stable" version first.


About large fonts issue: I'm trying something...I'll give you some feedback if it's ok or not.

Concerning #1795528 (http://sourceforge.net/tracker/index.php?func=detail&aid=1795528&group_id=156112&atid=798476)...it can be closed imo. It's not related to MeGUI and I can live without it.

About #1715064 (http://sourceforge.net/tracker/index.php?func=detail&aid=1715064&group_id=156112&atid=798476): I don't think it's related to MeGUI too...For what I've seen in the forum.

[Sharktooth]

both closed. what about #1811644 (http://sourceforge.net/tracker/index.php?func=detail&aid=1811644&group_id=156112&atid=798476) ? can you reproduce it?

[Kurtnoise]

cannot reproduce it here using the last revision...it works fine.

[Sharktooth]

so, once the muxing problems gets a confirmation they have been fixed, there are only 2 bugs left. the 120dpi fonts bug and a possible resource leak (#1816325 (http://sourceforge.net/tracker/index.php?func=detail&aid=1816325&group_id=156112&atid=798476)), unless the latter is a an external app (decoder, avisynth_MT or filters?) problem.

So, should we change the versioning for the upcoming stable release?

PS. There's still a bug with the updatecacher. the "aborted" files should be removed from the update_cache folder.

[berrinam]

But you've marked that in the tracker as fixed, haven't you?

[Sharktooth]

yup, i was pretty annoyed by the megui update problems kurtnoise had on vista that i just forgot to check for the "aborted" files (actually megui tries to delete and re-download 0 byte files... exceptions are not handled though...)... my bad.
on the other side, i have an idea in how to fix that but maybe it's not so good and quite complex. it requires flagging the aborted/incomplete files... if you have a simplier solution just let me know.

[berrinam]

I think I've got something that works quite simply. It's a bit clunky, as it requires the user to run update again, but at least it deletes the bad file. Fix committed in rev306.

[Sharktooth]

yep, it's ok for me. FlushFile() can be even reused in the updatecacher around line 70 but it can be just left as it is.
i just added the exception handling.

PS: Im updating the x264 encoder profiles and maybe i will change the profiles names.

[Kurtnoise]

About large fonts issue: I'm trying something...I'll give you some feedback if it's ok or not.
Not a bug in fact. I just moved to large fonts, updated each component forms to have a correct size (AutoSize=False was already correct), recompiled and it works fine with 120 DPI. Then I changed back the size (120 to 96), rebooted and...with normal size (96 DPI), forms are not scaled properly. So, I don't know what to do for the moment...

[Kurtnoise]

@berrinam: what do you think about a MSN or an IRC chan to chat between each others ?

[mini-moose]

the muxing tool's splitting-->select size... doesn't seem to be working well for me (at least). once I use the drop meu and choose "select size..." it's tab appears on my taskbar but it's not visible or usable for choosing.

also something I thought that can be nice is to add an option in the "settings" to set a different default container. default seems to be mp4. personally I'm using mkv only so everytime I launch megui I need to set it to that instead of mp4. same goes for the autoencode. even when the container is changed to a different one then mp4 it will still be mp4 on Autoencode (unlike on bitrate calculator). I will submit this on sourceforge.

[Sharktooth]

i just fixed the "Select size" dialog, but the next times, please report bugs on the MeGUI Bug Tracker.

[berrinam]

@berrinam: what do you think about a MSN or an IRC chan to chat between each others ?

Either's fine for me.

[Sharktooth]

irc.freenode.net #x264 until we have an official megui channel...

[Zulithe]

in MeGUI 0.2.6.1032, Tools menu -> AviSynth Script Creator, then for Video Input pick a file (any video file, i've tried probably a dozen different formats/codecs), meGUI closes and doesn't leave a log file as to why it closed.

I have AviSynth 2.5.7 (stable) installed in the default directory. Am I doing something wrong?

Using Vista Business 32bit.

[Sharktooth]

i suspect it's still a vista related problems since it works fine on XP...
could you try disabling the Preview from megui settings?

[Zulithe]

I went to settings -> Main tab ('other' subsection), unchecked "Open Preview after AviSynth script selection", clicked saved, exited MeGUI, restarted, reattempted, and it still closed with no error.
MeGUI Version: 0.2.6.1032
OS used: Microsoft Windows Vista
.Net Framework installed: 2.0.50727.312

[Sharktooth]

as you know, megui relys on directshow.
can you please grab graphedit and drop one of the files creating problems in it.
see if it plays correctly and post a screenshot of the graph.

[Zulithe]

http://i21.tinypic.com/316uid2.jpg

[Sharktooth]

does it play correctly in graphedit?
also, try using haali splitter instead of nero MP4 splitter...

[Zulithe]

Whew. Okay, installed haali splitter (version on the front page of their site), set it up to control .AVI files. here is graph edit now:
http://i21.tinypic.com/169rhgo.jpg
But megui still crashes the same...

[Sharktooth]

but does the file play correctly using graphedit?

[Zulithe]

Yes absolutely, it plays

*edit*
screenshot of it playing
http://i21.tinypic.com/icqoox.jpg

[Sharktooth]

as usual i forgot the changelogs
0.2.6.1031
- (berrinam) Fix #1817324 (Original audio is deleted when encoding fails)
- (Kurtnoise) cosmetic - remove an unused variable & ensure update_cache folder creation.
- (berrinam) Fix #1819627 (Analysis Pass broken)
- (Sharktooth) [AutoUpdate] Another try at fixing the update failures on Vista.
- (Sharktooth) Added MeGUI version to the Log
- (Sharktooth) [AutoUpdate] Fix #1819257 (Auto-update doesnt delete failed updates from update_cache). Now UpdateCacher ignores 0 bytes files.
- (berrinam) Fixed DAR detection for non-DVD DGIndex files.

0.2.6.1032
- (Sharktooth) [AutoUpdate] Added IOException handling when deleting 0 byte files.
- (Kurtnoise) [JobWorker] : Same separator width.
- (Kurtnoise) [Log] : Debug Infos were removed after jobs processing.
- (berrinam) [AutoUpdate] Fix #1819257 (Auto-update doesn't delete failed updates from update_cache)

0.2.6.1033
- (Sharktooth) [UpdateWindow] Fixed the enc_aacplus discrepancies.
- (Kurtnoise) [SettingsForm]: fixed a missing Click Action for avc2avi.
- (berrinam) Fix #1820411 (OneClick overcrop to mod16 doesn't appear to work)
- (Sharktooth) Fixed "Select Size" dialog appearing behind mux window.

[Sharktooth]

upgrading to 0.2.6.1033 may be problematic
please read this BEFORE upgrading: https://forum.doom9.org/showthread.php?p=1058892#post1058892

[Kurtnoise]

@devs: I've a patch for that (http://sourceforge.net/tracker/index.php?func=detail&aid=1820593&group_id=156112&atid=798479) but we need to create a fake file in the enc_aacplus package first (yeaaaaaaaah for Vista compatibility otherwise we get an access denied :devil:). What do you think ? useful or not ?

Index: core/gui/Form1.cs
===================================================================
--- core/gui/Form1.cs (revision 315)
+++ core/gui/Form1.cs (working copy)
@@ -1471,6 +1471,11 @@
updateCheck.Start();
}
Version ver = Environment.Version;
+ string ProgFilesFolder = Environment.GetFolderPath(Environment.SpecialFolder.ProgramFiles);
+ string encaacplusdll = ProgFilesFolder + "\\Winamp\\Plugins\\enc_aacplus.dll";
+ string nscrtdll = ProgFilesFolder + "\\Winamp\\nscrt.dll";
+ string Enc_AACPlusPath = Path.GetDirectoryName(settings.EncAacPlusPath);
+
logBuilder.AppendFormat("MeGUI Version: {0} {1}", Application.ProductVersion, Environment.NewLine);
logBuilder.AppendFormat("OS used: Microsoft {0} {1} {2} {3}", OSInfo.GetOSName(), OSInfo.GetOSProductType(), OSInfo.GetOSServicePack(), Environment.NewLine);
logBuilder.AppendFormat(".Net Framework installed: {0}{1}", ver, Environment.NewLine);
@@ -1478,6 +1483,12 @@
logBuilder.AppendFormat("------------------------------------------------------ {0}", Environment.NewLine);
logBuilder.AppendLine();
this.log.Text = logBuilder.ToString();
+
+ if (!File.Exists(settings.EncAacPlusPath.LastIndexOf(@"\") + "\\enc_aacplus.dll"))
+ {
+ if (File.Exists(encaacplusdll))
+ File.Copy(encaacplusdll, Enc_AACPlusPath + "\\enc_aacplus.dll", true);
+ }
}

private void beginUpdateCheck()

[Sharktooth]

Uhm... we need to get the winamp path from the registry.
we cant assume winamp is installed in "\Program files\Winamp"

EDIT: New bug... trying to open the bitrate calculator after loading an input file in the main form makes megui crash with a finite loop...
that happens if Acceptabe FPS rounding error is set to 0.

EDIT2: Fixed.

[Kurtnoise]

Uhm... we need to get the winamp path from the registry.
we cant assume winamp is installed in "\Program files\Winamp"
ok but how to retrieve the (Default) value ?

this :
public string WinampPath
{
get
{
Microsoft.Win32.RegistryKey key = Microsoft.Win32.Registry.CurrentUser.OpenSubKey(@"Software\Winamp");
if (key == null)
return null;
else
return (string)key.GetValue("(Default)");
}
}

returns nothing...and if I change the (Default) by something else, it works as excepted.

This is the only one key for Winamp in the Registry...

[Sharktooth]

there should be an uninstall key with the installation path... let me check.

EDIT: here HKEY_LOCAL_MACHINE\SOFTWARE\Microsoft\Windows\CurrentVersion\Uninstall\Winamp
there are 2 useful keys: UninstallString and DisplayIcon
those contain the installation path + some other data that can be stripped

[Maccara]

ok but how to retrieve the (Default) value ?


Before any "hacks" are done, normal way to do it is GetValue(null).

MS says for RegistryKey.GetValue Method:
A registry key can have one value that is not associated with any name. When this unnamed value is displayed in the registry editor, the string "(Default)" appears instead of a name. To retrieve this unnamed value, specify either a null reference (Nothing in Visual Basic) or the empty string ("") for name.

Edit: And I have software which uses this successfully, so I didn't just quote the docs, in case someone wonders if it might be buggy. ;)

[Kurtnoise]

thanks guys...

[Sharktooth]

3 bugs left... :)
the 120dpi font issue can be postponed after the "stable" release though...

just a curiosity... yesterday MeGUI surpassed the 600 downloads/day (sourceforge statistics (http://sourceforge.net/project/stats/?group_id=156112&ugn=megui)). keeping that pace, it generates about 18Gigz/month of traffic on the SF mirrors... plus the near 3000 project page views...
i wonder how much traffic is generated monthly on the autoupdate server...

[Sharktooth]

@devs: aud-x support and aud-x encoder could be removed since they have some problems (https://forum.doom9.org/showthread.php?p=1058935#post1058935). we should replace it with another encoder (mp3surround?).

[Sharktooth]

0.2.6.1034
- (Kurtnoise) [JobWorker] small glitch.
- (Kurtnoise) [AutoEncodeWindow] fixed #1821471.
- (Kurtnoise) [ScriptServer] removed temporarily SimpleResize (#1820850)
- (Kurtnoise) Fixed a recovery for the systray (#1820842).
- (Kurtnoise) [DGindexer] changed the delimiter character to avoid filename issues.
- (Sharktooth) Changed Log font to LucidaConsole (fixed width).
- (Sharktooth) Fixed #1820720 (Bitrate Cal. makes MeGUI crash). Acceptable FPS rounding error has now a minimum value of 0,001. Acceptable Aspect Error has a new default of 1% and a new max of 5%.
- (Sharktooth) MP3 Surround -> Aud-X (ISettingsProvider.cs).

[mroz]

Just wondering... would it be helpful if the default Megui program shortcuts contained a tools subfolder & therein links to run the appropriate bundled tools directly?

I occasionally need to dip into the programs folder to run stuff directly & I sometimes see user posts suggesting running tools directly to achieve certain tasks.

On a related note, the above seems to be the only way to allow AVIMuxGui to be configured. I use xvid mostly for standalones & they work best with no open dml & no rec lists, ime, so I have to reconfigure AMG.

A further aside: I (happily) see you're trying to include enough functionality in Megui to make it independent of external tools. Is this likely to include support for extraction of chapter info from ifos? I routinely use ChapterXtractor to allow chapter info to be embedded in mp4s.

[Kurtnoise]

@devs: aud-x support and aud-x encoder must be removed since they have legal issues for GPL violation (https://forum.doom9.org/showthread.php?p=1058935#post1058935). we should replace it ASAP with another encoder (mp3surround?).
speaking about it, I'll more or less interested by remove it completely. I think that mp3surround is restrictive. Look at this :

http://img180.imageshack.us/img180/4040/mp3sencvd7.th.png (http://img180.imageshack.us/my.php?image=mp3sencvd7.png)

You have to hit return to display the licence before to launch the compressor. :rolleyes:

Maybe using mp3sencoder.dll with our own cli tool could do the trick.

[Kurtnoise]

On a related note, the above seems to be the only way to allow AVIMuxGui to be configured. I use xvid mostly for standalones & they work best with no open dml & no rec lists, ime, so I have to reconfigure AMG.
No...if you already have it on your system, you just have to point MeGUI to the right path for this muxer.


A further aside: I (happily) see you're trying to include enough functionality in Megui to make it independent of external tools. Is this likely to include support for extraction of chapter info from ifos? I routinely use ChapterXtractor to allow chapter info to be embedded in mp4s.
It could be included as soon as we'll have our own IFO parser...

[Sharktooth]

Just wondering... would it be helpful if the default Megui program shortcuts contained a tools subfolder & therein links to run the appropriate bundled tools directly?

I occasionally need to dip into the programs folder to run stuff directly & I sometimes see user posts suggesting running tools directly to achieve certain tasks.
Could be done. I'll update the installer script as soon as i have some time.

[mroz]

@Kurtnoise: fair point re AMG if preinstalled, but still a problem if it isn't - unless you assume the default config is adequate for everyone. Nice to see an IFO parser is on the agenda.

@Sharktooth: cool :)

[COOLBGT]

Guys why did you removed the minimize to tray option in MeGUI. It was really useful an compact and i loved it

[Kurtnoise]

wtf...Can't see it ?

http://img139.imageshack.us/img139/5714/systraytt7.png

[mroz]

Guys why did you removed the minimize to tray option in MeGUI. It was really useful an compact and i loved it

You mean how minimise would automatically minimise to tray? There is still the explicit View->Minimise to tray menu entry. Is that not adequate?

[Sharktooth]

installer script updated. for the next release the installer will create links to tools, logs and auto-update cache folders.

[COOLBGT]

You mean how minimise would automatically minimise to tray? There is still the explicit View->Minimise to tray menu entry. Is that not adequate?
Ooops sorry. Didn't see that before :thanks:. Peace

[mroz]

installer script updated. for the next release the installer will create links to tools, logs and auto-update cache folders.

You guys almost make looking for bugs & usability issues fun :D

I've never seen a project where issues are so readily accepted or rapidly addressed. God knows where you find the time & energy. One happy user here :)

If I compare Megui with a well known commercial binary newsreader it's all the more sharp a contrast. Some projects ignore most bugs, have no idea how to triage & no interest in or concept of stable branches, instead relying on massive continuous code rewrites & updates to drown exisiting bugs, usually without success to no surprise.

:thanks:

[Sharktooth]

well, megui stalled several times too... we're just in a lucky period :)
however tonight im leaving for Lucca Comics and Games 2007 (http://www.luccacomicsandgames.com/page.php?page=7). So, i wont be here for a little while...

[Sharktooth]

@devs: to avoid #1822000 and for coherency, we should populate the profiles dropdowns (audio and video) with only the profiles belonging to the selected codecs.

[SpAwN_gUy]

@devs: to avoid #1822000 and for coherency, we should populate the profiles dropdowns (audio and video) with only the profiles belonging to the selected codecs.hm.. berrinam helped me with that thing.. (i guess my part will only be shown to public AFTER the Stable ;) ..)
it has to deal with Dictionary and IDictionary (the reason i can't merge my sources with latest.. :( ... 'coz i dunno how.. well i do,.. but it does not work..)

[Sharktooth]

uhm, well, it doesnt work since there were some major changes in the latest revisions...
maybe berrinam can help you porting your code to the latest megui version.

[SpAwN_gUy]

uhm, well, it doesnt work since there were some major changes in the latest revisions...
maybe berrinam can help you porting your code to the latest megui version.um.. yes he agreed with that.. some time ago..

[berrinam]

I don't have much time now, but in a week I will be able to help. And I agree about #1822000, the current behaviour is presently quite annoying and a pain to work with programatically. It definitely needs some looking at.

[Kurtnoise]

@devs: there is an issue with Avi-mux_GUI when the output and/or the temp paths contain special caracters (like é, à, è...). I reported the bug to Alexnoe but unfortunately he has no idea why it doesn't work properly for the moment. So, if you have some ideas to avoid this, let me know.

The most problematic is for the temp dir...


Note: it occurs only when we use the command line.

[superuser]

Today I updated MEGUI to the latest version 2.6.1034 from 2.5.1007. I am seeing suggested resolution resized in two versions to be different.

At present I was doing a source whose original resolution is 720*480 (appears to PAL->NTSC source). In latest MEGUI version Input DAR is reported as ITU 4:3; and suggested resolution for resize width 640 is 640*464. In previous version Megui gave suggested resolution as 640*480 for resized width of 640. I am confused here which one should be considered as correct one. Sorry if this has been already covered as I looked around but could not find what is the way to go with.

Thanks in advance for the help.

[Sharktooth]

the latest version has a more precise resizing.

[mroz]

/double checks

Wow, I didn't expect it to make that big a difference & thought something might be wrong but of course Sharktooth is correct.

The ITU DAR for 4:3 NTSC is 1.367088.

640/1.367088 = 468.148

So 464 is indeed the closest usable resolution.

BTW Sharktooth, I thought you were off to Lucca? Short visit? Hope it proved worthwhile. The festival url looked interesting :)

[Sharktooth]

There is definatly an internet connection at the ASUS stand...
Web surfing is not enabled but i love hamachi and adding routes...:D

[Eamon21]

Hey all, what's involved with adding/modifying code on this project? I want to add a new "tool" for transcoding that makes it much easier and batch-able. I'm part way through it already but figured that might as well offer it back into MeGUI if you guys want it?

The idea is to have a transcode profile that specifies a video profile, an audio profile, a container type and an AVISynth script template. Once it is set up, all you need to do is select a file(s) as input and the profile and it creates an avisynth script using the filename and template, and then does a basic transcode using the video and audio profiles specified. It is intended at this stage to be very simple with video and audio sources being the same avisynth script.

Was originally thinking that I could do this by modifying the "One Click Encode" (which really is not one click at all), but that seems very much geared towards encoding DVD's only and is certainly a lot more complex than I want or need. Maybe in time the functionality I am after could be incorporated into the oneclick tool, but at the moment it is too cluttered with excessive options and configuration to really be a "one click" solution, also it doesn't support batching.

This all started with me wanting to convert a lot of MOV files from my digital camera from MJPEG into H264 to save space, there are hundreds and i certainly don't want to have to do them one at a time.

Eamon

[Kurtnoise]

Glad you're volunteer to improve MeGUI...but I think there are some bugs to fix before to jump in the wind. :D

[mroz]

@Eamon21: I can't answer your question, being only a user, but I am interested in your modifications, as I'm in a similar situation (in my case, hundreds of mostly mpegs of 3 or 4 minutes duration with a few other formats thrown in to complicate the situation).

/lazy What tools are needed to compile Megui from source? Does one have to use Visual Studio? Is Express adequate? 2005 or the 2008 beta? I have a legit copy of 2005 Pro beta 2 - any good? My wife might have 2005 Pro (need to check).

I was thinking of just knocking a Perl script together to preprocess my sources & insert jobs into Megui's queue, but obviously a proper extension to Megui is much more worthwhile.

I don't know .net or C#, so there's only an outside chance I could contribute to coding - if you were interested - though I do find Megui an attractive project, so might use it to get into C# (anyone recommend some books?).

I did used to program extensively in C & ARM assembler, but that was a long time ago. Since then I've touched on OO concepts with a little bit of Java, but nothing to get my hands dirty (a Foundation degree involving networking & some programming/web design, but nothing of a standard to brag about).

Anyhow, what you're describing sounds like it meets most of my needs. I was also considering a prior stage - I'll need to run some analysis scripts to decide which AVS template to apply to each clip (at least to decide on whether deinterlacing is needed & possibly also to assess quality to select suitable denoising) or possibly to reject clips which don't fall into an obvious category, for manual checking. Would that be compatible with your ideas or is it more than you need?

At the very least I can help with testing your code, which hopefully will be of some use to you.

[Doom9]

You only need Visual Studio Express C# edition. The 2008 tools will also do if you keep the project to .NET 2.0 features only (finally in the 2008 edition you can turn on/off language features on a project level).

[Eamon21]

I realise that there are still bugs (aren't there always :)) but to be efficient and competent in tracking and fixing bugs I think I need to be more familiar with the code base which I hope to accomplish through developing a transcode tool. There are already a number of changes that I would like to make to various classes. The impact of these changes should not affect any existing code but all the same I am reluctant to stomp all over someone else's work. What is the process for submitting changes for review before they are incorporated into the main trunk?

Is there somewhere I can post changes that I think should be made and the reasons? I don't think they would belong in the "requested features" section as for the most part, they would not modify functionality, just re-factor the code and and add some best practices.

Eamon

[mroz]

You only need Visual Studio Express C# edition. The 2008 tools will also do if you keep the project to .NET 2.0 features only (finally in the 2008 edition you can turn on/off language features on a project level).

Cheers. Is that what most of the devs use? Seeing as if I explore this option I'll be installing only to build Megui it makes sense to stick to as similar a build environment as possible.

Don't suppose you have any suggestions for .net/C# books? I wasn't keen on .net/C# to begin with as I hate M$'s approach to most things & if I get into development again, I wanted it to be cross platform, but Mono offers some hope of that, so maybe C# is acceptable.

As an aside, I had been considering QT, but the last open source release was so delayed it virtually killed off Opie (http://opie.handhelds.org/cgi-bin/moin.cgi/WhatIsOpie) which I'd been following.

[cogman]

Where is the current source for megui? Right now the SVN is all the way back at 1028.

[Eamon21]

Anyhow, what you're describing sounds like it meets most of my needs. I was also considering a prior stage - I'll need to run some analysis scripts to decide which AVS template to apply to each clip (at least to decide on whether deinterlacing is needed & possibly also to assess quality to select suitable denoising) or possibly to reject clips which don't fall into an obvious category, for manual checking. Would that be compatible with your ideas or is it more than you need?

For my immediate needs this is probably more than I need, most of the movies I want to convert tend to be from the same source (digi cam) and I intended to use x264 in const. quality mode. The most I really need is to rotate by 90 degrees some of the movies, not sure how to automate a test for that though if it is even possible.

In some ways it also goes against what I am hoping to achieve here as well by making the process more complex . This could be largely mitigated through good UI design and well thought out defaults.

As I said earlier, part of my motivation here to create a new tool is the fact that the "One Click Encode" tool is not even close to one click and far too complex for a simple transcode operation. While MeGUI is quite powerful and I like the concept, it is not a very clean design and not very user friendly. I think this holds it back a great deal and as such I want to keep the transcode tool as "clean" and easy to use a possible.

Having said that, I certainly wouldn't rule out what you are after though I am not sure how it would work. It seems like it might be a very niche requirement and so might be better implemented in some scripting way, possible AVISynth? I'm not very familiar with the power of the AVISynth scripting language, but if what you are after was possible (in some part at least) in AVISynth then it would effectively work right in with what I am planning as I will take my input from an AVISynth template which could (possibly) contain your custom preprocessing.

Eamon

[mroz]

Where is the current source for megui? Right now the SVN is all the way back at 1028.

A very naive question coming up, so I might as well confess in advance to being ignorant about SVN as well as most things:

http://megui.svn.sourceforge.net/viewvc/megui/trunk/Changelog.txt?view=log shows the updates as far as 1034 eight days ago, so why do you say SVN only contains code to release 1028?

[cogman]

I'm not quite sure myself (Im not a SVN expert either). I just know that when I compiled and ran it, it reported itself as being the 1028 version. Im redownloading, it could be that I compiled it incorrectly.

[Sharktooth]

Is there somewhere I can post changes that I think should be made and the reasons? I don't think they would belong in the "requested features" section as for the most part, they would not modify functionality, just re-factor the code and and add some best practices.

Eamon
Yes, sure. The MeGUI project Patch Tracking System (http://sourceforge.net/tracker/?group_id=156112&atid=798478) on SF.
Before posting, ensure to create a patch (universal .diff file) against the latest SVN revision.