IRC log for #wowroster on 20080206

02:40.54*** join/#wowroster Boyo (n=Boyo@pool-71-244-155-139.chi.dsl-w.verizon.net)
02:45.25*** join/#wowroster poetter (n=poetter@e177172142.adsl.alicedsl.de)
07:19.50*** join/#wowroster Tupsi (n=Miranda@217.7.101.202)
14:07.32*** join/#wowroster Nefuh (n=nefuh@dslb-084-062-246-079.pools.arcor-ip.net)
15:11.59*** join/#wowroster Zanix|Work (n=Zanix@71.216.154.51)
15:11.59*** mode/#wowroster [+o Zanix|Work] by ChanServ
15:15.58*** join/#wowroster Nefuh[alt] (n=nefuh@dslb-084-062-224-018.pools.arcor-ip.net)
15:35.57*** join/#wowroster Gaxme (n=chatzill@64.247.200.42)
15:36.21GaxmeHitting an interesting difference between 1.7.3 and 2.0 in uploading larger files.
15:37.17GaxmeI've got a 3115 KB file (Raidtracker data) that I'm uploading to both. 1.7.3 has problems processing it due to allowed execution time, but 2.0 bombs out of allowed memory size long before that.
15:37.29GaxmeI've tried gzipping the data to ease its entry, and still receive the error.
15:37.58GaxmeI've upped the limit to 50M for testing, and it's still exhausting it.
15:38.24GaxmeNot sure whether there's a leak in the function lib, or whether 3meg files are just too much for the new system.
15:42.23GaxmeOr... Potentially I'm an idiot.
15:42.28GaxmeAnd it doesn't handle gz files.
15:43.23GaxmeNope, still bombs. Hrm.
16:43.01Zanix|Workhmmm
16:43.12Zanix|Workthe lua parser was optimized for 2.0
16:43.40Zanix|Workit doesnt convert the file to an array saving a bunch of memory
16:43.55Zanix|Workdo you know where it dies?
16:54.08*** join/#wowroster Zanix-Work (n=Zanix@71.216.154.51)
16:54.36*** mode/#wowroster [+o Zanix|Work] by ChanServ
16:54.54Zanix|Workstupid internet
16:55.05Zanix|WorkGaxme: did you get what I said
16:55.30GaxmeJust saw it, yeah
16:55.42GaxmeIt's actually giving an error from the function lib.
16:55.50GaxmeLemme run it again
16:57.52GaxmeFatal error: Allowed memory size of 52428800 bytes exhausted (tried to allocate 9022248 bytes) in /home/publicface/www/vestra/outside/roster20/lib/functions.lib.php on line 839
16:58.05GaxmeDoesn't make a ton of sense to me where it's erroring
16:58.12GaxmeThat's with the .gz
16:58.48GaxmeSame on the non-gzip
16:58.57GaxmeAnd nothing showing in the error log
17:00.52Zanix|Workhmmm
17:01.18Zanix|Workusing svn 1630?
17:01.23Zanix|Workor was that 1631
17:01.32Gaxme1627
17:01.34GaxmeLemme update
17:01.40Zanix|Workok, still the same file
17:01.49Zanix|Workno changes were made to the update process
17:01.57GaxmeHrm, I lied. 1631
17:02.01Zanix|Worklol
17:02.05GaxmeThe footer just isn't changed.
17:02.15GaxmeOr... Hrm, it's cached.
17:02.19GaxmeThere we go
17:02.24Zanix|Work??
17:02.25Zanix|Workfunction scrollbox(
17:02.31Zanix|Workis line 833
17:02.45GaxmeYeah, strange place to error.
17:02.55GaxmeUnless it's bombing on putting the data from the output into the srollbox
17:02.58GaxmeErr, scrollbox
17:03.05GaxmeI hadn't considered that. Let me turn off some debugging.
17:03.36Zanix|Workyeah
17:03.39Zanix|Workthat might do it
17:03.51GaxmeYep, that was definitely it.
17:04.03GaxmeBah, so I flooded the scroll box with too much data.
17:04.07GaxmeSorry for the fire alarm. =\
17:04.13Zanix|Worklol
17:04.16Zanix|Workits ok
17:04.37GaxmeFresh sets of eyes, I swear.
17:04.43Zanix|WorkI know what you mean
17:05.34GaxmeWell, at least it was an easy fix
17:05.37GaxmeHeh
17:07.06Zanix|Workindeed
17:07.28Zanix|WorkI just saw that as weird because the lua parser was greatly optimized
17:08.43GaxmeYeah, I guess it has, because that loaded a LOT faster than I expected 3 meg to. =)
17:08.43Zanix|Work"Edited the lua parser to read the lines directly from the file rather than caching it into memory (pleegwat)
17:08.43Zanix|Work<PROTECTED>
17:09.13Zanix|Workand memory usage is exponential with the old method we found out
17:09.22Zanix|Workso on larger files, it saved more memory
17:09.52GaxmeAnything that streamlines it is good, imo.
17:10.09GaxmeAnd it reads gzip, which makes it unnecessary to shrink in-game data sets.
17:10.58GaxmeWell, ok, LESS necessary. =)
17:13.45Zanix|Workyeah
17:14.15Zanix|Workand the new upload system can take a file without needing to upload CP.lua as well
17:15.15Zanix|Workby using the special hook update()
17:17.05Zanix|Workif you havent noticed
17:22.52GaxmeI did notice that. It's definitely a nice update.
17:23.03GaxmeI do need to get the raidtracker folks to modify their addon, though.
17:23.27GaxmeI'd really like to be able to pull player/realm/region info out of the data.
17:23.42GaxmeOtherwise there'll have to be some other mechanism for storing and displaying based on realm.
17:23.53GaxmeIt's not a big deal to sites with one guild
17:24.05GaxmeBut if you're running multiples, that's going to be useful.
17:34.23Zanix|Workyeah
17:34.27Zanix|WorkI know what you mean
18:04.50GaxmeExcellent, Eris is going to add char/player/realm info to CT_Raidtracker for me. =)
18:08.51Zanix|Worknifty
18:22.12Nefuh@Zanix iīve installed the new rosterfiles (get the latest from SVN) and it seems to i have found some bugs: the installer says ROSTER_PATH is not defined (but installer will work) and on siggen i must inserted $roster_login = new RosterLogin(); in the addons/siggen/admin/index.php on line 476 to get it functional.
18:22.38Zanix|Workyeah
18:22.48Zanix|WorkI updated siggen in the addon svn
18:23.01Zanix|Workand this is on a standalone install right?
18:23.15Nefuhyes
18:23.45Nefuhi made a clean standalone version to code some integration for Woltlab Burning Board for Luciver
18:24.57Zanix|Workok
18:25.22Zanix|Workdoes it say what file the error is in?
18:25.30Zanix|Work"ROSTER_PATH is not defined"
18:25.58NefuhYeah template.php on line 149
18:26.47Zanix|Workahhh
18:27.36Zanix|Workhmmm
18:27.37Zanix|Workok
18:27.49Zanix|WorkIll have to define it in the installer
18:28.05Zanix|Workand since the installer SHOULD only be used in the standalone, we should be ok
18:29.26Nefuhon com_wowroster i didnīt need the installer, this part will Joomla do ;) I donīt know if DragonFly needs it
18:29.45Zanix|Workright
18:29.58Zanix|Worka port of roster really shouldnt use the default installer
18:30.07Zanix|Workthey should make one specific for that cms
18:30.21Zanix|Workrosterdf will have an installer made specifically for DF
18:30.47Zanix|Workthe installer assumes a standalone install
18:34.46Zanix|Workfix commited
18:34.55Zanix|Workpr you can just put this near the top
18:35.05Zanix|Workdefine('ROSTER_PATH','');
18:36.56Nefuhok

Generated by irclog2html.pl Modified by Tim Riker to work with infobot.