Wednesday, 2009-07-22

*** tpb has joined #tp00:00
*** ChanServ sets mode: +o tpb00:00
tansellcherez, so have you actually started to get common stuff done yet?00:07
tansellcherez, if so you should upload work-in-progress stuff too00:07
chereztansell: Not much yet and I'm going to bed soon, so the new CL should be up tomorrow.00:10
Landonjmtan: hey, I'm still working on collision stuff atm, but was wondering if you had any ideas about how to load new xml00:11
tansellcherez, so what have you been doing all last week then?00:12
LandonI tried some naive approaches, but they all ended up similar to Camera $cameraname exists, Scenemanager $smname exists00:13
jmtanhey Landon00:13
jmtan"they all ended up similar to Camera $cameraname exists, Scenemanager $smname exists"00:13
jmtanwhat do you mean?00:13
jmtandid you try building or looking for a cegui load file dialog box?00:14
LandonI've found some file choosing stuff in the CEGUI wiki, but I haven't played with that yet00:15
tansellLandon, it definately makes sense to take it on the command line too00:15
Landonwhat 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 name00:15
Landontansell: 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 out00:16
tansellLandon, so I want to be able to launch the battle viewer from the tpclient-pywx by giving it the xml file to load00:16
Landonas a path? or stdin?00:17
tansellpath00:17
Landonok00:17
Landonjmtan: 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 stuff00:19
jmtanLandon, you could try adding a deconstructor method which deletes all the stuff you create in BattleScene class00:20
jmtani guess decoupling the battle specific stuff would be a good idea too00:21
llnzmithro: btw, mhilmi has parsek connected and logging into servers now (with the new protocol library and implementing a new eventloop adaptor)00:22
Landonyeah, I should probably start by splitting some of this stuff into files >.>00:22
Landoninstead of keeping this nearly monolithic battleviewer.py00:22
Greywhindtansell: well, i've gotten multiple files to download at once :)00:25
tansellGreywhind, yay!00:26
Greywhindtansell: i haven't tested it extensively enough yet to know if there are threading problems, but it looks pretty good so far00:26
tansellGreywhind, so have you cleaned up media.py too?00:26
Greywhindtansell: a little bit00:26
Greywhindtansell: but i'm not sure how to do things like checksums00:26
Greywhindtansell: i'll have to talk to you about that tomorrow, since it's getting pretty late00:27
tanselllook at hashlib00:27
Greywhindok00:27
Greywhindthe checksums were the main issue, right?00:27
tanselland it didn't do proper path searching00:29
tansellGreywhind, we do have irc logs for a reason00:30
Greywhindtansell: true.00:32
Greywhindtansell: well, i'm going to sleep00:32
llnzbbl00:34
*** llnz has quit IRC00:34
*** jmtan has quit IRC00:46
*** DTRemenak has joined #tp01:02
*** jnengland77 has quit IRC01:09
*** Greywhind has quit IRC01:19
*** llnz has joined #tp01:34
*** jmtan has joined #tp01:39
*** greenlion has joined #tp02:06
*** greenlion has quit IRC03:05
*** verhoevenv has joined #tp03:08
*** llnz has quit IRC03:16
*** llnz has joined #tp03:16
CIA-28landon tpclient-pyogre * r854abcc917bb /src/battleviewer.py: Spread participants out a bit so the collision detection would stop firing03:21
*** verhoevenv_ has joined #tp03:38
*** verhoevenv has quit IRC03:39
tansellverhoevenv_, ping?03:40
verhoevenv_tansell: pong03:40
*** tansell_laptop has quit IRC03:51
*** tansell--laptop has joined #tp03:51
CIA-28landon 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 object03:58
Landontl;dr03:58
*** cherez has quit IRC04:43
*** cherez has joined #tp04:43
*** cherez has joined #tp04:44
CIA-28landon tpclient-pyogre * r6fec767e41e0 /src/battleviewer.py: Added the -f/--file option to the command line arguments, but the file defaults to the example xml05:29
*** tansell--laptop has quit IRC06:00
*** mithro has quit IRC06:09
*** mithro has joined #tp06:48
*** verhoevenv_ has quit IRC07:31
* llnz wanders off07:56
llnzlater all07:56
*** llnz has quit IRC07:56
*** verhoevenv has joined #tp08:20
*** alanp_ has joined #tp09:00
*** jmtan has quit IRC09:16
*** tuna-fish has quit IRC09:27
*** jmtan has joined #tp10:40
*** verhoevenv has quit IRC11:07
*** verhoevenv has joined #tp11:07
*** Greywhind has joined #tp11:15
epyonalanp_: pong!11:30
alanp_epyon: you mentioned that most rulesets have problems in the design, what kind of things are you seeing?11:31
epyonUmm, #1 - I don't see the rulesets :P11:31
epyonInstead I see a lot of copy pasted gibberish code11:31
epyonWhich could be automated into common interfaces11:32
epyonSo the module writer may just focus on writing the module itself, not figuringing out the API11:32
alanp_yeah11:32
epyonor, god-forbid, mindless copy-pasting code from other rulesets11:33
epyonAnd, 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
epyonHmm, would need more space to elaborate :P11:34
alanp_haha11:34
epyonBut the most broken thing I think is frame streaming11:35
epyonBut it would be a complete blowup to fix that -_-11:35
epyonalso, there's a to of unsegregated files because of that.11:40
epyonI'd far more like to see a generic ordered Property mechanism11:41
epyonUnfortunately I'm afraid that would break the protocol.11:41
alanp_are you doing any work on rulesets?11:42
*** jmtan has quit IRC12:53
*** alanp_ has quit IRC13:48
*** verhoevenv has quit IRC13:52
*** verhoevenv has joined #tp14:03
*** DTRemenak|RDP has joined #tp14:05
*** DTRemenak|RDP is now known as Erroneous14:06
*** epyon has quit IRC14:48
*** alanp has quit IRC15:14
*** alanp has joined #tp15:14
*** alanp has quit IRC15:56
*** alanp has joined #tp15:57
cherezgit suddenly is very grumpy with me....16:08
Landonusually ends up better than svn being grumpy though16:09
*** llnz has joined #tp16:17
llnzmorning all16:18
CIA-28snwallace libtpproto2-py * r4b3979440fb8 / (36 files in 7 dirs): Merged Parser and the Structures improvements into the main branch.16:19
CIA-28snwallace libtpproto2-py * r4a9e55060477 / (tests/testxstruct.py tp/netlib/xstruct.py): (log message trimmed)16:22
CIA-28Squashed commit of the following:16:22
CIA-28commit a00742ae343b82526ffb323488fb4256ac31ed1f16:22
CIA-28Author: cherez <[email protected]>16:22
CIA-28Date: Mon Jul 20 22:38:26 2009 +000016:22
CIA-28Fixed the problem with lists that contain only a single list.16:22
CIA-28commit daa8b613bc66903e31366d096ad3a04b8476af6516:22
cherezMorning.16:22
llnzhi cherez16:22
cherezHey llnz.16:23
*** Greywhind has quit IRC17:46
*** Greywhind has joined #tp17:55
*** epyon has joined #tp17:59
*** cherez has quit IRC18:11
*** cherez has joined #tp18:14
*** nash1 has joined #tp18:40
*** nash1 is now known as nash18:41
*** peres has joined #tp18:57
*** bdefreese2 has joined #tp19:06
*** bddebian has quit IRC19:06
*** Greywhin1 has joined #tp19:22
*** verhoevenv has quit IRC19:27
*** Greywhind has quit IRC19:35
*** peres has quit IRC19:39
*** jnengland77 has joined #tp19:59
*** mithro has quit IRC20:06
tansellmorning20:13
*** mithro has joined #tp20:27
Greywhin1tansell: morning20:38
Greywhin1tansell: so i've been working on checksum stuff20:39
Greywhin1tansell: but it seems like all the comparison checksums in the media.gz file are "None"20:39
cherezGah!20:43
cherezWhy do you do this to me, coverage tests?!20:44
tansellGreywhin1, probably because we never generated them :)20:44
tansellGreywhin1, I should we hard started doing md5sum and sh1sums20:44
tansellbut it appears not20:44
Greywhin1tansell: :P20:45
Greywhin1ok20:45
Greywhin1well, if they exist, it should check them now20:45
llnzmedia-new.gz?20:46
Greywhin1MEDIA="media-new.gz"20:46
Greywhin1yeah20:46
llnzok20:47
tansellGreywhin1, can you upload your WIP stuff so far?20:47
llnzjust checking20:47
tansellI'm interested in making sure your going down the right track20:47
Greywhin1tansell: yeah20:47
Greywhin1i think i'll need a separate codereview for the libtpclient-py  changes, yeah?20:48
tansellhttp://svn.thousandparsec.net/cgi-bin/viewcvs.cgi/media/client/genmedia.sh?rev=18&view=markup20:49
tpb<http://ln-s.net/3dvd> (at svn.thousandparsec.net)20:49
tansellthat's the script which generates it20:49
tansellit could probably be replaced with a quick python script20:51
Greywhin1tansell: 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
tansella number of ways20:53
tansellchange requirements.py20:53
tansellor commit a change in the top level to reference the new libtpclient-py branch20:54
Greywhin1tansell: i assume the requirements.py change would be easier to revert later?20:54
tansellboth should be relatively easy to revert20:55
Greywhin1tansell: what would the change be in requirements.py?20:55
tansellit should be pretty obvious20:55
Greywhin1remove the git submodule update?20:56
chereztansell: Ping?20:58
tansellcherez, pong20:58
tansellGreywhin1, yeah20:58
chereztansell: I just noticed this line in ConnectionCommon._sendFrame: if packet.sequence == -1:20:59
Greywhin1tansell: http://codereview.mithis.com/1000220:59
tpbTitle: Issue 10002: Added threaded downloading, fixed up code in media.py - Code Review (at codereview.mithis.com)20:59
Greywhin1for the libtpclient-py changes20:59
chereztansell: packet.sequence is unsigned, so that can never happen.20:59
tansellso it should a SInt21:00
tansellactually.....21:00
cherezThat's to determine if the sequence should be automatically generated.21:01
tansellno I'm wrong21:01
tansellcherez, yeah21:01
Greywhin1tansell: and this one: http://codereview.mithis.com/900121:01
tpbTitle: Issue 9001: Improved media support by centralizing media calls and adding a download manager. - Code Review (at codereview.mithis.com)21:01
tansellcherez, figure out a good solution :)21:01
chereztansell: 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
tansellGreywhin1, couple of things21:02
*** Greywhin1 is now known as Greywhind21:03
tansellGreywhind, media.py shouldn't care about threading21:03
Greywhindtansell: hm. i'm not sure how i can do that without making a separate Media object for each thread...21:05
tansellbtw you have read http://www.thousandparsec.net/wiki/Media.gz right?21:06
tpbTitle: Media.gz - Thousand Parsec Wiki (at www.thousandparsec.net)21:06
Greywhindtansell: yes, but not in the past few days. i might need a refresher.21:07
tansellGreywhind, wrap Media in thread.py?21:09
Greywhindtansell: "wrap" it how?21:10
tansellclass ThreadedMedia(Media):21:10
Greywhindah. i thought that might have been what you meant21:10
Greywhindyeah, maybe that would work21:10
tansellwhy did you change the callback limiter?21:12
Greywhindtansell: 5 seconds seemed too slow21:13
tansellGreywhind, media.py is still mostly crap :P21:13
tansellGreywhind, it's also not multiprocess safe21:14
Greywhindtansell: 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
tansellGreywhind, so the problem is what if daneel-ai starts downloading media21:17
tansellyou could have like 5 daneel-ai running21:17
tansellbut they all use the same media directories21:17
Greywhindtansell: right21:17
tansellso you need to figure out a way to solve that21:18
tansellI think you should do things like21:19
tansellchange def media to be less magic21:20
Greywhindtansell: it seems fairly comprehensible to me?21:26
tansellGreywhind, but you call media and it magically loads the data and returns it21:27
Greywhindtansell: i'm really sorry that i'm being so slow right now, but i just can't figure out what's wrong with this function21:36
Greywhindis it the fact that it caches the result itself?21:36
tansellGreywhind, it's the fact that sometimes it takes *forever* and sometimes it's instant21:37
Greywhindtansell: ooooh. i see.21:37
tansellwhich you need to assume it's going to take forever all the time21:44
tansellGreywhind, so we where going to move the meeting right?21:50
Greywhindtansell: yeah, 7 minutes21:52
Greywhindlet me write a summary21:52
tansellso I have lunch in 7 minutes :P21:52
Greywhindtansell: oh. alright.21:53
Greywhindtansell: 1 hour later?21:53
tansellwe could do that if you can21:54
Greywhindyeah, it'll work21:54
tansellokay21:55
tanselli'll be back in an hour21:55
Greywhindj21:55
Greywhinderr21:55
Greywhindk21:55
*** llnz2 has joined #tp22:43
*** jmtan has joined #tp22:50
tansellGreywhind, ping?22:59
Greywhindtansell: pong22:59
tansellGreywhind, so22:59
*** llnz has quit IRC23:01
tansellGreywhind, so how goes things?23:01
Greywhindtansell: pretty well, although i feel like i'm having to ask you too many questions about every step of the media stuff :P23:02
*** bdefreese2 is now known as bddebian23:08
tansellso with 3 weeks to go23:10
tansellwhats the plans?23:10
tansellI think we should really go for a release23:11
Greywhindtansell: i'm planning to spend next week finishing the media.23:11
Greywhindtansell: after that, i am happy to do whatever final things you think are necessary to get ready to release it23:11
tansellGreywhind, so there are a number of things we need to do23:12
tansellGreywhind, make sure everything works on windows and mac23:12
Greywhindtansell: ok23:15
Greywhindtansell: what else?23:16
tanselleverything needs to be packaged up23:16
Greywhindok23:16
tansellrelease notes need to be created23:17
tansellGreywhind, I would also like to make sure that mtsec will work properly23:17
tanselleven if we don't have single player support yet23:17
Greywhindtansell: i will check, but i think it works23:17
tansellGreywhind, you also want to make sure it's playable23:18
tansellalanp, ping?23:18
Greywhindyeah23:18
tansellGreywhind, I would say it's going to take atleast a week to get the release ready to go23:19
Greywhindtansell: yeah, probably23:19
Greywhindtansell: i'll start on it immediately after this week23:19
tansellGreywhind, did you look into the order copy/paste/cut stuff?23:19
alanptansell: pong, just woke up23:19
Greywhindtansell: well, everything put paste seemed to work23:19
alanp(at 11:20PM :( )23:19
Greywhindtansell: i forget whether i looked into making paste work23:19
tansellalanp, shouldn't you just be going to bed?23:19
alanptansell: yes, lol23:19
tansellalanp, where is MTSec?23:19
alanpbad sleeping patterns23:19
alanptansell: I need to finish Missile/23:19
Greywhindalanp: wow... that's crazy23:19
alanpMissile/Torpedoe (today), and then combat tomorrow23:20
alanpwhich will give me almost 3 weeks to complete it23:20
alanplittle behind sched, but I think it's going to be okay23:20
tansellalanp, are you using the the tp04 version of the client?23:21
alanpGreywhinds?  I have to for the missile/torp stuff23:21
alanpBut mainly I've been using the stable client up to now23:21
tansellalanp, so in theory we should be able to build missile/torpedos and ships?23:24
Greywhindtansell: anything else you want to talk about?23:26
tansellGreywhind, I can't think of anything23:26
Greywhindtansell: ok23:26
tansellGreywhind, I'm really happy of where we have gotten the client too23:26
tansellGreywhind, oh - perspective saving/restoring is a feature we should really fix23:27
Greywhindtansell: me too :)23:27
Greywhindtansell: mm... i dunno how much i'll have time for, but i'll see what i can do23:27
tansellGreywhind, it should be a couple of lines23:27
alanptansell: ships today, missile/torps tomorow :-)23:29
tansellalanp, so you still can't build ships yet?23:29
alanpno, i can build ships23:30
tansellalanp, so ships can move about?23:32
tansell(once built)23:32
alanpyes23:32
alanpbut you just reminded me to shrink the size of the universe23:32
alanpbut yes23:32
tansellso what is there left to do on the ships?23:32
alanphmmm23:32
alanpi think ships are ok23:33
alanpi can create a fleet and move it about23:33
*** llnz2 is now known as llnz23:34
*** Greywhind has quit IRC23:40
tansellalanp, so why are torpedos for tommorrow?23:44
tansellcherez, ping?23:48
chereztansell: Pong.23:48
tansellcherez, so do you have anything outstanding I'm suppose to be reviewing?23:49
alanptansell: i need to finish the weapon order..23:50
tansellweapon order? as in the load/unload weapons?23:50
alanpwell we need an order on the planet to create the actual weapon no?23:50
*** bddebian has quit IRC23:51
tansellyeah23:51
tansellso the Build Weapons?23:51
alanpyes23:52
alanploading shouldn't be a problem23:52
chereztansell: 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/!