MeGUI development

[Kurtnoise]

Originally Posted by Greif

What are the design goals for MeGUI?

none that I know of...

Originally Posted by Greif

Is there a roadmap for the project? Are there milestones?

no roadmap, no milestones clearly define. Just random ideas...

Originally Posted by Greif

I see that there is a Visual Studio 2008 solution in the trunk. Can the project be upgraded to .NET 3.5 and Visual Studio 2005 support be dropped?

iirc, that's been already discussed before. Yes, the project can be updated to the .NET 3.5 but it's limited to Win XP/Vista/Server2003-2008. So, to avoid restrictions, we use still the 2.xx. Anyway, I'm still open about this...

Originally Posted by Greif

How does one get svn commit access? I`d like to take a stab at some of the smaller bugs to get started with the project.

1/ Install TortoiseSVN
2/ Right click on a folder where you want to store the projet --> SVN checkout
3/ Copy this url of repository in the right place : https://megui.svn.sourceforge.net/sv...ui/megui/trunk
4/ Click ok
5/ That's all

The best way for reviews is to split big patches into small ones.

To create a patch:
1/ Select the main folder where the megui code is located.
2/ Right click on this folder --> TortoiseSVN --> Create Patch.

[Greif]

Originally Posted by Kurtnoise13

yes, it uses eac3to. Let me know if you're interesting by patches, although this part is not finished yet.

I love eac3to, I use it all the time. I can help with getting this incorporated into MeGUI.

You can look to some of the GUIs available for eac3to to see how some of the more advanced settings are exposed from a usability perspective.

[Greif]

Originally Posted by Kurtnoise13

iirc, that's been already discussed before. Yes, the project can be updated to the .NET 3.5 but it's limited to Win XP/Vista/Server2003-2008.

At this point, anyone running versions of Windows prior to XP shouldn't be using MeGUI.

[Kurtnoise]

Originally Posted by Greif

You can look to some of the GUIs available for eac3to to see how some of the more advanced settings are exposed from a usability perspective.

keep in mind that I don't want to extend all eac3to settings in this part. Just use it to demux BD & HD-DVD discs and applied gaps.

[Greif]

Yeah, I definitely agree, keep it simple.

80% of the users only use 20% of the features.

[Kurtnoise]

Originally Posted by hydro

Once you've added the second patches in 2141406 it should all (hopefully) be working

A problem remains for those who don't use the oneclick feature (like me ). In the d2v creator, by applying your patch, getDGindexLogInfo() is running at the end. However, the log file is not created there.

So, here is an idea:
1/ create a boolean to use the DGindex preview.
2/ Apply it to "true" when we run the oneclick form or to "false" when we run the d2v creator tool.
3/ Hence, using this bool value to raise getDGindexLogInfo().

What do you think ?


btw, all patches & fixes from you have been committed now except the log. So, many thanks to you.

[saint-francis]

Originally Posted by Kurtnoise13

keep in mind that I don't want to extend all eac3to settings in this part. Just use it to demux BD & HD-DVD discs and applied gaps.

Will eac3to be also added to the audio encoding options in MeGUI so we can transcode some of the new HD audio formats? I understand that you don't want to make MeGUI a full front end for eac3to but the original purpose of eac3to was to deal with the new audio formats. For example if one demuxes all of the tracks from a BD, re encodes the video with x264 and ends up with a 4-8 GB file and then has only a truehd audio track which could be the same size as the video.

I'm really excited about all of this.
Thank you all for you time and effort.

Edit: Are there any updates on being able to donate money to the project?

[Kurtnoise]

well...with the current build you can already transcode HD audio streams (except eac3 format).

[Greif]

Any thoughts around prompting EUL agreements from the UpdateWindow?

Looking towards adding NeroAacEnc as an available update.

https://sourceforge.net/tracker/inde...12&atid=798479

[Kurtnoise]

where licence.txt file is located in your example ? in our server ?



@saint-francis: I believe there is a link to donate on SF but admins project don't seem to activate it in the settings. Wait an answer from Sharktooth or Berrinam or Doom9 himself about that...

[Greif]

I was thinking if UpdateWindow was to display the EUL prior to downloading, the EUL text would have to be downloaded and prompted to the user.

<neroaacenc type="file">
<eulpath>[path to license]</eulpath>
<filepath version="1.3.3.0">NeroDigitalAudio.zip</filepath>
</neroaacenc>

The [path to license] in the above example would need to parse the EUL from http://www.nero.com/eng/downloads-ne...-aac-codec.php. Not very clean. Unless the EUL text can be stored seperately on MeGUI servers (ie NeroDigitalAudio_eul.txt).

In this second example, the EUL is contained within the download and it specified by the 'eul' attribute. The update is downloaded and the license is prompted to the user. If the user does not agree, then the files will be deleted.

<neroaacenc type="file">
<filepath version="1.3.3.0"
eul="license.txt">NeroDigitalAudio.zip</filepath>
</neroaacenc>

It all depends on how the program authors allow their application to be distributed.

[Greif]

In the case of Nero, they have a public anonymous FTP which they offer the code from. There is nothing requiring users to agree to an EUL prior to downloading from the FTP site.

I'm not a lawyer though, so I don't know if that is legit.

[Kurtnoise]

yep...it's the Sharktooth idea.

Patch welcome.

[Greif]

working on it

[hydro]

Originally Posted by Kurtnoise13

A problem remains for those who don't use the oneclick feature (like me ). In the d2v creator, by applying your patch, getDGindexLogInfo() is running at the end. However, the log file is not created there.

So, here is an idea:
1/ create a boolean to use the DGindex preview.
2/ Apply it to "true" when we run the oneclick form or to "false" when we run the d2v creator tool.
3/ Hence, using this bool value to raise getDGindexLogInfo().

What do you think ?

Although I fixed getDGindexLogInfo() I'm not actually using it anymore. Let me revert to the trunk version and I'll see what's going on.

btw, all patches & fixes from you have been committed now except the log. So, many thanks to you.

Your welcome

[hydro]

Originally Posted by Kurtnoise13

A problem remains for those who don't use the oneclick feature (like me ). In the d2v creator, by applying your patch, getDGindexLogInfo() is running at the end. However, the log file is not created there.

I'm confused. it isn't getting called anywhere in the trunk version:

Find all "getDGindexLogInfo", Subfolders, Find Results 1, "Entire Solution"
  C:\Users\aaa\Desktop\megui_trunk\Changelog.txt(21):    - (kurtnoise) [VideoUtil] modify trackID & Type in the getDGindexLogInfo() function. Patch by ateeq.
  C:\Users\aaa\Desktop\megui_trunk\core\util\VideoUtil.cs(245):        public void getDGindexLogInfo(string logFile, out List<AudioTrackInfo> audioTrackIDs)
  Matching lines: 2    Matching files: 2    Total files searched: 324

And the call is commented out in my backup copy/patch

Find all "getDGindexLogInfo", Subfolders, Find Results 1, "Entire Solution"
  C:\Users\aaa\Desktop\megui_oneclick_mediainfo\core\util\VideoUtil.cs(239):        public void getDGindexLogInfo(string logFile, out List<AudioTrackInfo> audioTrackIDs)
  C:\Users\aaa\Desktop\megui_oneclick_mediainfo\core\util\VideoUtil.cs(335):            //getDGindexLogInfo(Path.GetDirectoryName(projectSource) + "\\" + Path.GetFileNameWithoutExtension(projectSource) + ".log" , out list);
  Matching lines: 2    Matching files: 1    Total files searched: 324

It seems to be working ok for me and demuxes the audio (if I remember to select a track or select demux all) into the project directory. It even loads it into the main form if you tick the load box, however, I did notice that it doesn't automatically load the video. Did that break recently?

[Greif]

Originally Posted by Kurtnoise13

yep...it's the Sharktooth idea.

Patch welcome.

Got an unstable patch submitted: https://sourceforge.net/tracker/inde...12&atid=798478

I don't know why but the eul attribute isn't being read in properly. I"m not sure if I have the property being set in the correct spot.

I'm still trying to learn the solution architecture...

[Greif]

Got the patch working. The EUL is downloaded prior to the update. Either the EUL text file is stored on the application author's server or on MeGUIs.

Please verify the way I am handling the EUL is proper.

As a side note, I'd like to redo the way updates are managed.

[Kurtnoise]

Originally Posted by hydro

I'm confused. it isn't getting called anywhere in the trunk version

yes as I said, the DGLog part hasn't been committed yet. (Only on my local tree for testings).

Originally Posted by hydro

It seems to be working ok for me and demuxes the audio (if I remember to select a track or select demux all) into the project directory. It even loads it into the main form if you tick the load box, however, I did notice that it doesn't automatically load the video. Did that break recently?

I read this in your patch:

+            //AAA: DGLog: List<AudioTrackInfo> list;
+            //AAA: DGLog: getDGindexLogInfo(Path.GetDirectoryName(projectSource) + "\\" + Path.GetFileNameWithoutExtension(projectSource) + ".log" , out list);
+            //AAA: DGLog: for (int counter = 0; counter < list.Count; counter++)

So, I assumed that to use the DGLog part, we have to enable these lines. Am I right ?

[Kurtnoise]

Originally Posted by Greif

Got the patch working. The EUL is downloaded prior to the update. Either the EUL text file is stored on the application author's server or on MeGUIs.

Please verify the way I am handling the EUL is proper.

I'm reviewing it. Thanks...

edit: looks ok to me. I need to test it though and I'm waiting Sharktooth reply on this before to commit it.

Originally Posted by Greif

As a side note, I'd like to redo the way updates are managed.

please, elaborate...