Log in

View Full Version : MeGUI development


Pages : 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 [62] 63 64 65 66 67 68 69 70 71 72 73 74 75 76 77 78 79 80 81 82 83 84 85 86 87 88 89 90 91 92

Sharktooth
29th September 2007, 14:28
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
29th September 2007, 16:07
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
30th September 2007, 04:46
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
30th September 2007, 06:08
if megui is not updating, just remove and install the latest manually.

berrinam
30th September 2007, 06:26
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
30th September 2007, 08:14
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
30th September 2007, 13:13
yeah... damn microsoft... they like screwing everything...
i hope Vista dies as fast as it got cracked (2 days after the release...)

Sharktooth
30th September 2007, 13:20
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
30th September 2007, 13:51
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
30th September 2007, 16:17
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
1st October 2007, 00:48
202?!?

Kurtnoise
1st October 2007, 06:47
222...sorry.

Sharktooth
1st October 2007, 13:20
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
1st October 2007, 14:13
yes...no problem for what I've done with it.
have you tried it even with interlaced/telecined stuff?

Kurtnoise
1st October 2007, 16:17
yeah...works fine for me.

dimzon
1st October 2007, 19:10
Hi everyone!

dimzon
1st October 2007, 19:13
@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
1st October 2007, 20:17
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
1st October 2007, 23:14
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
2nd October 2007, 06:07
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
2nd October 2007, 11:10
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
2nd October 2007, 12:58
http://www.webalice.it/f.corriga/megui/MeGUI-sources.7z

berrinam
2nd October 2007, 13:07
And I've also sent it to you with yousendit.com. Ah well, the more, the merrier. :p

dimzon
2nd October 2007, 13:23
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
2nd October 2007, 20:45
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
2nd October 2007, 21:05
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
3rd October 2007, 06:51
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
3rd October 2007, 12:42
Bitrate calculator throws an exception if no input file is selected.
Commandline options are no longer shown in the codec config window.

Warpman
3rd October 2007, 13:54
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
3rd October 2007, 14:06
yeah.. i already posted the update window thing in the bugtracker

COOLBGT
3rd October 2007, 16:14
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
3rd October 2007, 16:40
eh... im not at home though... so hope some dev is around to fix it...

mroz
3rd October 2007, 19:39
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
4th October 2007, 03:08
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
4th October 2007, 04:14
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
4th October 2007, 06:32
you can also manually download core_0_2_6_1017.zip from http://megui.org/auto/ and put the contents in megui directory.

Adub
4th October 2007, 08:01
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
4th October 2007, 13:16
done.

COOLBGT
4th October 2007, 17:31
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
4th October 2007, 18:26
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
5th October 2007, 06:40
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
5th October 2007, 07:16
Curious, did you forget to commit something called MultiTrackComponent.cs this last rev?

berrinam
5th October 2007, 07:30
0.2.6.1020
- (berrinam) Support backwards-compatible loading of jobs from 0.2.6.1017. (But not vice-versa!)

berrinam
5th October 2007, 07:31
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
5th October 2007, 08:38
Hi,

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

berrinam
5th October 2007, 09:05
Added.

berrinam
5th October 2007, 10:29
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
5th October 2007, 11:17
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
5th October 2007, 12:57
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
5th October 2007, 15:59
Why did you guys made the same mistake with .1020 . Again all the muxing windows has the open button instead of the save button