*** tpb has joined #tp | 00:00 | |
*** ChanServ sets mode: +o tpb | 00:00 | |
tansell | cherez, so have you actually started to get common stuff done yet? | 00:07 |
---|---|---|
tansell | cherez, if so you should upload work-in-progress stuff too | 00:07 |
cherez | tansell: Not much yet and I'm going to bed soon, so the new CL should be up tomorrow. | 00:10 |
Landon | jmtan: hey, I'm still working on collision stuff atm, but was wondering if you had any ideas about how to load new xml | 00:11 |
tansell | cherez, so what have you been doing all last week then? | 00:12 |
Landon | I tried some naive approaches, but they all ended up similar to Camera $cameraname exists, Scenemanager $smname exists | 00:13 |
jmtan | hey Landon | 00:13 |
jmtan | "they all ended up similar to Camera $cameraname exists, Scenemanager $smname exists" | 00:13 |
jmtan | what do you mean? | 00:13 |
jmtan | did you try building or looking for a cegui load file dialog box? | 00:14 |
Landon | I've found some file choosing stuff in the CEGUI wiki, but I haven't played with that yet | 00:15 |
tansell | Landon, it definately makes sense to take it on the command line too | 00:15 |
Landon | what I'm referring to is reinitializing the scenes, I would get plenty of complaints about the larger components (cameras, scene managers) already existing under one name | 00:15 |
Landon | tansell: with opts? I can easily do that, but the problems have been occuring with trying to load a new battle file after one has been played out | 00:16 |
tansell | Landon, so I want to be able to launch the battle viewer from the tpclient-pywx by giving it the xml file to load | 00:16 |
Landon | as a path? or stdin? | 00:17 |
tansell | path | 00:17 |
Landon | ok | 00:17 |
Landon | jmtan: oh nevermind, I see what was going on there, I'll need to reinitialize the battle stuff only but keep the scene stuff around, looks like I'll have to do a bit of moving around since the scene stuff is done alongside the battle specific stuff | 00:19 |
jmtan | Landon, you could try adding a deconstructor method which deletes all the stuff you create in BattleScene class | 00:20 |
jmtan | i guess decoupling the battle specific stuff would be a good idea too | 00:21 |
llnz | mithro: btw, mhilmi has parsek connected and logging into servers now (with the new protocol library and implementing a new eventloop adaptor) | 00:22 |
Landon | yeah, I should probably start by splitting some of this stuff into files >.> | 00:22 |
Landon | instead of keeping this nearly monolithic battleviewer.py | 00:22 |
Greywhind | tansell: well, i've gotten multiple files to download at once :) | 00:25 |
tansell | Greywhind, yay! | 00:26 |
Greywhind | tansell: i haven't tested it extensively enough yet to know if there are threading problems, but it looks pretty good so far | 00:26 |
tansell | Greywhind, so have you cleaned up media.py too? | 00:26 |
Greywhind | tansell: a little bit | 00:26 |
Greywhind | tansell: but i'm not sure how to do things like checksums | 00:26 |
Greywhind | tansell: i'll have to talk to you about that tomorrow, since it's getting pretty late | 00:27 |
tansell | look at hashlib | 00:27 |
Greywhind | ok | 00:27 |
Greywhind | the checksums were the main issue, right? | 00:27 |
tansell | and it didn't do proper path searching | 00:29 |
tansell | Greywhind, we do have irc logs for a reason | 00:30 |
Greywhind | tansell: true. | 00:32 |
Greywhind | tansell: well, i'm going to sleep | 00:32 |
llnz | bbl | 00:34 |
*** llnz has quit IRC | 00:34 | |
*** jmtan has quit IRC | 00:46 | |
*** DTRemenak has joined #tp | 01:02 | |
*** jnengland77 has quit IRC | 01:09 | |
*** Greywhind has quit IRC | 01:19 | |
*** llnz has joined #tp | 01:34 | |
*** jmtan has joined #tp | 01:39 | |
*** greenlion has joined #tp | 02:06 | |
*** greenlion has quit IRC | 03:05 | |
*** verhoevenv has joined #tp | 03:08 | |
*** llnz has quit IRC | 03:16 | |
*** llnz has joined #tp | 03:16 | |
CIA-28 | landon tpclient-pyogre * r854abcc917bb /src/battleviewer.py: Spread participants out a bit so the collision detection would stop firing | 03:21 |
*** verhoevenv_ has joined #tp | 03:38 | |
*** verhoevenv has quit IRC | 03:39 | |
tansell | verhoevenv_, ping? | 03:40 |
verhoevenv_ | tansell: pong | 03:40 |
*** tansell_laptop has quit IRC | 03:51 | |
*** tansell--laptop has joined #tp | 03:51 | |
CIA-28 | landon tpclient-pyogre * r158dc39e5c66 /src/battleviewer.py: Added in collision detection code that will prevent dupes from appearing and also return a list of collisions instead of the error prone previous method of checking every object against the moving object | 03:58 |
Landon | tl;dr | 03:58 |
*** cherez has quit IRC | 04:43 | |
*** cherez has joined #tp | 04:43 | |
*** cherez has joined #tp | 04:44 | |
CIA-28 | landon tpclient-pyogre * r6fec767e41e0 /src/battleviewer.py: Added the -f/--file option to the command line arguments, but the file defaults to the example xml | 05:29 |
*** tansell--laptop has quit IRC | 06:00 | |
*** mithro has quit IRC | 06:09 | |
*** mithro has joined #tp | 06:48 | |
*** verhoevenv_ has quit IRC | 07:31 | |
* llnz wanders off | 07:56 | |
llnz | later all | 07:56 |
*** llnz has quit IRC | 07:56 | |
*** verhoevenv has joined #tp | 08:20 | |
*** alanp_ has joined #tp | 09:00 | |
*** jmtan has quit IRC | 09:16 | |
*** tuna-fish has quit IRC | 09:27 | |
*** jmtan has joined #tp | 10:40 | |
*** verhoevenv has quit IRC | 11:07 | |
*** verhoevenv has joined #tp | 11:07 | |
*** Greywhind has joined #tp | 11:15 | |
epyon | alanp_: pong! | 11:30 |
alanp_ | epyon: you mentioned that most rulesets have problems in the design, what kind of things are you seeing? | 11:31 |
epyon | Umm, #1 - I don't see the rulesets :P | 11:31 |
epyon | Instead I see a lot of copy pasted gibberish code | 11:31 |
epyon | Which could be automated into common interfaces | 11:32 |
epyon | So the module writer may just focus on writing the module itself, not figuringing out the API | 11:32 |
alanp_ | yeah | 11:32 |
epyon | or, god-forbid, mindless copy-pasting code from other rulesets | 11:33 |
epyon | And, BTW, something is really wrong with Parameter handling, both for objects and for orders (why are they separated anyway O.o) | 11:33 |
alanp_ | what's wrong with it? | 11:34 |
epyon | Hmm, would need more space to elaborate :P | 11:34 |
alanp_ | haha | 11:34 |
epyon | But the most broken thing I think is frame streaming | 11:35 |
epyon | But it would be a complete blowup to fix that -_- | 11:35 |
epyon | also, there's a to of unsegregated files because of that. | 11:40 |
epyon | I'd far more like to see a generic ordered Property mechanism | 11:41 |
epyon | Unfortunately I'm afraid that would break the protocol. | 11:41 |
alanp_ | are you doing any work on rulesets? | 11:42 |
*** jmtan has quit IRC | 12:53 | |
*** alanp_ has quit IRC | 13:48 | |
*** verhoevenv has quit IRC | 13:52 | |
*** verhoevenv has joined #tp | 14:03 | |
*** DTRemenak|RDP has joined #tp | 14:05 | |
*** DTRemenak|RDP is now known as Erroneous | 14:06 | |
*** epyon has quit IRC | 14:48 | |
*** alanp has quit IRC | 15:14 | |
*** alanp has joined #tp | 15:14 | |
*** alanp has quit IRC | 15:56 | |
*** alanp has joined #tp | 15:57 | |
cherez | git suddenly is very grumpy with me.... | 16:08 |
Landon | usually ends up better than svn being grumpy though | 16:09 |
*** llnz has joined #tp | 16:17 | |
llnz | morning all | 16:18 |
CIA-28 | snwallace libtpproto2-py * r4b3979440fb8 / (36 files in 7 dirs): Merged Parser and the Structures improvements into the main branch. | 16:19 |
CIA-28 | snwallace libtpproto2-py * r4a9e55060477 / (tests/testxstruct.py tp/netlib/xstruct.py): (log message trimmed) | 16:22 |
CIA-28 | Squashed commit of the following: | 16:22 |
CIA-28 | commit a00742ae343b82526ffb323488fb4256ac31ed1f | 16:22 |
CIA-28 | Author: cherez <[email protected]> | 16:22 |
CIA-28 | Date: Mon Jul 20 22:38:26 2009 +0000 | 16:22 |
CIA-28 | Fixed the problem with lists that contain only a single list. | 16:22 |
CIA-28 | commit daa8b613bc66903e31366d096ad3a04b8476af65 | 16:22 |
cherez | Morning. | 16:22 |
llnz | hi cherez | 16:22 |
cherez | Hey llnz. | 16:23 |
*** Greywhind has quit IRC | 17:46 | |
*** Greywhind has joined #tp | 17:55 | |
*** epyon has joined #tp | 17:59 | |
*** cherez has quit IRC | 18:11 | |
*** cherez has joined #tp | 18:14 | |
*** nash1 has joined #tp | 18:40 | |
*** nash1 is now known as nash | 18:41 | |
*** peres has joined #tp | 18:57 | |
*** bdefreese2 has joined #tp | 19:06 | |
*** bddebian has quit IRC | 19:06 | |
*** Greywhin1 has joined #tp | 19:22 | |
*** verhoevenv has quit IRC | 19:27 | |
*** Greywhind has quit IRC | 19:35 | |
*** peres has quit IRC | 19:39 | |
*** jnengland77 has joined #tp | 19:59 | |
*** mithro has quit IRC | 20:06 | |
tansell | morning | 20:13 |
*** mithro has joined #tp | 20:27 | |
Greywhin1 | tansell: morning | 20:38 |
Greywhin1 | tansell: so i've been working on checksum stuff | 20:39 |
Greywhin1 | tansell: but it seems like all the comparison checksums in the media.gz file are "None" | 20:39 |
cherez | Gah! | 20:43 |
cherez | Why do you do this to me, coverage tests?! | 20:44 |
tansell | Greywhin1, probably because we never generated them :) | 20:44 |
tansell | Greywhin1, I should we hard started doing md5sum and sh1sums | 20:44 |
tansell | but it appears not | 20:44 |
Greywhin1 | tansell: :P | 20:45 |
Greywhin1 | ok | 20:45 |
Greywhin1 | well, if they exist, it should check them now | 20:45 |
llnz | media-new.gz? | 20:46 |
Greywhin1 | MEDIA="media-new.gz" | 20:46 |
Greywhin1 | yeah | 20:46 |
llnz | ok | 20:47 |
tansell | Greywhin1, can you upload your WIP stuff so far? | 20:47 |
llnz | just checking | 20:47 |
tansell | I'm interested in making sure your going down the right track | 20:47 |
Greywhin1 | tansell: yeah | 20:47 |
Greywhin1 | i think i'll need a separate codereview for the libtpclient-py changes, yeah? | 20:48 |
tansell | http://svn.thousandparsec.net/cgi-bin/viewcvs.cgi/media/client/genmedia.sh?rev=18&view=markup | 20:49 |
tpb | <http://ln-s.net/3dvd> (at svn.thousandparsec.net) | 20:49 |
tansell | that's the script which generates it | 20:49 |
tansell | it could probably be replaced with a quick python script | 20:51 |
Greywhin1 | tansell: so if i make a separate branch on my local libtpclient-py for the code review, how can i get it not to switch to the old version when i run the client? | 20:51 |
tansell | a number of ways | 20:53 |
tansell | change requirements.py | 20:53 |
tansell | or commit a change in the top level to reference the new libtpclient-py branch | 20:54 |
Greywhin1 | tansell: i assume the requirements.py change would be easier to revert later? | 20:54 |
tansell | both should be relatively easy to revert | 20:55 |
Greywhin1 | tansell: what would the change be in requirements.py? | 20:55 |
tansell | it should be pretty obvious | 20:55 |
Greywhin1 | remove the git submodule update? | 20:56 |
cherez | tansell: Ping? | 20:58 |
tansell | cherez, pong | 20:58 |
tansell | Greywhin1, yeah | 20:58 |
cherez | tansell: I just noticed this line in ConnectionCommon._sendFrame: if packet.sequence == -1: | 20:59 |
Greywhin1 | tansell: http://codereview.mithis.com/10002 | 20:59 |
tpb | Title: Issue 10002: Added threaded downloading, fixed up code in media.py - Code Review (at codereview.mithis.com) | 20:59 |
Greywhin1 | for the libtpclient-py changes | 20:59 |
cherez | tansell: packet.sequence is unsigned, so that can never happen. | 20:59 |
tansell | so it should a SInt | 21:00 |
tansell | actually..... | 21:00 |
cherez | That's to determine if the sequence should be automatically generated. | 21:01 |
tansell | no I'm wrong | 21:01 |
tansell | cherez, yeah | 21:01 |
Greywhin1 | tansell: and this one: http://codereview.mithis.com/9001 | 21:01 |
tpb | Title: Issue 9001: Improved media support by centralizing media calls and adding a download manager. - Code Review (at codereview.mithis.com) | 21:01 |
tansell | cherez, figure out a good solution :) | 21:01 |
cherez | tansell: The solutions I see are make sequence semi-signed or change the magic number to 0 instead of -1, but I don't know if either change would have some unexpected influence on any older code. | 21:02 |
tansell | Greywhin1, couple of things | 21:02 |
*** Greywhin1 is now known as Greywhind | 21:03 | |
tansell | Greywhind, media.py shouldn't care about threading | 21:03 |
Greywhind | tansell: hm. i'm not sure how i can do that without making a separate Media object for each thread... | 21:05 |
tansell | btw you have read http://www.thousandparsec.net/wiki/Media.gz right? | 21:06 |
tpb | Title: Media.gz - Thousand Parsec Wiki (at www.thousandparsec.net) | 21:06 |
Greywhind | tansell: yes, but not in the past few days. i might need a refresher. | 21:07 |
tansell | Greywhind, wrap Media in thread.py? | 21:09 |
Greywhind | tansell: "wrap" it how? | 21:10 |
tansell | class ThreadedMedia(Media): | 21:10 |
Greywhind | ah. i thought that might have been what you meant | 21:10 |
Greywhind | yeah, maybe that would work | 21:10 |
tansell | why did you change the callback limiter? | 21:12 |
Greywhind | tansell: 5 seconds seemed too slow | 21:13 |
tansell | Greywhind, media.py is still mostly crap :P | 21:13 |
tansell | Greywhind, it's also not multiprocess safe | 21:14 |
Greywhind | tansell: i'm thinking about moving the "multiple attempts to get the same file at once" prevention to media.py, which should mostly fix that, yes? | 21:14 |
tansell | Greywhind, so the problem is what if daneel-ai starts downloading media | 21:17 |
tansell | you could have like 5 daneel-ai running | 21:17 |
tansell | but they all use the same media directories | 21:17 |
Greywhind | tansell: right | 21:17 |
tansell | so you need to figure out a way to solve that | 21:18 |
tansell | I think you should do things like | 21:19 |
tansell | change def media to be less magic | 21:20 |
Greywhind | tansell: it seems fairly comprehensible to me? | 21:26 |
tansell | Greywhind, but you call media and it magically loads the data and returns it | 21:27 |
Greywhind | tansell: i'm really sorry that i'm being so slow right now, but i just can't figure out what's wrong with this function | 21:36 |
Greywhind | is it the fact that it caches the result itself? | 21:36 |
tansell | Greywhind, it's the fact that sometimes it takes *forever* and sometimes it's instant | 21:37 |
Greywhind | tansell: ooooh. i see. | 21:37 |
tansell | which you need to assume it's going to take forever all the time | 21:44 |
tansell | Greywhind, so we where going to move the meeting right? | 21:50 |
Greywhind | tansell: yeah, 7 minutes | 21:52 |
Greywhind | let me write a summary | 21:52 |
tansell | so I have lunch in 7 minutes :P | 21:52 |
Greywhind | tansell: oh. alright. | 21:53 |
Greywhind | tansell: 1 hour later? | 21:53 |
tansell | we could do that if you can | 21:54 |
Greywhind | yeah, it'll work | 21:54 |
tansell | okay | 21:55 |
tansell | i'll be back in an hour | 21:55 |
Greywhind | j | 21:55 |
Greywhind | err | 21:55 |
Greywhind | k | 21:55 |
*** llnz2 has joined #tp | 22:43 | |
*** jmtan has joined #tp | 22:50 | |
tansell | Greywhind, ping? | 22:59 |
Greywhind | tansell: pong | 22:59 |
tansell | Greywhind, so | 22:59 |
*** llnz has quit IRC | 23:01 | |
tansell | Greywhind, so how goes things? | 23:01 |
Greywhind | tansell: pretty well, although i feel like i'm having to ask you too many questions about every step of the media stuff :P | 23:02 |
*** bdefreese2 is now known as bddebian | 23:08 | |
tansell | so with 3 weeks to go | 23:10 |
tansell | whats the plans? | 23:10 |
tansell | I think we should really go for a release | 23:11 |
Greywhind | tansell: i'm planning to spend next week finishing the media. | 23:11 |
Greywhind | tansell: after that, i am happy to do whatever final things you think are necessary to get ready to release it | 23:11 |
tansell | Greywhind, so there are a number of things we need to do | 23:12 |
tansell | Greywhind, make sure everything works on windows and mac | 23:12 |
Greywhind | tansell: ok | 23:15 |
Greywhind | tansell: what else? | 23:16 |
tansell | everything needs to be packaged up | 23:16 |
Greywhind | ok | 23:16 |
tansell | release notes need to be created | 23:17 |
tansell | Greywhind, I would also like to make sure that mtsec will work properly | 23:17 |
tansell | even if we don't have single player support yet | 23:17 |
Greywhind | tansell: i will check, but i think it works | 23:17 |
tansell | Greywhind, you also want to make sure it's playable | 23:18 |
tansell | alanp, ping? | 23:18 |
Greywhind | yeah | 23:18 |
tansell | Greywhind, I would say it's going to take atleast a week to get the release ready to go | 23:19 |
Greywhind | tansell: yeah, probably | 23:19 |
Greywhind | tansell: i'll start on it immediately after this week | 23:19 |
tansell | Greywhind, did you look into the order copy/paste/cut stuff? | 23:19 |
alanp | tansell: pong, just woke up | 23:19 |
Greywhind | tansell: well, everything put paste seemed to work | 23:19 |
alanp | (at 11:20PM :( ) | 23:19 |
Greywhind | tansell: i forget whether i looked into making paste work | 23:19 |
tansell | alanp, shouldn't you just be going to bed? | 23:19 |
alanp | tansell: yes, lol | 23:19 |
tansell | alanp, where is MTSec? | 23:19 |
alanp | bad sleeping patterns | 23:19 |
alanp | tansell: I need to finish Missile/ | 23:19 |
Greywhind | alanp: wow... that's crazy | 23:19 |
alanp | Missile/Torpedoe (today), and then combat tomorrow | 23:20 |
alanp | which will give me almost 3 weeks to complete it | 23:20 |
alanp | little behind sched, but I think it's going to be okay | 23:20 |
tansell | alanp, are you using the the tp04 version of the client? | 23:21 |
alanp | Greywhinds? I have to for the missile/torp stuff | 23:21 |
alanp | But mainly I've been using the stable client up to now | 23:21 |
tansell | alanp, so in theory we should be able to build missile/torpedos and ships? | 23:24 |
Greywhind | tansell: anything else you want to talk about? | 23:26 |
tansell | Greywhind, I can't think of anything | 23:26 |
Greywhind | tansell: ok | 23:26 |
tansell | Greywhind, I'm really happy of where we have gotten the client too | 23:26 |
tansell | Greywhind, oh - perspective saving/restoring is a feature we should really fix | 23:27 |
Greywhind | tansell: me too :) | 23:27 |
Greywhind | tansell: mm... i dunno how much i'll have time for, but i'll see what i can do | 23:27 |
tansell | Greywhind, it should be a couple of lines | 23:27 |
alanp | tansell: ships today, missile/torps tomorow :-) | 23:29 |
tansell | alanp, so you still can't build ships yet? | 23:29 |
alanp | no, i can build ships | 23:30 |
tansell | alanp, so ships can move about? | 23:32 |
tansell | (once built) | 23:32 |
alanp | yes | 23:32 |
alanp | but you just reminded me to shrink the size of the universe | 23:32 |
alanp | but yes | 23:32 |
tansell | so what is there left to do on the ships? | 23:32 |
alanp | hmmm | 23:32 |
alanp | i think ships are ok | 23:33 |
alanp | i can create a fleet and move it about | 23:33 |
*** llnz2 is now known as llnz | 23:34 | |
*** Greywhind has quit IRC | 23:40 | |
tansell | alanp, so why are torpedos for tommorrow? | 23:44 |
tansell | cherez, ping? | 23:48 |
cherez | tansell: Pong. | 23:48 |
tansell | cherez, so do you have anything outstanding I'm suppose to be reviewing? | 23:49 |
alanp | tansell: i need to finish the weapon order.. | 23:50 |
tansell | weapon order? as in the load/unload weapons? | 23:50 |
alanp | well we need an order on the planet to create the actual weapon no? | 23:50 |
*** bddebian has quit IRC | 23:51 | |
tansell | yeah | 23:51 |
tansell | so the Build Weapons? | 23:51 |
alanp | yes | 23:52 |
alanp | loading shouldn't be a problem | 23:52 |
cherez | tansell: Right now I'm fixing some code, but I can put up a CL with what I have so far. | 23:52 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!