| *** tpb has joined #tp | 00:00 | |
| *** ChanServ sets mode: +o tpb | 00:00 | |
| Greywhind | tansell-laptop, llnz, you have time to talk? | 00:04 |
|---|---|---|
| tansell-laptop | in about 20 minutes | 00:04 |
| Greywhind | ok | 00:04 |
| llnz | going to be leaving in 20 minutes | 00:04 |
| Greywhind | :-/ | 00:05 |
| llnz | i'll be back about 45 minutes later | 00:05 |
| Greywhind | k | 00:06 |
| alanp | grrrr browser | 00:16 |
| alanp | hmm | 00:27 |
| llnz | bbl | 00:34 |
| *** llnz has quit IRC | 00:34 | |
| tansell-laptop | Greywhind, what did you want to talk about? | 00:38 |
| tansell-laptop | media stuff I'm assuming? | 00:38 |
| Greywhind | llnz isn't back yet :-/ | 01:25 |
| *** llnz has joined #tp | 01:32 | |
| * llnz is back | 01:34 | |
| Greywhind | hey llnz | 01:34 |
| Greywhind | still there tansell-laptop ? | 01:34 |
| tansell-laptop | not really | 01:35 |
| Greywhind | well, anyway, llnz - do you have any ideas? | 01:40 |
| llnz | well... i guess i have two | 01:42 |
| llnz | one is locks on files being download/written, the other is to avoid that by having a local media server | 01:42 |
| Greywhind | what would the local media server entail? | 01:43 |
| llnz | brb, swapping room, wireless might drop out | 01:43 |
| Greywhind | k | 01:44 |
| llnz | ok | 01:44 |
| llnz | well, it would act like a proxy server | 01:44 |
| Greywhind | i'm not terribly familiar with proxy servers | 01:48 |
| Greywhind | how do they work? | 01:48 |
| llnz | basically, it will provide the media straight away if it's avaliable locally, otherwise it will download it in the background and provide it when complete | 01:51 |
| Greywhind | mm. k. | 01:51 |
| llnz | it might not be a literal proxy server, it might talk a slightly different protocol | 01:51 |
| Greywhind | so each client on startup would have to check if it was running, if not, start it? | 01:51 |
| llnz | yes | 01:51 |
| llnz | which i guess would still require a lock file | 01:52 |
| tansell-laptop | I would recommend writing the file as | 01:52 |
| tansell-laptop | abc.<pidname> | 01:52 |
| llnz | tansell-laptop: normally the pid gets written into the file | 01:52 |
| tansell-laptop | no I mean write the file to that name | 01:52 |
| tansell-laptop | then do a rename at the last minute | 01:52 |
| Greywhind | well, i don't think this kind of thing is within the scope of what i could do now | 01:53 |
| Greywhind | maybe i could leave it for someone else to write | 01:53 |
| llnz | the other option is to have a lock file every media file, as needed | 01:54 |
| Greywhind | right | 01:54 |
| Greywhind | the question then, though, is what the client does when it sees a lock file | 01:54 |
| Greywhind | does it return None, wait until the lock disappears, or what? | 01:55 |
| llnz | it should wait, i think | 01:57 |
| Greywhind | probably | 01:58 |
| Greywhind | but then, how does it wait without busy looping to check whether the file is there? | 01:58 |
| llnz | check periodically | 01:58 |
| Greywhind | check, sleep 1 second, check? | 01:59 |
| Greywhind | that kind of thing? | 01:59 |
| llnz | check, set timer for 2-5 seconds time, return to event loop, ... timer expire, check... | 01:59 |
| llnz | yes | 02:00 |
| Greywhind | yeah | 02:00 |
| llnz | whatever is decided, it should be documented on the Media.gz page on the wiki | 02:04 |
| Greywhind | tansell-laptop: what do you think is the better or more reasonable option? | 02:05 |
| llnz | a proxy could act like a client and use the lock files as well | 02:06 |
| *** tansell-laptop has quit IRC | 02:16 | |
| *** tansell-laptop has joined #tp | 02:18 | |
| llnz | afk, dinner | 02:32 |
| llnz | back | 03:06 |
| *** tansell-laptop has quit IRC | 03:20 | |
| *** gau_veldt has joined #tp | 03:26 | |
| *** gau_veldt has left #tp | 03:26 | |
| *** nash has quit IRC | 03:27 | |
| *** tansell has joined #tp | 03:29 | |
| *** mithro_ has joined #tp | 03:40 | |
| *** Vadtec_ has quit IRC | 04:02 | |
| *** Vadtec has quit IRC | 04:02 | |
| *** edison has quit IRC | 04:02 | |
| *** Vadtec_ has joined #tp | 04:02 | |
| *** Vadtec has joined #tp | 04:02 | |
| *** edison has joined #tp | 04:02 | |
| *** Greywhind has quit IRC | 04:26 | |
| *** Demitar has quit IRC | 04:39 | |
| epyon | llnz, houston I have a problem | 05:42 |
| epyon | I guessed sow | 05:42 |
| epyon | arrr | 05:42 |
| epyon | typo | 05:43 |
| epyon | llnz: ping? | 05:43 |
| llnz | hi epyon | 05:43 |
| epyon | I have a problem with sequence number working | 05:43 |
| epyon | A sequence generates a modified list | 05:44 |
| epyon | but that list is stored on server | 05:44 |
| epyon | So if two players request modlists at the same time, the server will send them both Invalid Sequence number | 05:44 |
| epyon | in the second frame of the sequence | 05:45 |
| epyon | I assume that is a known bug? | 05:45 |
| llnz | no, not a known bug | 05:45 |
| llnz | almost all of the seq num for id list frames are per-user | 05:46 |
| epyon | Aaaaaaah, because that's a PlayerView! | 05:46 |
| llnz | those that aren't are generally fixed and only change when the underlying list changes | 05:46 |
| epyon | so it's per-player :D | 05:46 |
| *** mithro_ has quit IRC | 05:46 | |
| llnz | yes | 05:46 |
| epyon | Great, that clarifies things | 05:47 |
| epyon | Thanks :) | 05:47 |
| llnz | each player has their own playerview | 05:47 |
| llnz | no problem | 05:47 |
| *** mithro_ has joined #tp | 06:08 | |
| *** mithro_ has quit IRC | 06:27 | |
| *** mithro_ has joined #tp | 06:28 | |
| *** Demitar has joined #tp | 06:38 | |
| *** tuna-fish has joined #tp | 06:40 | |
| *** jmtan has joined #tp | 06:47 | |
| *** peres has joined #tp | 06:49 | |
| CIA-26 | epyon tpserver-cpp-refactor * rf0b1fb445bc5 /tpserver/ (playerview.cpp playerview.h): GenerateModList added to playerview EntityList | 06:54 |
| CIA-26 | epyon tpserver-cpp-refactor * r7604b99574b4 /tpserver/ (persistence.cpp persistence.h): Persistence::saveProtocolObject added | 06:54 |
| CIA-26 | epyon tpserver-cpp-refactor * rcaddb95f1b9f /tpserver/ (component.cpp component.h design.cpp design.h): Design and Component made ProtocolObjects | 06:55 |
| CIA-26 | epyon tpserver-cpp-refactor * r3780d2ddfd43 /tpserver/ (persistence.cpp persistence.h): | 06:55 |
| CIA-26 | Persistence ProtocolObject support updated | 06:55 |
| CIA-26 | * added retrieveProtocolObject | 06:55 |
| CIA-26 | * added getMaxProtocolObjectId | 06:55 |
| CIA-26 | * added getProtocolObjectIds | 06:55 |
| CIA-26 | epyon tpserver-cpp-refactor * rb0e59195f269 /tpserver/ (object.cpp object.h persistence.cpp): | 06:55 |
| CIA-26 | Object as ProtocolObject | 06:55 |
| CIA-26 | * In game Object is now a ProtocolObject | 06:55 |
| CIA-26 | * Removed assignment operator | 06:55 |
| CIA-26 | * added Persistence ProtocolObject support for Object | 06:55 |
| mithro_ | Landon: ping? | 07:20 |
| *** verhoevenv has joined #tp | 08:05 | |
| llnz | epyon: mysqlpersistence, tpmzscheme and avahi are broken currently | 08:30 |
| * llnz wanders off | 08:33 | |
| llnz | later all | 08:33 |
| *** llnz has quit IRC | 08:33 | |
| *** alanp_ has joined #tp | 08:41 | |
| *** verhoevenv has quit IRC | 09:22 | |
| *** verhoevenv has joined #tp | 09:33 | |
| *** alanp_ is now known as alanp- | 09:44 | |
| *** alanp_ has joined #tp | 09:44 | |
| *** alanp has quit IRC | 09:59 | |
| tote | when i apply a order to the cache, how can i send it to the server? cache.update doesnt seem to work | 10:26 |
| *** bddebian has joined #tp | 10:33 | |
| bddebian | Heya | 10:34 |
| *** peres has quit IRC | 10:41 | |
| tote | nvm, solved it | 11:01 |
| *** DTRemenak has quit IRC | 11:59 | |
| *** DTRemenak has joined #tp | 12:15 | |
| *** alanp- has quit IRC | 12:36 | |
| *** Greywhind has joined #tp | 13:05 | |
| *** Erroneous has joined #tp | 13:36 | |
| *** alanp has joined #tp | 14:27 | |
| Landon | mithro_: pong | 15:04 |
| *** alanp has quit IRC | 15:13 | |
| *** llnz has joined #tp | 16:04 | |
| *** alanp_ is now known as alanp | 16:38 | |
| *** verhoevenv has quit IRC | 16:57 | |
| *** verhoevenv has joined #tp | 17:06 | |
| *** bddebian has quit IRC | 17:18 | |
| llnz | morning all | 18:01 |
| cherez | Must... wake... up.... | 18:55 |
| epyon | arrr | 19:00 |
| *** kinglion has joined #tp | 19:04 | |
| *** kinglion has left #tp | 19:05 | |
| *** nash has joined #tp | 19:12 | |
| *** mithro_ has quit IRC | 19:37 | |
| *** mithro_ has joined #tp | 19:58 | |
| *** verhoevenv has quit IRC | 20:32 | |
| * llnz sighs | 20:40 | |
| cherez | Do you need encouragement, llnz? | 20:45 |
| llnz | thankfully it's lunch time, so no | 20:46 |
| tansell | cherez, pong? | 20:56 |
| tansell | Landon, pong? | 20:56 |
| Landon | gnop | 20:56 |
| tansell | Landon, so have you had a chance to catch up with jmtan? | 20:56 |
| Landon | not recently, no | 20:57 |
| tansell | well | 20:57 |
| cherez | tansell: I... don't remember pinging you? | 20:57 |
| cherez | Almost have my blog post up. | 20:57 |
| tansell | cherez, well I guess it should be a ping :) | 20:57 |
| cherez | My code is worthless today. :( | 20:58 |
| tansell | cherez, oh? | 21:12 |
| cherez | I think I'm sick; I've had periods staring at code for half an hour not sure how to continue. | 21:13 |
| tansell | cherez, sometimes it's just an idea to jump on IRC and chat with people about it | 21:13 |
| cherez | That is probably a good idea. | 21:14 |
| tansell | cherez, so it seems that you have been doing pretty will with the daily blog posts | 21:25 |
| tansell | so where are you at? | 21:26 |
| *** Erroneous has quit IRC | 21:26 | |
| Greywhind | tansell: i've updated issue 9001 and 10002 on codereview | 21:27 |
| tansell | link me? | 21:27 |
| Greywhind | http://codereview.mithis.com/9001 | 21:28 |
| tpb | Title: Issue 9001: Improved media support by centralizing media calls and adding a download manager. - Code Review (at codereview.mithis.com) | 21:28 |
| Greywhind | and the same with 10002 on it instead of 9001 | 21:28 |
| tansell | cherez, poke? | 21:30 |
| cherez | tansell: | 21:31 |
| tansell | cherez, so | 21:31 |
| tansell | where are you at? | 21:31 |
| cherez | Sorry, keyboard went out on me. | 21:31 |
| tansell | do you have a recently coverage report? | 21:31 |
| tansell | if you recall, I asked for you to start uploading coverage reports regularly | 21:32 |
| tansell | preferably in a way which means that I can compare the history | 21:33 |
| cherez | tansell: I'm uploading. | 21:33 |
| cherez | Alright, I'll put together a script to put them up with timestamps. | 21:34 |
| tansell | that would be great | 21:34 |
| tansell | it might be worth to go back and do it for older commits too | 21:34 |
| tansell | great way to show how you have progressed over GSoC | 21:35 |
| cherez | tansell: I'm a little lost on how to unit test the SSL wrapper. | 21:35 |
| tansell | cherez, oh? | 21:35 |
| tansell | cherez, might be worth doing something closer to an integration test | 21:36 |
| cherez | It supports I think 3 different SSL libraries as fallbacks, and I'm not too sure how I could set up a test for falling back correctly. | 21:36 |
| tansell | IE start up a test https server | 21:36 |
| tansell | s/https/ssl server/ | 21:36 |
| cherez | I'm also a bit uncertain about testing the socket wrapper. I just always feel that I might be overlooking some special case where they'll stop working. | 21:38 |
| tansell | well upload it to the code review and go "mithro can you see anything I'm missing" :) | 21:39 |
| tansell | and if I say no - then its fine right? :) | 21:39 |
| tansell | cherez, oh something else I wanted to mention | 21:40 |
| cherez | :) | 21:40 |
| cherez | I guess. | 21:40 |
| tansell | does your parser validate against the dtd? | 21:40 |
| cherez | No; I'm not really sure how to do that. | 21:40 |
| cherez | Should I be looking it up? | 21:40 |
| tansell | normally it's part of the xml parsing stuff | 21:41 |
| tansell | I would put it at a low priority thanks to your other testing | 21:41 |
| *** bddebian has joined #tp | 21:41 | |
| cherez | As long as I can't competently write any complex code today, that might be a good use of my time. | 21:42 |
| tansell | cherez, so your currently using figleaf to generate the coverage report right? | 21:43 |
| cherez | Yeah, it's proven more reliable than coverage. | 21:43 |
| tansell | but your test.sh in libtpproto2-py still uses coverage | 21:44 |
| cherez | Oh, I haven't uploaded the test.sh I'm using for it. | 21:44 |
| cherez | I'll put that in my next CL. | 21:44 |
| tansell | cherez, I would recommend putting in a separate CL | 21:45 |
| cherez | Alright. | 21:45 |
| tansell | so are we on-target to reach our goal? | 21:45 |
| cherez | I believe so. | 21:47 |
| tansell | Greywhind, do you know about mixin classes? | 21:47 |
| Greywhind | tansell: not really. | 21:47 |
| tansell | Greywhind, as well you didn't look into the for/else loop in python | 21:47 |
| tansell | Greywhind, they let you put common functionality into a class which you then inherit from | 21:48 |
| Greywhind | tansell: is there a difference from just a base class? | 21:49 |
| tansell | nope | 21:49 |
| tansell | except mixin's are limited to a very specific set to functionality | 21:49 |
| Greywhind | k | 21:50 |
| tansell | cherez, what is there still to do and how long have you set for each part? | 21:50 |
| tansell | Greywhind, the waiting stuff is a classic candidate for a mixin class | 21:50 |
| Greywhind | tansell: well, it's a bit different for each one though | 21:50 |
| tansell | Greywhind, is it? | 21:50 |
| cherez | tansell: Well, there's Client, Server, and integrating the library with libtpclient-py. | 21:51 |
| cherez | Client is pretty much written, so I expect it will be pretty methodical. | 21:51 |
| Greywhind | tansell: because some of them want everything for an object, some want everything for multiple objects, the info panel wants to track by URL | 21:52 |
| cherez | I'm expecting that to only be a few days. | 21:52 |
| tansell | cherez, so you have common done then? | 21:52 |
| tansell | Greywhind, why does the info panel want to track by URL? | 21:53 |
| Greywhind | tansell: because it takes images by URL rather than by object | 21:53 |
| tansell | hrm? | 21:54 |
| cherez | Functionally I just have a few bits to finish up for Common. It would probably be done if I I hadn't lost half my IQ the last few days. | 21:54 |
| tansell | Greywhind, that sounds wrong - everything should be looking at the media the same way | 21:55 |
| Greywhind | tansell: but it creates separate panels for each URL, so it separates it out into URLs first. | 21:55 |
| tansell | Greywhind, splitting on URL doesn't really make sense | 21:56 |
| Greywhind | def AddPanelForParam(self, group, parent): | 21:57 |
| Greywhind | elif isinstance(group, parameters.ObjectParamMedia): | 21:57 |
| Greywhind | it has to do it by URL, because it's creating panels by parameter | 21:57 |
| Greywhind | otherwise, it would need a separate construction to add the media panels | 21:58 |
| Greywhind | which is unnecessary | 21:58 |
| Greywhind | it still uses the same media code in threads.py | 21:58 |
| tansell | why is panelPicture still doing averaging? | 21:59 |
| Greywhind | tansell: it has a separate progress bar from the download window's progress bar. i should probably remove it now | 22:00 |
| tansell | Greywhind, yeah I think so | 22:00 |
| Greywhind | k | 22:00 |
| tansell | Greywhind, but the waiting on image always waits on a URL right? | 22:01 |
| Greywhind | tansell: which waiting on image? | 22:02 |
| Greywhind | tansell: oh, you mean the list of downloading images? | 22:02 |
| tansell | yes | 22:02 |
| Greywhind | tansell: yeah, it always contains URLs, just gets them from different places | 22:02 |
| tansell | Greywhind, so I think it still belongs in a mixin class | 22:03 |
| Greywhind | tansell: i'll look into it | 22:03 |
| *** mithro_ has quit IRC | 22:03 | |
| tansell | comments added too 9001 | 22:03 |
| Greywhind | ok | 22:03 |
| Greywhind | i'll be back | 22:03 |
| tansell | cherez, I recommend uploading the CL now | 22:04 |
| tansell | cherez, upload early and often :) | 22:04 |
| *** mithro_ has joined #tp | 22:04 | |
| shenki | some projects who have inferoir version control use rietveld for snapshotting their changes | 22:29 |
| shenki | s/inferoir/inferior/ | 22:29 |
| *** jmtan has quit IRC | 22:43 | |
| Landon | ho-boy, eyerest time oO | 23:13 |
| *** jmtan has joined #tp | 23:23 | |
| tansell | Landon, you still here? | 23:24 |
| Greywhind | tansell: http://codereview.mithis.com/8003/show | 23:34 |
| tpb | Title: Issue 8003: Made a popup for all SystemLevelOverlays containing panelPicture. - Code Review (at codereview.mithis.com) | 23:34 |
| tansell | Greywhind, is this the image popup? | 23:34 |
| Greywhind | yep | 23:35 |
| Greywhind | decided to fix them up real quick before going back to media stuff | 23:35 |
| tansell | got a new screenshot? | 23:40 |
| Greywhind | tansell: well, it's just the same with the panel either in the lower right or upper left corner | 23:42 |
| Greywhind | but i'll take two | 23:42 |
| tansell | Greywhind, oh? | 23:42 |
| Greywhind | http://img169.imageshack.us/img169/9146/screenshot2g.png | 23:44 |
| tpb | <http://ln-s.net/3ipD> (at img169.imageshack.us) | 23:44 |
| Greywhind | http://img259.imageshack.us/img259/2088/screenshot3v.png | 23:45 |
| tpb | <http://ln-s.net/3ipJ> (at img259.imageshack.us) | 23:45 |
| tansell | Greywhind, apparently jeffery signed on :) | 23:48 |
| Greywhind | tansell: heh | 23:48 |
| tansell | Greywhind, that looks okay | 23:49 |
| Greywhind | tansell: k | 23:50 |
| tansell | Greywhind, have you thought about adding tests for the media stuff? | 23:57 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
