| *** tpb has joined #tp | 00:00 | |
| *** ChanServ sets mode: +o tpb | 00:00 | |
| tansell | cherez, how did you clone? | 00:00 |
|---|---|---|
| tansell | git clone git+ssh://git.thousandparsec.net/git/libtpproto2-py.git | 00:00 |
| tansell | or | 00:00 |
| cherez | I'm guessing I did it from just the git:// | 00:00 |
| cherez | Shall I reclone? | 00:01 |
| tansell | no | 00:01 |
| tansell | just edit ~/.git/config | 00:01 |
| tansell | opps | 00:01 |
| tansell | that should be | 00:01 |
| tansell | ./.git/config | 00:01 |
| tansell | rather then ~/ | 00:01 |
| tansell | you might need to change it to git+ssh://username@ | 00:05 |
| tansell | depending on if your remote username matches your given one | 00:05 |
| tansell | s/given/local/ | 00:05 |
| tansell | you can check your ssh keys are working by doing a | 00:08 |
| tansell | ssh [email protected] | 00:08 |
| tansell | and see if you get the message "what do you think I am, a shell?" | 00:08 |
| cherez | Yeah. | 00:08 |
| cherez | So then it's authenticating right.... | 00:09 |
| cherez | Want me to pastebin my .git/config? | 00:09 |
| mithro | sure - but it should be pretty trivial to modify | 00:09 |
| mithro | just change the | 00:09 |
| mithro | url = git://git.thousandparsec.net/git/libtpproto2-py.git | 00:09 |
| mithro | line to | 00:09 |
| mithro | url = git+ssh://[email protected]/git/libtpproto2-py.git | 00:10 |
| cherez | Replacing "mithro" with my username? | 00:11 |
| tansell | yes | 00:12 |
| cherez | Then that's what it now says. | 00:12 |
| tansell | what does it say now? | 00:15 |
| cherez | $ git push | 00:16 |
| cherez | Everything up-to-date | 00:16 |
| tansell | cherez, have you actually committed your change? | 00:16 |
| cherez | ...Doh. :( | 00:16 |
| tansell | git status | 00:16 |
| greywhind_ | tansell: ok, i have a file called "objectutils.py", in extra, with the following function: | 00:17 |
| greywhind_ | http://pastebin.com/d260a2833 | 00:17 |
| CIA-46 | cherez libtpproto2-py * ra90235fb087a / (testxstruct.py tp/netlib/__init__.py tp/netlib/xstruct.py): | 00:17 |
| CIA-46 | Initial xstruct unit test. __init__.py now doesn't fail. xstruct now | 00:17 |
| CIA-46 | properly supports 'c' and 's'. | 00:17 |
| tpb | Title: pastebin - collaborative debugging tool (at pastebin.com) | 00:17 |
| greywhind_ | do you think that this is a reasonable solution, or should i think about it another way? | 00:18 |
| tansell | greywhind_, I would inverse a bunch of those tests to make it much more readable | 00:19 |
| greywhind_ | tansell: ok. but do you think the general idea is reasonable? | 00:20 |
| tansell | and that code looks wrong | 00:20 |
| greywhind_ | tansell: in what way? | 00:23 |
| greywhind_ | it seems to be working | 00:23 |
| tansell | there should only be one property which has a tuple of three values | 00:24 |
| greywhind_ | i thought there could be multiple positions? | 00:24 |
| tansell | greywhind_, yes there can be | 00:24 |
| tansell | and they will be named like | 00:24 |
| tansell | self.start = (10, 5, 6) | 00:25 |
| tansell | self.end = (100, 100, 5) | 00:25 |
| greywhind_ | it should pick up each of those with the for property in propertygroup.structures loop | 00:25 |
| greywhind_ | and add them all to a list of tuples | 00:26 |
| tansell | this part looks wrong | 00:26 |
| tansell | positionattrsstruct = getattr(obj, propertygroup.name) | 00:26 |
| tansell | positionvectorstruct = getattr(positionattrsstruct, property.name) | 00:26 |
| tansell | positionstruct = getattr(positionvectorstruct, property.structures[0].name) | 00:26 |
| tansell | xpos = getattr(positionstruct, property.structures[0].structures[0].name) | 00:26 |
| tansell | ypos = getattr(positionstruct, property.structures[0].structures[1].name) | 00:26 |
| tansell | zpos = getattr(positionstruct, property.structures[0].structures[2].name) | 00:26 |
| tansell | referenceobj = getattr(positionvectorstruct, property.structures[1].name) | 00:26 |
| greywhind_ | well, each ObjectParamPosition3d struct | 00:27 |
| greywhind_ | has a tuple with structs for: (x, y, z) and (reference object) | 00:27 |
| greywhind_ | so it's like this: ((x, y, z), reference object) | 00:28 |
| greywhind_ | so i get the first one, which contains the x, y, and z, and extract the three coordinates | 00:28 |
| greywhind_ | then i get the second one, and add the reference object | 00:28 |
| mithro | lass OrderParamAbsSpaceCoords(GroupStructure): | 00:28 |
| mithro | """\ | 00:28 |
| mithro | Coordinates in absolute space. (Relative to the center of the Universe) | 00:28 |
| mithro | """ | 00:28 |
| mithro | def __init__(self, *args, **kw): | 00:28 |
| mithro | kw['structures'] = [ | 00:28 |
| mithro | GroupStructure('coordinates', 'No description', structures=[ | 00:28 |
| mithro | IntegerStructure('x', 'No description', size=64, type='signed'), | 00:28 |
| mithro | IntegerStructure('y', 'No description', size=64, type='signed'), | 00:28 |
| mithro | IntegerStructure('z', 'No description', size=64, type='signed'), | 00:28 |
| mithro | ]), | 00:28 |
| mithro | ] | 00:28 |
| mithro | GroupStructure.__init__(self, *args, **kw) | 00:28 |
| greywhind_ | right. | 00:29 |
| mithro | just use the names x,y,z directly | 00:29 |
| greywhind_ | instead of structures[0].name, etc? | 00:29 |
| mithro | coords = getattr(obj, property.name).coordinates | 00:29 |
| greywhind_ | ah | 00:30 |
| mithro | position += [(coords.x, coords.y, coords.z)] | 00:30 |
| mithro | be back in 5 | 00:30 |
| greywhind_ | ok | 00:30 |
| greywhind_ | thanks | 00:30 |
| cherez | mithro: Any objection with be putting the tests in the topmost directory? | 00:33 |
| llnz | bbl | 00:35 |
| *** llnz has quit IRC | 00:35 | |
| *** tansell-laptop has joined #tp | 00:38 | |
| tansell-laptop | greywhind_, for hard-defined structures - you can just assume they are going to always be that way | 00:38 |
| greywhind_ | tansell-laptop: ok. | 00:39 |
| greywhind_ | i've got the code fixed up | 00:39 |
| tansell-laptop | I'm on a different computer so you'll have to link me again | 00:39 |
| greywhind_ | http://pastebin.com/d74d0be6a | 00:40 |
| tpb | Title: pastebin - collaborative debugging tool (at pastebin.com) | 00:40 |
| greywhind_ | tansell-laptop: do you think i should add the name of the coordinate set to the tuple? | 00:41 |
| tansell-laptop | you probably want a dictionary which is the name + coordinate or something | 00:51 |
| tansell-laptop | depends how you are going to use it | 00:51 |
| greywhind_ | mm... but i'm not at all sure how i'll be using it yet | 00:52 |
| greywhind_ | i guess i can always change it later | 00:52 |
| tansell-laptop | order probably matters | 00:55 |
| greywhind_ | i was thinking: | 00:56 |
| greywhind_ | Each tuple has the form: (x, y, z, reference object, coord name) | 00:56 |
| greywhind_ | so you can ignore everything after the first three if you don't care | 00:56 |
| greywhind_ | or the first 4 | 00:56 |
| tansell-laptop | are most things using a reference? | 00:58 |
| greywhind_ | tansell-laptop: seems like not in minisec at least | 01:16 |
| tansell-laptop | yeah coord references are pretty uncommon at the moment I think | 01:16 |
| greywhind_ | i guess i should deal with them though | 01:16 |
| tansell-laptop | greywhind_, it actually made me thing of a bug | 01:17 |
| greywhind_ | maybe i should deal with them inside that method, and return absolute coords? | 01:17 |
| *** greenlion has joined #tp | 01:25 | |
| tansell | greenlion, possibly | 01:43 |
| tansell | opps | 01:43 |
| tansell | which coordinate is the object relative two | 01:45 |
| tansell | s/two/too | 01:45 |
| tansell | greywhind_, you know what I mean? - IE if the parent has two coordinates | 01:45 |
| greywhind_ | tansell: it's not specified. | 01:46 |
| greywhind_ | i'm currently assuming it's the first one | 01:46 |
| *** llnz has joined #tp | 01:48 | |
| tansell | greywhind_, yeah - I would post to the list about that | 01:49 |
| greywhind_ | tansell: ok, will do | 01:50 |
| *** tansell-laptop has quit IRC | 01:51 | |
| greywhind_ | tansell: sent | 01:55 |
| CIA-46 | noegnud tpclient-pywx-tp04 * r0a5e48122493 / (extra/objectutils.py windows/main/overlays/Path.py): | 01:59 |
| CIA-46 | Added objectutils.py, which will hold functions that are useful in dealing with | 01:59 |
| CIA-46 | dynamic objects. The first function in this file gets the coordinate sets for an | 01:59 |
| CIA-46 | object. Also fixed the FindPath method in Path.py to use the new DynamicObject | 01:59 |
| CIA-46 | method of dealing with coordinates. | 01:59 |
| CIA-46 | noegnud tpclient-pywx-tp04 * rc1a3421ded76 /windows/main/overlays/Path.py: | 01:59 |
| CIA-46 | Fixed the other reference in the FindPath method to object positions to use the new | 01:59 |
| CIA-46 | method for finding the position of a DynamicObject. | 01:59 |
| tansell | greywhind_, you still here? | 02:18 |
| greywhind_ | tansell: yep | 02:18 |
| greywhind_ | tansell: just writing a blog post about today's work | 02:18 |
| tansell | greywhind_, I'm thinking that maybe we should move towards having a code review step before submitting | 02:19 |
| greywhind_ | tansell: sure. sorry if i screwed something up. | 02:19 |
| tansell | nah | 02:19 |
| tansell | it just make it much easier to go through the process we did today | 02:19 |
| greywhind_ | tansell: yeah | 02:19 |
| tansell | rather then using pastebin all the time | 02:19 |
| shenki | retvield is pretty cool once you get used to the workflow. but it's a fair bit of overhead for first time/one time patch submitters, so you'd only want to use it for patches that need discussion | 02:21 |
| tansell | http://src.chromium.org/svn/trunk/tools/depot_tools/README.git-cl.codereview | 02:21 |
| tpb | <http://ln-s.net/3ECy> (at src.chromium.org) | 02:21 |
| tansell | shenki, do you use git-cl? | 02:21 |
| shenki | tansell: ive done one patch with git cl, and one with gcl (the subversion one) | 02:21 |
| tansell | shenki, how was it? | 02:22 |
| shenki | gcl was eaiser, but perhaps it's because it's better documente,d and the projects actually use svn (you only do git by using git svn) | 02:22 |
| shenki | in general, it's well worth having. just have to get your devs to commit to it | 02:24 |
| shenki | having patches in codereview for extended amounts of time is offputting | 02:24 |
| greywhind_ | tansell: if you can tell me how to set it up, i'll use it | 02:26 |
| shenki | http://code.google.com/appengine/articles/rietveld.html | 02:27 |
| tpb | <http://ln-s.net/3ED4> (at code.google.com) | 02:27 |
| *** greenlion has quit IRC | 02:27 | |
| tansell | shenki, oh I totally agree | 02:27 |
| tansell | shenki, just reading that | 02:27 |
| mithro | I think I'll setup an instance on mithis.com for now | 02:34 |
| greywhind_ | greywhind (11:35 PM):~/Programming/tp04/tpclient-pywx > git cl config | 02:35 |
| greywhind_ | Traceback (most recent call last): | 02:35 |
| greywhind_ | File "/Users/greywhind/bin/git-cl", line 9, in ? | 02:35 |
| greywhind_ | import readline | 02:35 |
| greywhind_ | ImportError: No module named readline | 02:35 |
| mithro | oh your on mac? :( | 02:35 |
| greywhind_ | for the next couple of days, yaeh | 02:35 |
| greywhind_ | *yeah | 02:36 |
| tansell | that is crap they didn't compile read-line | 02:36 |
| greywhind_ | until my computer gets back | 02:36 |
| tansell | greywhind_, you can probably get away without readline with a bit of hacking | 02:40 |
| *** tansell-laptop has joined #tp | 02:41 | |
| greywhind_ | tansell: well, i should have my computer back in about 3 days... | 02:41 |
| tansell | http://codereview.mithis.com/ | 02:41 |
| tpb | Title: Recent Issues - Code Review (at codereview.mithis.com) | 02:41 |
| greywhind_ | tansell: looks nice | 02:41 |
| shenki | mithro: is that running on your server, or on app engine? | 02:42 |
| shenki | mithro: it's asking us for a @mithis.com account | 02:42 |
| tansell | shenki, on app-engine under @mithis.com | 02:43 |
| tansell | it should allow anyone to upload | 02:43 |
| shenki | https://www.google.com/a/mithis.com/ServiceLogin?service=ah&passive=true&continue=http://codereview.mithis.com/_ah/login%3Fcontinue%3Dhttp://codereview.mithis.com/a/mithis.com%253Fcontinue%253D/<mpl=ga&ahname=Google+Code+Reviews&sig=eee3f38978e84beb46012cf9254cfd37 | 02:45 |
| tpb | <http://ln-s.net/3ED:> (at www.google.com) | 02:45 |
| shenki | if i go there | 02:45 |
| shenki | hwich is the "log in" page on cr.mithis.com | 02:45 |
| greywhind_ | File "/Users/greywhind/bin/upload.py", line 42, in ? | 02:45 |
| greywhind_ | import subprocess | 02:45 |
| greywhind_ | ImportError: No module named subprocess | 02:45 |
| shenki | your python is too old? | 02:46 |
| shenki | hrm, subprocess was 2.4 | 02:46 |
| greywhind_ | greywhind (11:45 PM):~/Programming/tp04/tpclient-pywx > python --version | 02:47 |
| greywhind_ | Python 2.5.1 | 02:47 |
| tansell | subprocess is definately python2.5 | 02:47 |
| shenki | yep | 02:47 |
| shenki | mac's python is retarted? | 02:47 |
| tansell | maybe upload.py is using the wrong python version? | 02:47 |
| greywhind_ | #!/usr/bin/env python | 02:48 |
| tansell | greywhind_, what does that show if you run it? | 02:48 |
| tansell | usr/bin/env python | 02:48 |
| greywhind_ | Python 2.5.1 (r251:54869, Apr 18 2007, 22:08:04) | 02:48 |
| greywhind_ | [GCC 4.0.1 (Apple Computer, Inc. build 5367)] on darwin | 02:48 |
| greywhind_ | Type "help", "copyright", "credits" or "license" for more information. | 02:48 |
| greywhind_ | i'm going to install 3.0.1 | 02:49 |
| shenki | python? that probably wont work. try 2.6? it probably won't solve your problem, as 2.5.1 should work fine too | 02:50 |
| greywhind_ | python 3.0.1 has a mac installer | 02:50 |
| greywhind_ | why wouldn't it work? | 02:50 |
| mithro | 3.0.1 is not compatible with 2.x series | 02:51 |
| greywhind_ | ah | 02:51 |
| shenki | because 3.0 broke lots of things | 02:51 |
| mithro | first time they have ever broken compatibility | 02:51 |
| greywhind_ | 2.6.2 still has no subprocess | 03:03 |
| mithro | greywhind_: it sure does | 03:04 |
| mithro | http://docs.python.org/library/subprocess.html | 03:04 |
| tpb | Title: 18.1. subprocess — Subprocess management Python v2.6.2 documentation (at docs.python.org) | 03:04 |
| greywhind_ | well, git cl can't find it | 03:04 |
| greywhind_ | i can import it in the interactive shell though... | 03:04 |
| mithro | what is at the top of upload.py ? | 03:05 |
| *** nash has quit IRC | 03:05 | |
| greywhind_ | #!/usr/bin/env python | 03:05 |
| greywhind_ | i can import readline as well in the interactive shell | 03:06 |
| tansell | hrm | 03:06 |
| tansell | type | 03:06 |
| tansell | whereis python; which python | 03:06 |
| greywhind_ | greywhind (12:06 AM):~/Programming/tp04/tpclient-pywx > whereis python; which python | 03:06 |
| greywhind_ | usr/bin/python | 03:07 |
| greywhind_ | Library/Frameworks/Python.framework/Versions/Current/bin/python | 03:07 |
| tansell | oh that is possibly why | 03:07 |
| tansell | do a | 03:07 |
| tansell | ls -l /usr/bin/python | 03:07 |
| greywhind_ | greywhind (12:06 AM):~/Programming/tp04/tpclient-pywx > ls -l /usr/bin/python | 03:08 |
| greywhind_ | lrwxr-xr-x 1 root wheel 9 Jul 9 2006 /usr/bin/python -> python2.3 | 03:08 |
| tansell | that would be why | 03:09 |
| greywhind_ | what should i do about it? | 03:09 |
| tansell | edit the files to have a python2.5 at the top | 03:10 |
| shenki | 2.3. welcome to the past. | 03:10 |
| shenki | that's worse than RHEL ;) | 03:10 |
| greywhind_ | hmm... there's no python2.5 in usr/bin | 03:11 |
| greywhind_ | only python and python2.3 | 03:11 |
| mithro | so where is your python2.5 installed? | 03:12 |
| greywhind_ | maybe ln -s Library/Frameworks/Python.framework/Versions/Current/bin/python /usr/bin/python? | 03:12 |
| mithro | greywhind_: give it a try - but I'm not sure | 03:12 |
| greywhind_ | fixed it | 03:14 |
| greywhind_ | what do i put in for Rietveld server? | 03:14 |
| greywhind_ | codereview.mithis.com? | 03:15 |
| shenki | codereview.mithis.cm | 03:15 |
| shenki | yep | 03:15 |
| shenki | greywhind_: you should record the answers and put them on the wiki :) | 03:15 |
| greywhind_ | any cc list? | 03:15 |
| shenki | i'd suggest cc'ing the tp dev list. advantage is you get more people giving input, disadvantage is people start commenting on the mailing list which means the codereview server doesn't get to record your comments | 03:16 |
| mithro | well | 03:17 |
| mithro | I wouldn't at the moment | 03:17 |
| greywhind_ | k. i'll leave it blank for now. | 03:17 |
| mithro | seeing as all the developers hang out here | 03:17 |
| greywhind_ | tree status URL? | 03:17 |
| mithro | tree status url? | 03:17 |
| greywhind_ | that's all it says | 03:17 |
| tansell | http://src.chromium.org/svn/trunk/tools/depot_tools/README.git-cl.codereview | 03:17 |
| tpb | <http://ln-s.net/3ECy> (at src.chromium.org) | 03:17 |
| tansell | I can see anything there | 03:17 |
| greywhind_ | tansell: this seems to be an example: | 03:19 |
| greywhind_ | http://chromium-status.appspot.com/current | 03:19 |
| greywhind_ | leave it blank? | 03:20 |
| tansell | yeah | 03:20 |
| shenki | your tree doesn't really close, so it's not relevant :) | 03:20 |
| greywhind_ | do we have ViewVC? | 03:20 |
| tansell | we have gitweb | 03:20 |
| greywhind_ | ViewVC URL: | 03:21 |
| greywhind_ | i don't know if it would work with gitweb | 03:21 |
| greywhind_ | i'll leave it blank | 03:21 |
| tansell | guess we should patch reitveld to support gitweb | 03:24 |
| shenki | s/gitweb/git/ | 03:24 |
| greywhind_ | guess so | 03:24 |
| greywhind_ | tansell: unless there's anything else, i'm going to go to sleep | 03:25 |
| mithro | greywhind_: did you try uploading anything? | 03:25 |
| greywhind_ | nope | 03:25 |
| greywhind_ | don't have any changes yet, beyond what i already pushed | 03:25 |
| tansell | okay | 03:26 |
| shenki | tansell: have you used pydev before? | 03:28 |
| tansell | shenki, I attempted once | 03:28 |
| shenki | the license is...scary | 03:28 |
| tansell | scary? | 03:32 |
| shenki | it's unclear as to weather im actually allowed to use it for more than 30 days or not | 03:33 |
| tansell | url? | 03:35 |
| tansell | This plugin is licensed under the Eclipse Public License 1.0 license. | 03:37 |
| tansell | For legal terms, see the Eclipse Public License. For my own enjoyment, take a deep breath. Really deep one, until you can feel in your toes and your head gets dizzy. Doesn't that feel good? Yum. Aleks and Fabio. | 03:37 |
| tansell | 03:37 | |
| tansell | 03:37 | |
| shenki | thats what the website says. but when you go to install it it shows a diff licence | 03:37 |
| shenki | all of the LICENSE.txt files inside the plugins are EPL | 03:39 |
| shenki | ah, here it is | 03:39 |
| shenki | http://pastebin.com/f3231cc33 | 03:39 |
| tpb | Title: pastebin - collaborative debugging tool (at pastebin.com) | 03:39 |
| tansell | http://svn.aptana.com/svn/pydev/trunk/plugins/org.python.pydev/LICENSE.txt | 03:43 |
| tpb | <http://ln-s.net/3EE3> (at svn.aptana.com) | 03:43 |
| * llnz ponders | 03:48 | |
| tansell | llnz, code conventions? | 03:48 |
| llnz | nope, just wasn't doing anything and pondering what to do | 03:49 |
| shenki | tansell: the evil license is the one for pydev _extensions_. pydev itself is EPL. | 03:52 |
| shenki | mystery solved | 03:52 |
| mithro | what is pydev _extensions_ ? | 03:53 |
| * shenki shrugs | 03:53 | |
| shenki | i dont think i want to use pydev anyway | 03:53 |
| tansell | it looked pretty good but I could never get it to work properly | 03:57 |
| mithro | looks like I might need to add some features to git-cl | 04:55 |
| *** jmtan has quit IRC | 05:11 | |
| *** mithro has quit IRC | 05:48 | |
| *** mithro has joined #tp | 05:55 | |
| *** greenlion has joined #tp | 06:23 | |
| * llnz wanders off | 07:57 | |
| llnz | later all | 07:57 |
| *** llnz has quit IRC | 07:57 | |
| *** verhoevenv has joined #tp | 07:59 | |
| *** zzorn_ has joined #tp | 08:11 | |
| *** verhoevenv has quit IRC | 09:28 | |
| *** verhoevenv has joined #tp | 09:58 | |
| *** bddebian has joined #tp | 10:07 | |
| bddebian | Heya | 10:08 |
| *** greywhind has joined #tp | 11:07 | |
| *** greywhind_ has quit IRC | 11:07 | |
| *** greywhind_ has joined #tp | 11:08 | |
| *** peres has joined #tp | 11:09 | |
| *** greywhind has quit IRC | 11:24 | |
| *** peres has left #tp | 11:34 | |
| *** greywhind has joined #tp | 13:08 | |
| *** greywhind_ has quit IRC | 13:08 | |
| *** verhoevenv has quit IRC | 13:14 | |
| CIA-46 | joelboh tpclient-pyweb * rf8c8218156cf / (16 files in 5 dirs): Removed old code. Added initial code for the map component (frontend). | 13:21 |
| *** Erroneous has joined #tp | 13:55 | |
| *** verhoevenv has joined #tp | 14:02 | |
| *** Noughmad has joined #tp | 15:29 | |
| *** Landon has quit IRC | 15:30 | |
| *** llnz has joined #tp | 16:35 | |
| llnz | morning all | 16:36 |
| *** Landon has joined #tp | 16:39 | |
| *** Landon has quit IRC | 16:45 | |
| *** Landon has joined #tp | 16:45 | |
| *** bddebian has quit IRC | 16:45 | |
| *** Landon has quit IRC | 16:52 | |
| *** Landon has joined #tp | 16:52 | |
| *** greenlion has quit IRC | 17:44 | |
| *** Noughmad has quit IRC | 17:56 | |
| *** nash has joined #tp | 19:00 | |
| *** greywhind_ has joined #tp | 19:11 | |
| *** greywhind has quit IRC | 19:11 | |
| *** greywhind has joined #tp | 19:13 | |
| *** greywhind_ has quit IRC | 19:13 | |
| CIA-46 | cherez libtpproto2-py * rf61f89d9eb7e /testxstruct.py: Added integer tests. | 19:16 |
| *** mithro has quit IRC | 19:39 | |
| *** alanp has joined #tp | 20:01 | |
| alanp | llnz: ping | 20:27 |
| llnz | hi alanp | 20:27 |
| alanp | hey | 20:28 |
| alanp | did you write the current mtsec? | 20:28 |
| llnz | some of it, the bits copied from minisec | 20:28 |
| alanp | llnz: did my proposal look correct to you, that the components that make a ship have mostly not been created? | 20:29 |
| llnz | yes | 20:29 |
| llnz | if anything, they need to be better organised | 20:30 |
| alanp | what do you mean? | 20:30 |
| tansell | cherez, ping? | 20:30 |
| greywhind | tansell: oh, hey - i have a question for you | 20:31 |
| *** verhoevenv has quit IRC | 20:31 | |
| tansell | greywhind, shoot | 20:31 |
| llnz | alanp: the ones that are there are hard to identify, and clearly the mtsec.cpp file it much too long | 20:31 |
| greywhind | i've been trying to figure out how to switch over the code that finds the type of an object ("Universe," "Galaxy," etc.) | 20:31 |
| llnz | it would be helpful to group them together | 20:31 |
| greywhind | and i just can't seem to figure out how to compare the dynamic-class stuff to anything that would allow me to do so | 20:32 |
| tansell | greywhind, you are of course right - that old scheme needs to go away | 20:32 |
| * alanp double-reads mtsec.cpp | 20:33 | |
| greywhind | tansell: i still need some way to figure out whether an object is the unverse, etc. is there a way to compare type(obj) to something? | 20:34 |
| alanp | maybe the ships should be in their own class? | 20:34 |
| alanp | would you agree with that/ | 20:34 |
| greywhind | because it gives me <dynamic-class 'Planet' (3) at 0x11e55470> | 20:34 |
| greywhind | and i'm not sure what to compare that to | 20:34 |
| tansell | greywhind, the universe is always object 0 | 20:34 |
| greywhind | what about galaxies, though? | 20:35 |
| greywhind | a lot of the code tries to ignore the universe and galaxy objects | 20:35 |
| tansell | greywhind, well I'm sure you can come up with a good idea | 20:35 |
| alanp | and also i'm guessing the missile rack is really the missile bay? stuff like that i guess needs to be changed? | 20:35 |
| greywhind | tansell: so there's nothing to compare dynamic-class types to that would let me find galaxies that way? | 20:36 |
| tansell | alanp, racks store missiles before they have been launched | 20:36 |
| alanp | what does a bay do? | 20:36 |
| alanp | ohhhh | 20:36 |
| alanp | bay stores the missile about to be launched? | 20:37 |
| tansell | bays are what you launch missles from | 20:39 |
| tansell | IE A ship might have 20 missile racks and only one missile bay? | 20:39 |
| alanp | ok | 20:39 |
| tansell | llnz, is that your understanding | 20:39 |
| tansell | s/bay/tube/ | 20:39 |
| alanp | ahhhhhhh ok | 20:39 |
| alanp | launch tube | 20:39 |
| alanp | yeah, some things aren't matching up to the spec | 20:40 |
| alanp | i will rename | 20:40 |
| tansell | I could be wrong - ask llnz | 20:40 |
| tansell | greywhind, not really - maybe you could ignore objects which have no interesting properties | 20:40 |
| alanp | no, i believe you are correct | 20:40 |
| greywhind | tansell: hm... would galaxies always have the universe as a parent? | 20:41 |
| alanp | llnz: please verify that missile bay = launch tube | 20:41 |
| llnz | yes, missile tube controls how many launched at once, missile rack is how many can be fired before they run out | 20:41 |
| llnz | yes | 20:41 |
| alanp | ok | 20:41 |
| tansell | greywhind, probably - but remeber ships can have a parent of the universe too | 20:41 |
| greywhind | tansell: mm... i see. | 20:42 |
| alanp | and it looks like that is the only component already created? | 20:42 |
| llnz | alanp: quite possibly | 20:42 |
| alanp | ok, cool | 20:42 |
| llnz | i think a few of the missile components have been created as well | 20:42 |
| alanp | well yes | 20:43 |
| alanp | i was only including the ship components | 20:43 |
| alanp | the components that make a ship i guess | 20:43 |
| llnz | yeah | 20:45 |
| tansell | greywhind, it's a hard problem | 20:45 |
| tansell | greywhind, do you want to test the git-cl stuff? | 20:45 |
| greywhind | tansell: quite... i wish i had more to work with, like an 'object type' property | 20:46 |
| alanp | llnz: should all of these components be in another class? | 20:46 |
| greywhind | tansell: sure, but i don't have anything to commit because i've been stuck on this problem | 20:46 |
| alanp | or.. a few others? | 20:46 |
| greywhind | tansell: would ships have resource lists? | 20:47 |
| greywhind | fleets, that is | 20:47 |
| llnz | alanp: that could be a good idea, keeping things tidy | 20:47 |
| tansell | greywhind, fleets will have lots of interesting properties | 20:47 |
| tansell | the problem of ignoring objects which don't have any interesting properties is that catches starsystems | 20:47 |
| alanp | llnz, could? hehe | 20:48 |
| tansell | another option would be to just hard code it for now | 20:50 |
| greywhind | hard-code it how? | 20:50 |
| greywhind | also, this is going to make the icon code difficult | 20:50 |
| greywhind | def Icon(self, obj): | 20:50 |
| greywhind | if isinstance(obj, Fleet): | 20:50 |
| greywhind | return FleetIcon(self.cache, obj, self.Colorizer) | 20:50 |
| greywhind | elif isinstance(obj, Wormhole): | 20:50 |
| greywhind | return WormholeIcon(self.cache, obj, self.Colorizer) | 20:50 |
| greywhind | else: | 20:50 |
| greywhind | return SystemIcon(self.cache, obj, self.Colorizer) | 20:50 |
| tansell | greywhind, I didn't say that making the client support tp04 was going to be easy :P | 20:51 |
| greywhind | each object will need to hold which icon it wants to use, i'd guess | 20:51 |
| greywhind | in the media section | 20:51 |
| tansell | cherez, we have our weekly status meeting now | 20:51 |
| tansell | greywhind, when did we decided to meet for our weekly meeting? | 20:51 |
| greywhind | tansell: i think it was... wednesdays, at 9:00 pm my time? | 20:52 |
| tansell | greywhind, I think that is probably | 20:52 |
| tansell | greywhind, what if your timezone again? | 20:52 |
| greywhind | tansell: i'm in MST | 20:52 |
| tansell | city? | 20:53 |
| greywhind | Phoenix, AZ | 20:53 |
| tansell | aparently you have scattered clouds | 20:54 |
| greywhind | heh | 20:54 |
| greywhind | tansell: do objects already associate themselves with media that can be used to draw them? or will that need to be added to the rulesets or something? | 20:54 |
| tansell | greywhind, they should - they probably don't at the moment yet | 20:56 |
| greywhind | ok | 20:57 |
| alanp | tansell: I have posted my first status report | 20:57 |
| alanp | Please check it out, http://alanp.ca/blog/2009/05/25/gsoctp-weekly-report-beginning-may-25-2009/ | 20:57 |
| tpb | <http://ln-s.net/3ERC> (at alanp.ca) | 20:57 |
| alanp | let me know if there are any issues with it | 20:57 |
| greywhind | for now, i'm going to just draw everything with a system icon | 20:57 |
| tansell | greywhind, sounds fair | 20:59 |
| tansell | alanp, I would recommend that maybe you look at putting the information in xml/csv files and parse them in that way - will make balancing much easier | 21:00 |
| tansell | tinyxml is already used in the risk ruleset for map loading | 21:00 |
| alanp | which information? | 21:00 |
| tansell | alanp, components/resources/etc | 21:01 |
| *** mithro has joined #tp | 21:02 | |
| alanp | hmm that sounds like a good idea | 21:02 |
| llnz | alanp: feel free to suggest things to be moved into the core of the server | 21:03 |
| alanp | llnz: ok | 21:03 |
| llnz | tinyxml is already on the list (for battlexml) | 21:03 |
| alanp | so maybe just import that stuff from xml | 21:03 |
| alanp | i'm not very familiar with tinyxml, but i'm sure i could learn and implement it pretty easily | 21:04 |
| * alanp eyes mapimport.cpp | 21:04 | |
| tansell | greywhind, you should have a meeting request for the correct time hopefully | 21:05 |
| alanp | next week i'm going to have a much better place to develop :-) | 21:05 |
| * alanp on couch w/ laptop right now | 21:05 | |
| mithro | I wrote most of tpclient-pywx sitting on couches :P | 21:06 |
| alanp | this couch is not particularly comfortable | 21:06 |
| tansell | greywhind, so it looks like git-cl needs some extra code to make it totally useful - but it should still work for now | 21:06 |
| greywhind | tansell: ok | 21:07 |
| alanp | next week i will have my 2x22" monitor setup, 3360x1050 | 21:07 |
| greywhind | tansell: where is this meeting request? | 21:07 |
| tansell | greywhind, it should have gone to your cox.net email address | 21:07 |
| greywhind | tansell: ah. ok. | 21:07 |
| greywhind | i'll check it | 21:08 |
| *** greywhind has quit IRC | 21:11 | |
| tansell | llnz, did you see greywhind's email about the coordinates stuff? | 21:12 |
| *** greywhind has joined #tp | 21:12 | |
| llnz | tansell: yes i did | 21:12 |
| *** mithro_ has joined #tp | 21:12 | |
| llnz | i was going to reply but haven't yet | 21:12 |
| llnz | probably will tonight | 21:12 |
| tansell | llnz, what are your thoughts? | 21:13 |
| llnz | an object with multiple coordinates is underspecified, let along relative coords to it | 21:13 |
| llnz | s/along/alone/ | 21:15 |
| tansell | underspecified? | 21:15 |
| *** alanp has quit IRC | 21:16 | |
| llnz | which coords do you use? | 21:19 |
| tansell | llnz, for relative coordinates you mean | 21:20 |
| llnz | if there are more than one? | 21:20 |
| cherez | tansell: Pong. | 21:20 |
| llnz | no, on any object | 21:20 |
| tansell | llnz, so we draw a multiple coordinate object as something like a wormhole | 21:20 |
| tansell | cherez, hey - so our meeting was suppose to start 20 minutes ago | 21:21 |
| llnz | i don't think that's the best way | 21:21 |
| tansell | (IE the object is at two positions at once) | 21:21 |
| llnz | i think a wormhold should be two object (one at each end) | 21:21 |
| llnz | with a parameter that references the other end | 21:21 |
| cherez | Doh. :/ | 21:22 |
| tansell | what about things like mine fields which might be an arbitary shaped object | 21:22 |
| tansell | cherez, so I don't see a status report on your blog either? | 21:24 |
| cherez | I have a blog? | 21:25 |
| cherez | Not much to report so far. I've been working on xstruct and aside from the bit about characters it has passed all my tests so far. | 21:25 |
| *** bddebian has joined #tp | 21:26 | |
| tansell | cherez, you where suppose to set up one so that you can post weekly status reports | 21:26 |
| tansell | and your status reports are due before our weekly meeting | 21:26 |
| cherez | Well, I better set up a blog now. | 21:27 |
| tansell | cherez, yes - I would also like to start doing post-push code reviews | 21:27 |
| *** mithro has quit IRC | 21:28 | |
| cherez | Post-push? | 21:28 |
| tansell | opp | 21:28 |
| tansell | that should be | 21:28 |
| tansell | pre-push | 21:28 |
| llnz | tansell: true, maybe we need a better way of specifying them | 21:28 |
| llnz | with them == coordinates, size and shape | 21:29 |
| cherez | tansell: So you want to see my commits before I push them? | 21:31 |
| tansell | cherez, yeah so I can give feedback a bit easier | 21:37 |
| cherez | Is there any easy way to do that with git? | 21:38 |
| tansell | Using a thing called git-cl | 21:38 |
| tansell | http://www.thousandparsec.net/wiki/GitCL | 21:47 |
| tpb | Title: GitCL - Thousand Parsec Wiki (at www.thousandparsec.net) | 21:47 |
| tansell | cherez, so where are we at? | 21:47 |
| cherez | I have tests for all the numeric types with the pack-unpack tests. | 21:48 |
| greywhind | tansell: do any of the AI clients work with tp04? | 21:50 |
| tansell | greywhind, no | 21:51 |
| greywhind | tansell: hm. | 21:51 |
| tansell | greywhind, why? | 21:51 |
| greywhind | well, i can't give orders at the moment | 21:51 |
| greywhind | so it's hard to test my fleet drawing | 21:51 |
| greywhind | because they're all still docked | 21:52 |
| greywhind | i was hoping an AI would move some fleets so i could check if it drew | 21:52 |
| cherez | Then I guess I sit down with a piece of paper so I can figure out the correct strings to be generated for all the pack functions is. | 21:53 |
| greywhind | tansell: the good news is that systems now draw properly! | 21:53 |
| tansell | greywhind, well the AI can play with tp03 on the same user | 21:53 |
| greywhind | tansell: how can i set that up? | 21:54 |
| tansell | start up a server | 21:54 |
| tansell | connect with an AI | 21:54 |
| greywhind | is there a minisec AI, or should i use Risk? | 21:54 |
| tansell | connect with a client using the same username/password | 21:54 |
| tansell | greywhind, I don't think minisec AI was ever finished | 21:54 |
| tansell | poke reac and iwanowitch | 21:55 |
| greywhind | tansell: i'm not sure how to start an AI except through the single-player dialog | 21:55 |
| greywhind | and that didn't seem to work | 21:55 |
| tansell | greywhind, it's just a ./daneel-ai <arguments> | 21:55 |
| tansell | use --help to find out want to give it | 21:55 |
| greywhind | ok | 21:55 |
| greywhind | thanks | 21:55 |
| llnz | tpsai-py works for minisec | 21:55 |
| llnz | (tp03) | 21:55 |
| greywhind | oh, good | 21:56 |
| greywhind | No module named logilab.constraint | 21:59 |
| greywhind | The following requirements where not met: | 21:59 |
| greywhind | python-constraint | 21:59 |
| greywhind | anyone know how i can install what it needs? | 22:03 |
| greywhind | the download link at: http://www.logilab.org/project/logilab-constraint | 22:04 |
| tpb | <http://ln-s.net/21dk> (at www.logilab.org) | 22:04 |
| greywhind | seems to be broken | 22:04 |
| tansell | greywhind, it's easy to fix | 22:05 |
| tansell | just get it to list the directory | 22:06 |
| greywhind | ah. | 22:06 |
| greywhind | that makes sense - should have thought of that | 22:06 |
| tansell | or you could always get it from the deb package - http://ftp.de.debian.org/debian/pool/main/c/constraint/constraint_0.4.0.orig.tar.gz | 22:08 |
| tpb | <http://ln-s.net/3ERx> (at ftp.de.debian.org) | 22:08 |
| tansell | cherez, I'll be back in an hour - heading to lunch now | 22:10 |
| tansell | back now | 22:52 |
| greywhind | tansell: shall we test the git cl stuff now? | 22:53 |
| tansell | I'm having some problems getting it to work | 22:53 |
| tansell | greywhind, but we can see if it works for you | 22:53 |
| tansell | http://www.thousandparsec.net/wiki/GitCL | 22:53 |
| tpb | Title: GitCL - Thousand Parsec Wiki (at www.thousandparsec.net) | 22:53 |
| greywhind | tansell: except for logging in, it seems to work | 23:06 |
| tansell | greywhind, it uploaded the issue? | 23:07 |
| tansell | I don't see anything at | 23:07 |
| tansell | http://codereview.mithis.com/all | 23:07 |
| tpb | Title: Recent Issues - Code Review (at codereview.mithis.com) | 23:07 |
| greywhind | greywhind (08:04 PM):~/Programming/tp04/tpclient-pywx > git cl upload origin/tp04 | 23:07 |
| greywhind | windows/main/overlays/Overlay.py | 8 +---- | 23:07 |
| greywhind | windows/main/overlays/Systems.py | 54 +++++++++++++++++++++++-------------- | 23:07 |
| greywhind | 2 files changed, 35 insertions(+), 27 deletions(-) | 23:07 |
| greywhind | Upload server: codereview.mithis.com (change with -s/--server) | 23:07 |
| greywhind | Email (login for uploading to codereview.mithis.com): [email protected] | 23:07 |
| greywhind | Password for [email protected]: | 23:07 |
| greywhind | Saving authentication cookies to /Users/greywhind/.codereview_upload_cookies | 23:07 |
| greywhind | then it said nothing else and finished, seemingly successfully | 23:08 |
| tansell | greywhind, is [email protected] a google account? | 23:08 |
| greywhind | tansell: yes | 23:09 |
| tansell | hrm | 23:09 |
| greywhind | although your login screen seems not to take @ symbols | 23:09 |
| greywhind | tansell: i have to eat dinner. be back in about half an hour, probably | 23:12 |
| tansell | greywhind, okay | 23:12 |
| *** greywhind_ has joined #tp | 23:13 | |
| *** greywhind has quit IRC | 23:13 | |
| shenki | tansell: i think this is the same issue as i was having yesterday; it wants us to have @mithis.com accounts | 23:26 |
| tansell | shenki, well it doesn't work with @mithis.com accounts either | 23:29 |
| shenki | oh. that sucks | 23:30 |
| *** Erroneous has quit IRC | 23:31 | |
| tansell | greywhind_, you can try uploading to codereview.appspot.com by doing a git-cl config | 23:32 |
| tansell | we can use the temporarily | 23:38 |
| *** JLP has quit IRC | 23:44 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!