| *** tpb has joined #tp | 00:00 | |
| *** ChanServ sets mode: +o tpb | 00:00 | |
| tansell | Greywhind, you have extra semicolons in objectutils.py | 00:01 |
|---|---|---|
| Greywhind | tansell: i'm going to remove them. stupid C++ | 00:01 |
| cherez | tansell: How does ObjectDesc know that it makes Objects? | 00:04 |
| cherez | I don't believe they reference one-another in protocol.xml. | 00:04 |
| tansell | you mean Object classes? the fact that it's called "ObjectDesc" :) | 00:04 |
| tansell | Greywhind, closer to this http://pastebin.com/dfa93543 | 00:05 |
| tpb | Title: pastebin - collaborative debugging tool (at pastebin.com) | 00:05 |
| tansell | cherez, they don't it's just kinda implied | 00:05 |
| cherez | So how would parser know that ObjectDesc generates Object classes? | 00:06 |
| cherez | Cutting the last 4 characters off any packet with desc at the end? | 00:06 |
| tansell | actually take a look at the experimental branch | 00:07 |
| tansell | of document.git | 00:07 |
| tansell | cherez, | 00:08 |
| tansell | check out | 00:08 |
| tansell | 1665 <typefield>otype</typefield> | 00:08 |
| tansell | 1666 <typeframe name="OrderDesc" idfield="id"> | 00:08 |
| tansell | 1667 <getlist name="parameters"> | 00:08 |
| tansell | 1668 <getfield name="type"/> | 00:08 |
| tansell | 1669 </getlist> | 00:08 |
| tansell | 1670 </typeframe | 00:08 |
| tansell | in the object | 00:08 |
| tansell | s/object/order/ | 00:08 |
| tansell | http://git.thousandparsec.net/gitweb/gitweb.cgi?p=documents.git;a=blob;f=protocol/protocol.xml;h=94b5296cbab06e11ccfa8ba71a1cb99ff02fd304;hb=master#l1665 | 00:08 |
| tpb | <http://ln-s.net/3QeR> (at git.thousandparsec.net) | 00:08 |
| tansell | so I guess you could say it's "backwards" | 00:10 |
| cherez | That's so much nicer with which to work. | 00:11 |
| tansell | cherez, the experimental branch or? | 00:11 |
| cherez | Yeah. | 00:11 |
| tansell | well I think the experimental branch is dead | 00:12 |
| tansell | the typefield/typeframe describes the same information | 00:12 |
| tansell | it's pretty easy to convert between the two | 00:12 |
| cherez | So how is the parser going to figure out what describes what? | 00:15 |
| tansell | it can use that typefield/typeframe stuff | 00:16 |
| tansell | which lets you figure out that Objects are described by ObjectDesc frames | 00:17 |
| cherez | Oh, that would do it. | 00:17 |
| cherez | I think I've got that all in-head, but it's pretty late so I'm going to bed. | 00:20 |
| cherez | Thanks; I'll see what I can get written up tomorrow. | 00:20 |
| tansell | cherez, I would draw a diagram before I go to bed if I was you :) | 00:20 |
| cherez | I'll try. | 00:21 |
| tansell | Greywhind, that make sense? | 00:23 |
| Greywhind | tansell: working on it | 00:25 |
| Greywhind | i think i'll be able to get it | 00:25 |
| tansell | Greywhind, no guarantee that the code actually works :P | 00:25 |
| Greywhind | tansell: i didn't use it :P | 00:25 |
| Greywhind | thanks though | 00:25 |
| tansell | Greywhind, you understand the difference? | 00:28 |
| Greywhind | tansell: yeah | 00:28 |
| llnz | bbl | 00:28 |
| *** llnz has quit IRC | 00:28 | |
| tansell | so from looking at the code, the ObjectDesc doesn't actually set any attributes on itself | 00:29 |
| tansell | instead everything is in a list called arguments | 00:29 |
| tansell | and I'm pretty sure the length calculation is wrong | 00:30 |
| tansell | Landon, any luck with the video stuff yet? | 00:33 |
| Greywhind | tansell: there is no list called arguments | 00:34 |
| tansell | self.arguments = arguments | 00:35 |
| Greywhind | File "./extra/objectutils.py", line 168, in getOrderQueueLimit | 00:36 |
| Greywhind | print desc.arguments | 00:36 |
| Greywhind | AttributeError: type object 'DynamicObject' has no attribute 'arguments' | 00:36 |
| Greywhind | desc = objects.ObjectDescs()[subtype] | 00:36 |
| tansell | file:///usr/local/google/_blaze_tansell/14d81459ffe92eb4baf5fb15912c2215/google3/_coverage/_html/index.html | 00:36 |
| tansell | oh | 00:36 |
| tansell | so it appears this is totally broken then :P | 00:37 |
| tansell | those arguments are never transferred to the DynamicObject class | 00:38 |
| Greywhind | tansell: is there any way to get the value from an IntegerStructure? | 00:39 |
| tansell | Greywhind, what do you mean? | 00:39 |
| Greywhind | <GroupStructure 0xa3969cc Orders> | 00:39 |
| Greywhind | Queue: <ObjectParamOrderQueue 0xa3969ec Order Queue> | 00:39 |
| Greywhind | <IntegerStructure 0xa396a0c queueid> | 00:39 |
| Greywhind | can i get the queueid from that? | 00:39 |
| Greywhind | that's what's in the properties | 00:40 |
| tansell | object.Orders.OrderQueue.queueid | 00:42 |
| tansell | something along those lines | 00:42 |
| tansell | same way you would access the value normally | 00:42 |
| Greywhind | but you said it didn't set attrs on itself | 00:42 |
| Greywhind | it's the object desc | 00:42 |
| Greywhind | and i need to match the queueid with the one from the object | 00:42 |
| tansell | Greywhind, you are confusing yourself | 00:42 |
| *** nash has quit IRC | 00:43 | |
| tansell | you missed the difference that I was trying to point-out in my pastebin example | 00:43 |
| Greywhind | tansell: well, yeah, you're looking through the object itself, then getting just the maxslots from the object desc | 00:44 |
| Greywhind | but since the object desc doesn't set attributes on itself like that, i can't do it that way | 00:44 |
| Greywhind | so i need to traverse the .properties of the ObjectDesc and find the one related to this object, then somehow get the values from that | 00:45 |
| Greywhind | *this order queue | 00:45 |
| tansell | you can't do this -> so i need to traverse the .properties of the ObjectDesc and find the one related to this object, then somehow get the values from that | 00:45 |
| Greywhind | what can i do? | 00:46 |
| tansell | so I'm just fixing libtpclient-py now | 00:49 |
| tansell | which should let you do the first option | 00:50 |
| Greywhind | ok | 00:52 |
| tansell | Greywhind, pushing the fixes now | 01:04 |
| CIA-21 | mithro libtpproto-py-tp04 * rdfa261974c75 /tp/netlib/objects/ObjectDesc.py: Make the extra data stuff work correctly. | 01:05 |
| Greywhind | tansell: ok | 01:06 |
| Landon | tansell: recordmydesktop seems to work, but I'll need to do a bit more work before I'm in a recording state again | 01:07 |
| Landon | (my functions used to use id as an argument, now they use the entity which is harder to get at from console in the viewer) | 01:07 |
| CIA-21 | mithro tpclient-pywx-tp04 * r2d2788e25559 /requirements.py: Don't die on a gconf error. | 01:14 |
| CIA-21 | mithro tpclient-pywx-tp04 * r904eb25b2ffa /libtpproto-py: Update the libtpproto-py submodule. | 01:14 |
| tansell | Greywhind, in theory | 01:14 |
| tansell | the ObjectQueueParam should have a .maxslots now | 01:15 |
| Greywhind | tansell: ok. i'll look into it soon. | 01:15 |
| *** llnz has joined #tp | 01:32 | |
| *** nash has joined #tp | 01:41 | |
| Greywhind | tansell: hmm | 01:43 |
| Greywhind | for propertygroup in obj.properties: group = getattr(obj, propertygroup.name) | 01:43 |
| Greywhind | print getattr(desc, propertygroup.name) | 01:43 |
| Greywhind | that doesn't seem to work | 01:43 |
| *** llnz has quit IRC | 02:10 | |
| tansell | so you know how you do a | 02:15 |
| *** greenlion has joined #tp | 02:57 | |
| *** llnz has joined #tp | 03:03 | |
| *** nash has quit IRC | 03:19 | |
| tansell | Greywhind, you still about? | 03:25 |
| tansell | I got halfway through describing how to do it and then got called away from the desk :P | 03:25 |
| Greywhind | tansell: s'ok | 03:35 |
| Greywhind | i'm here, but i'm going to sleep soon | 03:35 |
| tansell | Greywhind, so you can just read the value directly from parameter | 03:36 |
| tansell | ie | 03:36 |
| tansell | take a look at http://pastebin.com/d6bbaa4fa | 03:37 |
| tpb | Title: pastebin - collaborative debugging tool (at pastebin.com) | 03:37 |
| Greywhind | oh, you put it in the object itself? | 03:38 |
| Greywhind | ok | 03:38 |
| tansell | well | 03:54 |
| tansell | it's on the object structure which is per class | 03:54 |
| Greywhind | tansell: http://codereview.mithis.com/7001 | 04:05 |
| tpb | Title: Issue 7001: Added code to hide the list of orders in a queue when that queue can hold only one order at a time. - Code Review (at codereview.mithis.com) | 04:05 |
| Greywhind | and now, i sleep | 04:05 |
| *** llnz2 has joined #tp | 04:16 | |
| *** llnz has quit IRC | 04:16 | |
| *** llnz2 is now known as llnz | 04:16 | |
| tansell | hey llnz | 04:21 |
| tansell | how are things? | 04:21 |
| tansell | llnz, did the wedding photos ever get uploaded? | 04:21 |
| llnz | hi tansell | 04:22 |
| llnz | not bad | 04:22 |
| llnz | i have the set of photos on CD | 04:22 |
| llnz | should be online next weekend | 04:22 |
| tansell | llnz, great! | 04:23 |
| tansell | so you planning to go to LCA in wellington? | 04:24 |
| llnz | i'm in wellington, so yes | 04:26 |
| llnz | if things are going well, i should be in a flat with megan | 04:26 |
| *** tuna-fish has joined #tp | 05:22 | |
| *** jmtan has joined #tp | 05:44 | |
| * llnz actually looks at starting the third rewrite of minisec combat, this time actually getting it right | 05:58 | |
| * tansell is pondering porting tpserver-py to tp04 | 06:07 | |
| tansell | not going to happen, but I am thinking about it :P | 06:21 |
| llnz | hehe | 06:22 |
| tansell | I need a non-computer hobby | 06:25 |
| *** mithro has quit IRC | 06:34 | |
| Landon | hm | 06:57 |
| Landon | is it dinner time or breakfast time | 06:57 |
| Landon | my body can't seem to make up its mind | 06:57 |
| *** tuna-fish has quit IRC | 07:10 | |
| * llnz wanders off | 07:44 | |
| llnz | later all | 07:44 |
| *** llnz has quit IRC | 07:44 | |
| *** mithro has joined #tp | 07:48 | |
| CIA-21 | landon tpclient-pyogre * r1c135aa950cb / (media/battleviewer/battleviewer.layout src/battleviewer.py): The battleviewer now has a (non-functional) gui | 08:28 |
| mithro | Landon: screenshot? :P | 08:37 |
| Landon | not much to look at atm | 08:42 |
| Landon | well | 08:42 |
| Landon | of course its a gui | 08:42 |
| Landon | but its just the controls atm | 08:42 |
| Landon | thinking about how I want to go about getting the info boxes in | 08:42 |
| Landon | so that bit is blank :P | 08:42 |
| Landon | http://lfowles.org/images/battleviewer/actualgui.png | 08:44 |
| tpb | <http://ln-s.net/3QqP> (at lfowles.org) | 08:44 |
| mithro | Landon: your using text characters for the buttons? | 08:44 |
| Landon | yeah | 08:45 |
| Landon | I might change that though later | 08:45 |
| Landon | unless I can find a good one for play/stop | 08:45 |
| Landon | :\ | 08:45 |
| mithro | figure out at by the \ :) | 08:47 |
| mithro | I would move it to the corner and make the corner be the top left instead of the bottom right | 08:48 |
| Landon | I think the corner is baked in | 08:49 |
| mithro | have a chat to jmtan, he might be able to point you in the right direction | 08:50 |
| mithro | you could move the whole thing to the top left corner | 08:51 |
| mithro | tote: ping? | 08:51 |
| Landon | looking at the guis imageset right now and that's just how it is :\ | 08:52 |
| Landon | http://git.thousandparsec.net/gitweb/gitweb.cgi?p=tpclient-pyogre.git;a=blob;f=media/gui/SleekSpace/sleekspace.png;h=8e3e147c9804efed273c23e748b937f196aefdd6;hb=HEAD | 08:52 |
| tpb | <http://ln-s.net/3Qqb> (at git.thousandparsec.net) | 08:52 |
| mithro | Landon: you could use the images in that for the buttons? | 08:53 |
| Landon | hm I think I can | 08:54 |
| mithro | Landon: dunno | 08:54 |
| *** mithro has quit IRC | 09:21 | |
| *** peres has quit IRC | 10:32 | |
| *** CIA-21 has quit IRC | 11:47 | |
| *** mhilmi has quit IRC | 11:47 | |
| *** matthewd has quit IRC | 11:47 | |
| *** CIA-21 has joined #tp | 11:47 | |
| *** mhilmi has joined #tp | 11:47 | |
| *** matthewd has joined #tp | 11:47 | |
| *** welterde has joined #tp | 12:57 | |
| *** tuna-fish has joined #tp | 13:03 | |
| tote | tansell: pong | 13:38 |
| *** Erroneous has joined #tp | 13:47 | |
| *** jmtan has quit IRC | 14:18 | |
| CIA-21 | joelboh tpclient-pyweb * re4fd4e0c6085 / (3 files in 2 dirs): Frontend: Displays a message now in the message component. | 14:32 |
| *** tuna-fish has quit IRC | 14:42 | |
| *** Greywhind has quit IRC | 16:04 | |
| *** Greywhind has joined #tp | 17:37 | |
| alanp | ping | 18:17 |
| CIA-21 | landon tpclient-pyogre * r95b2ecd257b1 /media/battleviewer/battleviewer.layout: Added a MultiLineEditBox for Log messages | 18:59 |
| *** nash has joined #tp | 19:41 | |
| *** greenlion has quit IRC | 19:47 | |
| tansell | morning people | 20:11 |
| tansell | alanp, you proabaly want to add a name to that ping | 20:12 |
| tansell | tote, I wanted to know how you are/where going | 20:12 |
| Greywhind | tansell: i added a -1 default return value. should i resubmit for review, or just assume it's LGTM? | 20:21 |
| Greywhind | i mean, it's basically a 1-line change | 20:21 |
| Greywhind | because at the moment, the code that uses that function doesn't care what the return value is if it's not 1 | 20:22 |
| tansell | Greywhind, just submit it | 20:22 |
| Greywhind | tansell: ok | 20:22 |
| Greywhind | wait, for review? or push it? | 20:23 |
| Greywhind | "submit" could mean either :P | 20:23 |
| *** mithro has joined #tp | 20:23 | |
| tansell | Greywhind, commit it :) | 20:26 |
| Greywhind | tansell: ok | 20:26 |
| Greywhind | tansell: comments on http://codereview.mithis.com/6003 ? | 20:28 |
| tpb | Title: Issue 6003: Three small code cleaning fixes. - Code Review (at codereview.mithis.com) | 20:28 |
| tansell | LGTM | 20:37 |
| Greywhind | k | 20:40 |
| Greywhind | since that one is really three changes, should i merge without the --squash? | 20:41 |
| alanp | tansell: did you catch my ML e-mail? | 21:00 |
| alanp | re: "Save" in the client's design window | 21:00 |
| tansell | so your first patch looks right - can you send me a copy of that? | 21:03 |
| *** epyon has quit IRC | 21:06 | |
| *** DTRemenak|laptop has joined #tp | 21:17 | |
| *** DTRemenak has joined #tp | 21:22 | |
| *** DTRemenak|laptop has quit IRC | 21:24 | |
| *** DTRemenak is now known as DTRemenak|laptop | 21:24 | |
| tansell | alanp, poke? | 21:35 |
| alanp | hello | 21:38 |
| tansell | alanp, do you want to send me a copy of the first patch that you created for the design window? | 21:39 |
| tansell | I can then look at your second problem | 21:39 |
| alanp | sure | 21:40 |
| alanp | e-mail, dcc? | 21:40 |
| tansell | email me the git patch is probably best | 21:44 |
| tansell | so currently when logging into a mtsec server I just get a segfault | 22:03 |
| tansell | alanp, want a copy of the stack trace? | 22:04 |
| tansell | alanp, http://pastebin.com/d23c1fa72 | 22:07 |
| tpb | Title: pastebin - collaborative debugging tool (at pastebin.com) | 22:07 |
| alanp | sorry about that | 22:18 |
| alanp | tansell, sorry, I included a relative path in the configuration file | 22:19 |
| alanp | 2009-07-02 12:04:17 < Debug > Error: could not load components XML file | 22:19 |
| alanp | I should probably make the server stop when that happens | 22:19 |
| alanp | if you run it from the tpserver-cpp directory it willw ork | 22:20 |
| alanp | (that was on my todo to fix) | 22:20 |
| tansell | alanp, did you fix the CacheDirty problem? | 22:51 |
| tansell | welp I fixed tpclient-pywx | 22:56 |
| alanp | ? | 22:56 |
| alanp | We tried some things, but it ended up giving more errors | 22:56 |
| alanp | did you get the server running properly and connected now? | 22:56 |
| tansell | yes | 23:00 |
| *** nash has quit IRC | 23:00 | |
| CIA-21 | mithro libtpclient-py-stable * r4bcdb81d4683 /tp/client/cache.py: Only chec the nodes when they actually exist. | 23:01 |
| CIA-21 | mithro tpclient-pywx-stable * rcfbab40b714b /requirements.py: Don't die on a gconf error. | 23:02 |
| CIA-21 | mithro tpclient-pywx-stable * r1d7e9c6c1a0d /windows/main/winDesign.py: Fixed an overzealous search and replace. | 23:02 |
| CIA-21 | mithro tpclient-pywx-stable * r48412c3ab5a3 /windows/main/winDesign.py: Design modification time must be positive. | 23:02 |
| CIA-21 | mithro tpclient-pywx-stable * rd3e974213b7a /libtpclient-py: libtpclient-py bump. | 23:02 |
| alanp | what did those changes fix? | 23:05 |
| tansell | the problem you where having | 23:07 |
| alanp | awesome | 23:07 |
| tansell | now it's a server side problem | 23:07 |
| alanp | awesome! | 23:07 |
| tansell | alanp, you where so close | 23:07 |
| alanp | sigh | 23:07 |
| tansell | alanp, you needed one more line change in winDesign.py | 23:07 |
| tansell | Greywhind, ping? | 23:24 |
| alanp | 2009-07-01 23:25:51 < Debug > Odd, client sent wrong player id -1 | 23:26 |
| alanp | heh | 23:26 |
| tansell | for? | 23:29 |
| alanp | on "Save" | 23:29 |
| tansell | alanp, that doesn't explain which field that is wrong | 23:32 |
| alanp | nod | 23:33 |
| alanp | it's definitely sending the design to the server now, that was my main thing | 23:33 |
| tansell | probably self.owner | 23:33 |
| tansell | the client has no idea who'll end up as the owner when it sends a new design | 23:34 |
| tansell | so it's perfectly reasonable to send -1 | 23:34 |
| alanp | nod | 23:34 |
| alanp | it does look like a soft warning too | 23:34 |
| alanp | ERROR: Missing or extra expression in (define New DesignDesignType (make-design-vtable "pwpwpwpwpwpwpwpwpwpwpwpwpwpwpwpwpwpwpwpwpwpwpwpw")). | 23:35 |
| alanp | "DesignDesignType" | 23:35 |
| tansell | that is rather "interesting" :P | 23:36 |
| alanp | not sure what's up with the pw* either | 23:37 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!
