*** tpb has joined #tp | 00:00 | |
*** ChanServ sets mode: +o tpb | 00:00 | |
*** tpb has joined #tp | 00:08 | |
*** ChanServ sets mode: +o tpb | 00:08 | |
*** CIA-21 has joined #tp | 00:11 | |
tansell | Greywhind, ping? | 00:11 |
---|---|---|
* tansell pokes Greywhind with something sharp and pointy | 00:14 | |
*** DTRemenak|laptop has quit IRC | 00:25 | |
cherez | tansell: In a description frame, where do the descstructs of the parameters go? | 00:32 |
tansell | today has been a day full of bugs | 00:33 |
tansell | cherez, they go as part of the | 00:34 |
tansell | 1561 <list> | 00:34 |
tansell | 1562 <name>parameters</name | 00:34 |
tansell | but there is nothing in the protocol.xml which tells you that :( | 00:34 |
cherez | Gah. :( | 00:34 |
tansell | oh | 00:36 |
tansell | actually I lie | 00:36 |
tansell | 1369 <descparameter ref="ObjectParams" typefield="type"> | 00:36 |
tansell | 1370 <name>extradata</name> | 00:36 |
tansell | 1371 <longname>extra data for this parameter</longname> | 00:36 |
tansell | 1372 </descparameter | 00:36 |
cherez | Ahh. | 00:37 |
cherez | So the descstruct is just stuck in there? | 00:38 |
cherez | Meaning it's a heterogeneous list? | 00:38 |
*** tansell_laptop has quit IRC | 00:40 | |
tansell | heterogeneous? | 00:40 |
tansell | remember the code I was showing you yesterday in libtpproto-py? | 00:41 |
cherez | Yeah, I'm pulling it up. | 00:41 |
cherez | Yep, that's what I thought.... | 00:43 |
*** nash has joined #tp | 00:44 | |
cherez | The length of the list is still prepended to it, right? | 00:45 |
tansell | the length of the list the the number of elements | 00:49 |
*** tansell_laptop has joined #tp | 00:54 | |
cherez | Would it be legal if protocol.xml had a packet with multiple descparameters? | 00:57 |
tansell | cherez, possibly - but unlikely | 01:02 |
tansell | cherez, It would be okay to not support that for now | 01:02 |
cherez | Goody. | 01:22 |
*** jmtan has joined #tp | 01:22 | |
*** nash has quit IRC | 01:51 | |
*** nash has joined #tp | 01:57 | |
Greywhind | tansell: pong | 02:00 |
tansell | Greywhind, so we where suppose to have a meeting about 2 hours ago? | 02:01 |
Greywhind | tansell: sorry about that, it was my Dad's birthday and I took him out to a movie | 02:01 |
Greywhind | i totally forgot :( | 02:01 |
tansell | Greywhind, no problem - want to do it now? | 02:01 |
Greywhind | tansell: sure - want me to write a summary real fast? | 02:01 |
tansell | yes please | 02:02 |
tansell | cherez, so I'm expecting a blog post tonight right? | 02:06 |
tansell | mhilmi, poke? | 02:07 |
tansell | JLP, poke? | 02:08 |
cherez | tansell: As soon as I figure out a good way to implement packing and unpacking I'm going to blog it. | 02:08 |
*** nash has quit IRC | 02:10 | |
*** llnz has joined #tp | 02:11 | |
tansell | Greywhind, you fall asleep at your computer :) | 02:25 |
Greywhind | tansell: http://greywhind.wordpress.com/2009/07/01/gsoc-week-6-summary/ | 02:25 |
Greywhind | just finished | 02:25 |
tpb | <http://ln-s.net/3RCB> (at greywhind.wordpress.com) | 02:25 |
Greywhind | takes a bit of hunting to figure out all the things i did and write about them :P | 02:26 |
tansell | git log is handy for that :) | 02:27 |
Greywhind | true - i used the blog posts and code reviews | 02:28 |
cherez | tansell: The parameters of a packet that has use parameters are always appended at the end? | 02:29 |
tansell | cherez, yes | 02:29 |
cherez | Great. | 02:29 |
*** nash has joined #tp | 02:30 | |
tansell | cherez, assuming I understand what you are saying | 02:30 |
tansell | Greywhind, so after looking at the Design window today, it could use some love | 02:31 |
Greywhind | tansell: ok | 02:31 |
cherez | The values of whatever parameters and Object or Order takes are appended to the end in the order they were given in the corresponding *desc object. | 02:31 |
Greywhind | tansell: what does it need, exactly? | 02:31 |
tansell | Greywhind, "although it no longer has a type column" <- Umm they sure do have names now | 02:32 |
tansell | Greywhind, mainly that it's ugly :) | 02:32 |
tansell | the tree on the right side should be hidden until it's needed | 02:33 |
Greywhind | tansell: hm... where can i get the names? | 02:33 |
tansell | the Order/ObjectDesc has the name | 02:33 |
Greywhind | ok, i'll look into it | 02:33 |
tansell | in the .name value | 02:33 |
tansell | cherez, yes | 02:33 |
cherez | Good, that's easy. | 02:34 |
tansell | cherez, so you can actually use the ObjectDesc/OrderDesc to generate a new python class which has the appropriate structures | 02:35 |
cherez | Right, that's the plan. | 02:35 |
tansell | Greywhind, so - what is the plan for next week? | 02:42 |
Greywhind | tansell: well, i want to do more cleaning | 02:42 |
Greywhind | on the design window, since you want that cleaned | 02:42 |
tansell | Greywhind, oh the duplicate button on the design window doesn't work either | 02:42 |
Greywhind | and then on anything else i can find | 02:42 |
tansell | Greywhind, you want to forward port the patches I pushed today | 02:42 |
Greywhind | tansell: ok, i'll make sure to do that | 02:43 |
tansell | Greywhind, you should just be able to cherry pick them into the tp04 branches | 02:43 |
Greywhind | ok | 02:43 |
tansell | or you could try doing a merge | 02:43 |
Greywhind | ok | 02:44 |
Greywhind | i'll probably try the merge | 02:44 |
cherez | tansell: Should Structures know how to pack/unpack themselves? | 02:46 |
tansell | cherez, they do already right? | 02:47 |
cherez | They know their xstruct string, but that's it as far as I know. | 02:47 |
cherez | Should I be adding pack and unpack methods? | 02:48 |
tansell | cherez, well it's more efficient to have one pack/unpack method which just calls xstruct | 02:48 |
tansell | rather then doing a loop | 02:48 |
cherez | Yes, but it won't work with description frames, since xstruct's can't handle lists whose elements aren't all the same type. | 02:49 |
cherez | So at the least it will have to iterate through that list and determine the structure is for each element of the list. | 02:52 |
tansell | cherez, you can use the same method the xstruct in the tp04 branch does | 02:56 |
tansell | cherez, you might want to merge the changes in the tp04 xstruct into your xstruct | 03:02 |
cherez | Yeah, I just took a look at it. | 03:02 |
cherez | I've got some unit tests to change. | 03:03 |
tansell | it should be pretty simple to merge | 03:07 |
*** nash has quit IRC | 03:23 | |
*** greenlion has joined #tp | 03:34 | |
*** Greywhind has quit IRC | 04:01 | |
*** greenlion has quit IRC | 04:42 | |
*** jmtan has quit IRC | 06:51 | |
*** Landon has quit IRC | 07:35 | |
*** Landon has joined #tp | 07:38 | |
* llnz wanders off | 08:19 | |
llnz | later all | 08:19 |
*** llnz has quit IRC | 08:19 | |
*** alanp_ has joined #tp | 08:48 | |
*** tansell_laptop has quit IRC | 10:05 | |
*** Erroneous has quit IRC | 10:05 | |
*** gau_veldt has quit IRC | 10:05 | |
*** tansell_laptop has joined #tp | 10:05 | |
*** Erroneous has joined #tp | 10:05 | |
*** gau_veldt has joined #tp | 10:05 | |
*** DTRemenak has joined #tp | 10:13 | |
*** jmtan has joined #tp | 10:13 | |
CIA-21 | landon tpclient-pyogre * re5d5072ad0c5 /media/battleviewer/battleviewer.layout: Changed the FrameWindow the controls were in to a DragContainer to minimize the UI as well as avoid sizing issues with SleekSpace FrameWindows | 10:26 |
CIA-21 | landon tpclient-pyogre * r3999e91de5a5 /src/battleviewer.py: Bound methods to the controls buttons only | 10:26 |
tansell | Landon, so does that mean we can view a battle now? | 10:27 |
Landon | sort of a little | 10:28 |
Landon | 1) it only goes forward, 2) no timing between events in rounds | 10:28 |
Landon | but it will show you the end result of each round :P | 10:28 |
alanp_ | hey guys | 10:29 |
*** Erroneous has quit IRC | 10:30 | |
tansell | timing between events in rounds? | 10:31 |
Landon | tansell: well right now everything happens at once | 10:31 |
tansell | Landon, well that is kinda what I would expect? | 10:32 |
Landon | everyone fires at once, dies at once, only the last log message is shown | 10:32 |
Landon | it's confusing | 10:32 |
tansell | Landon, everyone firing at once is what I think is correct thing to do | 10:33 |
tansell | but I guess it should be something like | 10:33 |
tansell | move -> fire -> death | 10:33 |
tansell | (IE everyone moves, then everyone fires, then everyone dies) | 10:33 |
tansell | (in one round that is) | 10:33 |
Landon | hm ok | 10:34 |
Landon | but, it gets confusing when a laser beam blips out of nowhere too fast to see and kills someone off in the round :P | 10:34 |
Landon | that's the sort of thing I need to introduce timing for | 10:34 |
tansell | well the laser should take a certain amount of time to charge up, and then fire right? | 10:36 |
Landon | right | 10:37 |
tansell | (ie the animation takes a certain amount of time) | 10:37 |
tansell | same with the death animation | 10:37 |
tansell | I think it's important to show if two ships are recorded as firing at the same time, they should fire at the same time | 10:38 |
tansell | right? | 10:38 |
Landon | yeah | 10:38 |
tansell | of course that'll be a little tricky if the animation for different weapons take a different amount of time, but I don't think we should worry about that too much | 10:40 |
Landon | well, there are no animations right now | 10:41 |
Landon | but when there are it'd still probably be appropriate to have timing unrelated to animations, like keeping a laser focused on the target for a second or two | 10:42 |
tansell | I think that is just part of the animation | 10:42 |
tansell | the first animation you have of course is "nothing happens for 5 seconds" | 10:42 |
tansell | you know what I mean? | 10:42 |
Landon | ah ok | 10:43 |
tansell | I was thinking that eventually you probably want to have "hyperspace in" type animations at the start too :) | 10:45 |
tansell | anyway I should have been in bed like 1.5 hours ago | 10:46 |
tansell | and I'm still at work | 10:46 |
alanp_ | hehe | 10:49 |
CIA-21 | landon tpclient-pyogre * r348be1c953d5 / (media/battleviewer/battleviewer.layout src/battleviewer.py): Logs now use the MultiLineEditBox to display | 11:28 |
CIA-21 | landon tpclient-pyogre * r01d6f00f49a6 /src/battleviewer.py: Logs box now scrolls to the bottom automatically when new logs are added | 11:28 |
* Landon -> bed now | 11:29 | |
Landon | hey jmtan, I'm going to be home this weekend (leaving tomorrow morning I think) | 11:29 |
Landon | I might be able to get some work done at home, but probably won't be able to test it | 11:30 |
jmtan | Landon: alright, sure | 11:30 |
*** greenlion has joined #tp | 12:19 | |
*** epyon has joined #tp | 14:56 | |
*** alanp_ has quit IRC | 15:15 | |
*** jmtan has quit IRC | 15:19 | |
CIA-21 | epyon tpserver-cpp-refactor * r4d7e83bd7b09 / (Doxyfile tpserver/frame.cpp tpserver/frame.h): | 16:01 |
CIA-21 | Frame update | 16:01 |
CIA-21 | * Removed dangerous frame methods | 16:01 |
CIA-21 | * updated doxyfile to generate full documentation for tpserver only | 16:01 |
CIA-21 | epyon tpserver-cpp-refactor * r237775c7ed32 /tpserver/ (10 files): String packing into frame had completely unneccesary passed char* | 16:10 |
*** llnz has joined #tp | 16:13 | |
llnz | epyon: so to pack arbitrary data into the frame, i should put it in a string first, then pack the string? | 16:22 |
epyon | llnz: basically char* data, and int len is evil | 16:23 |
epyon | use std::string everywhere | 16:23 |
llnz | ok | 16:23 |
epyon | I'll try to do that for the existing code, but it will create a bughell --- | 16:23 |
epyon | -_- | 16:23 |
epyon | but it'll be far more stable afterwards and valgrind-safe :P | 16:24 |
CIA-21 | epyon tpserver-cpp-refactor * r7a758afe2966 /tpserver/ (Makefile.am packable.h): Packable trait class added | 16:25 |
llnz | i know packData wasn't used yet | 16:25 |
epyon | triple inheritance... duh | 16:29 |
epyon | But this will be fixed afterwards when I move all traits to a single ProtocolObject class | 16:29 |
CIA-21 | epyon tpserver-cpp-refactor * rbf2d10ed0d93 /tpserver/packable.h: Fix virtuality of pack | 16:39 |
CIA-21 | epyon tpserver-cpp-refactor * r13ce4b68ed87 /tpserver/ (board.cpp board.h playeragent.cpp): Board now derives from packable | 16:40 |
CIA-21 | epyon tpserver-cpp-refactor * rf660e3acd99e /tpserver/ (message.cpp message.h): Message now is packable | 16:40 |
CIA-21 | epyon tpserver-cpp-refactor * r5dc915f1812d /tpserver/ (category.cpp category.h playeragent.cpp): Category now is packable | 17:02 |
CIA-21 | epyon tpserver-cpp-refactor * r5905255d6763 /tpserver/ (Makefile.am protocolobject.h): ProtocolObject class added -- currently more of a stub | 17:02 |
CIA-21 | epyon tpserver-cpp-refactor * r9dcc9af9dec4 /tpserver/ (board.cpp board.h): Board made a ProtocolObject | 17:02 |
CIA-21 | epyon tpserver-cpp-refactor * r0cb18d4b6592 /tpserver/ (board.cpp protocolobject.h): ProtocolObject now stores frametype and can pack a headerwq | 17:02 |
CIA-21 | epyon tpserver-cpp-refactor * r422fcad0dc60 /tpserver/ (category.cpp category.h): Category made a ProtocolObject | 17:24 |
CIA-21 | epyon tpserver-cpp-refactor * rdf243d276f27 /tpserver/ (player.cpp player.h playeragent.cpp): Player is now a ProtocolObject | 17:24 |
CIA-21 | epyon tpserver-cpp-refactor * ra095b208eb61 /tpserver/ (frame.cpp frame.h playeragent.cpp property.cpp property.h): | 17:24 |
CIA-21 | Property now a ProtocolObject | 17:24 |
CIA-21 | * also added packIdSet to Frame | 17:24 |
*** greenlion has quit IRC | 17:43 | |
CIA-21 | epyon tpserver-cpp-refactor * r8ea6fb8d2af6 /tpserver/ (Makefile.am protocolview.h): ProtocolView base class added | 17:50 |
CIA-21 | epyon tpserver-cpp-refactor * rf36cdda64318 /tpserver/ (componentview.cpp componentview.h protocolview.h): ComponentView a ProtocolView + minor refactoring | 18:33 |
CIA-21 | epyon tpserver-cpp-refactor * rea5680cb73b7 /tpserver/ (designview.cpp designview.h playerview.cpp protocolview.h): DesignView a ProtocolView + minor refactoring | 18:33 |
CIA-21 | epyon tpserver-cpp-refactor * r0498311809f1 /tpserver/ (objectview.cpp objectview.h): ObjectView a ProtocolView + minor refactoring | 18:33 |
*** Greywhind has joined #tp | 18:52 | |
CIA-21 | epyon tpserver-cpp-refactor * r6dee72552b71 /tpserver/ (7 files): ProtocolViews now have proper pack instead of packFrame | 18:59 |
CIA-21 | epyon tpserver-cpp-refactor * r9c883b9a693f /tpserver/ (objectview.cpp objectview.h playerview.cpp): PlayerView updates | 18:59 |
CIA-21 | epyon tpserver-cpp-refactor * r863ddf8ef350 /modules/games/ (minisec/minisec.cpp rfts/rfts.cpp tae/fleetbuilder.cpp): Fixes for the rulesets | 19:09 |
epyon | duh | 19:10 |
*** nash has joined #tp | 19:16 | |
Greywhind | hm | 19:31 |
Greywhind | anyone know what might cause this: | 19:31 |
Greywhind | config.status: error: cannot find input file: libltdl/Makefile.in | 19:31 |
Greywhind | when compiling tpserver-cpp | 19:31 |
Greywhind | wait, sorry, new error instead: | 19:32 |
Greywhind | === configuring in libltdl (/home/Greywhind/Programming/tp04/tpserver-cpp/libltdl) | 19:32 |
Greywhind | configure: WARNING: no configuration information is in libltdl | 19:32 |
llnz | ummm... | 19:32 |
Greywhind | i do have libtool-ltdl installed, and libtool-ldtl-devel | 19:32 |
llnz | maybe you messed up the libltdl directory, try to revert any changes to it, then run autogen.sh and configure again | 19:32 |
Greywhind | well, all i did was update to Fedora 11 | 19:33 |
Greywhind | i'll try doing a git reset --hard | 19:33 |
Greywhind | nope, after git reset --hard and ./autogen.sh | 19:34 |
Greywhind | ./configure still gives: | 19:34 |
Greywhind | config.status: executing libtool commands | 19:34 |
Greywhind | === configuring in libltdl (/home/Greywhind/Programming/tp04/tpserver-c | 19:34 |
Greywhind | l) | 19:34 |
Greywhind | configure: WARNING: no configuration information is in libltdl | 19:34 |
llnz | odd | 19:35 |
llnz | are there change in libltdl? (git status to check) | 19:35 |
Greywhind | well, there weren't after the git reset | 19:36 |
Greywhind | but now: | 19:36 |
Greywhind | [Greywhind@localhost tpserver-cpp]$ git status | 19:36 |
Greywhind | # On branch multiqueue | 19:36 |
Greywhind | # Changed but not updated: | 19:36 |
Greywhind | # (use "git add/rm <file>..." to update what will be committed) | 19:36 |
Greywhind | # (use "git checkout -- <file>..." to discard changes in working directory) | 19:36 |
Greywhind | # | 19:36 |
Greywhind | #modified: libltdl/Makefile.am | 19:36 |
Greywhind | #modified: libltdl/Makefile.in | 19:36 |
Greywhind | #modified: libltdl/README | 19:36 |
Greywhind | #deleted: libltdl/acinclude.m4 | 19:36 |
Greywhind | #deleted: libltdl/aclocal.m4 | 19:37 |
Greywhind | #deleted: libltdl/config-h.in | 19:37 |
Greywhind | #deleted: libltdl/configure | 19:37 |
Greywhind | #deleted: libltdl/configure.ac | 19:37 |
Greywhind | #modified: libltdl/ltdl.c | 19:37 |
Greywhind | #modified: libltdl/ltdl.h | 19:37 |
llnz | pastebin would have been a good idea (or just the changed lines) | 19:37 |
llnz | revert just configure and configure.ac | 19:38 |
llnz | it appears that your version of libtoolize is not creating them | 19:38 |
CIA-21 | epyon tpserver-cpp-refactor * r72e9e50a1f14 /tpserver/ (3 files): ResouceDescription now a ProtocolObject | 19:39 |
CIA-21 | epyon tpserver-cpp-refactor * rd95285d80826 /tpserver/resourcedescription.h: ResouceDescription -- removed unneccesary declarations | 19:39 |
CIA-21 | epyon tpserver-cpp-refactor * rab706c4a2299 /tpserver/ (7 files): OrderParameter now a Describable object | 19:39 |
Greywhind | llnz: reverting just those two files seems to fix the problem | 19:39 |
Greywhind | wait, not quite | 19:39 |
Greywhind | configure: creating ./config.status | 19:39 |
Greywhind | config.status: creating Makefile | 19:39 |
Greywhind | config.status: error: cannot find input file: config-h.in | 19:39 |
Greywhind | configure: error: ./configure failed for libltdl | 19:39 |
llnz | it might be all the files that were deleted | 19:40 |
Greywhind | i'll try reverting that one too | 19:40 |
llnz | also can you *email* me the output for libtoolize --help please? | 19:41 |
Greywhind | ok | 19:41 |
Greywhind | [Greywhind@localhost tpserver-cpp]$ make | 19:41 |
Greywhind | Making all in libltdl | 19:41 |
Greywhind | make[2]: *** No rule to make target `@MAINTAINER_MODE_TRUE@', needed by `Makefile.in'. Stop. | 19:41 |
llnz | oh, and libtoolize --version | 19:42 |
Greywhind | ok | 19:42 |
llnz | mhilmi: i have you email, will reply shortly | 19:42 |
llnz | s/you/your/ | 19:43 |
Greywhind | llnz: e-mailed | 19:43 |
llnz | Greywhind: thanks | 19:43 |
CIA-21 | epyon tpserver-cpp-refactor * rf1f65ce31838 / (4 files in 2 dirs): Objects now don't have setID | 19:59 |
tansell | Greywhind, why in gods name would you run fedora? | 20:15 |
llnz | hummm... | 20:17 |
llnz | Greywhind: can you edit autogen.sh and change libtoolize --automake --copy --force --ltdl --recursive to libtoolize --automake --copy --force --ltdl --subproject | 20:18 |
llnz | then run autogen.sh and configure | 20:19 |
*** mithro has quit IRC | 20:35 | |
*** mithro has joined #tp | 20:36 | |
Greywhind | llnz: will do | 20:44 |
Greywhind | tansell: Because Ubuntu didn't work on this machine the first time I tried to install it (new hardware and all), and I haven't bothered to reinstall now that the new version would work | 20:45 |
Greywhind | llnz: that got me to here, still: | 20:47 |
Greywhind | [Greywhind@localhost tpserver-cpp]$ make | 20:47 |
Greywhind | Making all in libltdl | 20:47 |
Greywhind | make[2]: *** No rule to make target `@MAINTAINER_MODE_TRUE@', needed by `Makefile.in'. Stop. | 20:47 |
llnz | odd, i don't know what that means | 20:47 |
llnz | though good about the changing recursive to subproject solved part of the issue | 20:48 |
Greywhind | yeah | 20:48 |
tansell_laptop | cherez, all sorted now? | 21:04 |
cherez | Sorted? | 21:04 |
CIA-21 | epyon tpserver-cpp-refactor * r5cbc1490c682 /tpserver/protocolobject.h: getFrameType added to ProtocolObject | 21:05 |
CIA-21 | epyon tpserver-cpp-refactor * r456d38d71084 /tpserver/ (persistence.cpp persistence.h): Persistence has generic View retrieval and save | 21:05 |
CIA-21 | epyon tpserver-cpp-refactor * r96e347204a7f /tpserver/ (playerview.cpp playerview.h): PlayerView now has a generic addVisible | 21:05 |
CIA-21 | epyon tpserver-cpp-refactor * r2d52c005e49e /tpserver/ (playerview.cpp playerview.h protocolobject.h): PlayerView has generic View retrieval | 21:05 |
CIA-21 | epyon tpserver-cpp-refactor * r21a0dd3b4552 /tpserver/ (Makefile.am protocolobject.cpp protocolobject.h): Correction to ProtocolObject linkage | 21:05 |
*** epyon has quit IRC | 21:13 | |
Greywhind | llnz: any ideas? | 21:15 |
tansell_laptop | Greywhind, can you try a fresh checkout? | 21:21 |
Greywhind | tansell_laptop: i'll try | 21:21 |
Greywhind | tansell_laptop: still the same | 22:31 |
llnz | Greywhind: what version of automake? | 22:35 |
Greywhind | automake (GNU automake) 1.11 | 22:35 |
llnz | i believe i fun automake 1.9 (or 1.7) | 22:37 |
llnz | or 1.10 | 22:37 |
llnz | s/fun/run/ | 22:37 |
*** DTRemenak has quit IRC | 23:11 | |
*** jmtan has joined #tp | 23:22 | |
tansell | wtf is up with the automake people recently? | 23:31 |
tansell | they seem to be breaking everything with each new version | 23:31 |
* llnz wanders off | 23:58 | |
llnz | busy weekend, not sure when I will be around on IRC | 23:58 |
*** llnz has quit IRC | 23:58 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!