Thursday, 2009-07-02

*** tpb has joined #tp00:00
*** ChanServ sets mode: +o tpb00:00
*** tpb has joined #tp00:08
*** ChanServ sets mode: +o tpb00:08
*** CIA-21 has joined #tp00:11
tansellGreywhind, ping?00:11
* tansell pokes Greywhind with something sharp and pointy00:14
*** DTRemenak|laptop has quit IRC00:25
chereztansell: In a description frame, where do the descstructs of the parameters go?00:32
tanselltoday has been a day full of bugs00:33
tansellcherez, they go as part of the00:34
tansell1561                 <list>00:34
tansell1562                         <name>parameters</name00:34
tansellbut there is nothing in the protocol.xml which tells you that :(00:34
cherezGah. :(00:34
tanselloh00:36
tansellactually I lie00:36
tansell1369                                                 <descparameter ref="ObjectParams" typefield="type">00:36
tansell1370                                                         <name>extradata</name>00:36
tansell1371                                                         <longname>extra data for this parameter</longname>00:36
tansell1372                                                 </descparameter00:36
cherezAhh.00:37
cherezSo the descstruct is just stuck in there?00:38
cherezMeaning it's a heterogeneous list?00:38
*** tansell_laptop has quit IRC00:40
tansellheterogeneous?00:40
tansellremember the code I was showing you yesterday in libtpproto-py?00:41
cherezYeah, I'm pulling it up.00:41
cherezYep, that's what I thought....00:43
*** nash has joined #tp00:44
cherezThe length of the list is still prepended to it, right?00:45
tansellthe length of the list the the number of elements00:49
*** tansell_laptop has joined #tp00:54
cherezWould it be legal if protocol.xml had a packet with multiple descparameters?00:57
tansellcherez, possibly - but unlikely01:02
tansellcherez, It would be okay to not support that for now01:02
cherezGoody.01:22
*** jmtan has joined #tp01:22
*** nash has quit IRC01:51
*** nash has joined #tp01:57
Greywhindtansell: pong02:00
tansellGreywhind, so we where suppose to have a meeting about 2 hours ago?02:01
Greywhindtansell: sorry about that, it was my Dad's birthday and I took him out to a movie02:01
Greywhindi totally forgot :(02:01
tansellGreywhind, no problem - want to do it now?02:01
Greywhindtansell: sure - want me to write a summary real fast?02:01
tansellyes please02:02
tansellcherez, so I'm expecting a blog post tonight right?02:06
tansellmhilmi, poke?02:07
tansellJLP, poke?02:08
chereztansell: 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 IRC02:10
*** llnz has joined #tp02:11
tansellGreywhind, you fall asleep at your computer :)02:25
Greywhindtansell: http://greywhind.wordpress.com/2009/07/01/gsoc-week-6-summary/02:25
Greywhindjust finished02:25
tpb<http://ln-s.net/3RCB> (at greywhind.wordpress.com)02:25
Greywhindtakes a bit of hunting to figure out all the things i did and write about them :P02:26
tansellgit log is handy for that :)02:27
Greywhindtrue - i used the blog posts and code reviews02:28
chereztansell: The parameters of a packet that has use parameters are always appended at the end?02:29
tansellcherez, yes02:29
cherezGreat.02:29
*** nash has joined #tp02:30
tansellcherez, assuming I understand what you are saying02:30
tansellGreywhind, so after looking at the Design window today, it could use some love02:31
Greywhindtansell: ok02:31
cherezThe 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
Greywhindtansell: what does it need, exactly?02:31
tansellGreywhind, "although it no longer has a type column" <- Umm they sure do have names now02:32
tansellGreywhind, mainly that it's ugly :)02:32
tansellthe tree on the right side should be hidden until it's needed02:33
Greywhindtansell: hm... where can i get the names?02:33
tansellthe Order/ObjectDesc has the name02:33
Greywhindok, i'll look into it02:33
tansellin the .name value02:33
tansellcherez, yes02:33
cherezGood, that's easy.02:34
tansellcherez, so you can actually use the ObjectDesc/OrderDesc to generate a new python class which has the appropriate structures02:35
cherezRight, that's the plan.02:35
tansellGreywhind, so - what is the plan for next week?02:42
Greywhindtansell: well, i want to do more cleaning02:42
Greywhindon the design window, since you want that cleaned02:42
tansellGreywhind, oh the duplicate button on the design window doesn't work either02:42
Greywhindand then on anything else i can find02:42
tansellGreywhind, you want to forward port the patches I pushed today02:42
Greywhindtansell: ok, i'll make sure to do that02:43
tansellGreywhind, you should just be able to cherry pick them into the tp04 branches02:43
Greywhindok02:43
tansellor you could try doing a merge02:43
Greywhindok02:44
Greywhindi'll probably try the merge02:44
chereztansell: Should Structures know how to pack/unpack themselves?02:46
tansellcherez, they do already right?02:47
cherezThey know their xstruct string, but that's it as far as I know.02:47
cherezShould I be adding pack and unpack methods?02:48
tansellcherez, well it's more efficient to have one pack/unpack method which just calls xstruct02:48
tansellrather then doing a loop02:48
cherezYes, 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
cherezSo at the least it will have to iterate through that list and determine the structure is for each element of the list.02:52
tansellcherez,  you can use the same method the xstruct in the tp04 branch does02:56
tansellcherez, you might want to merge the changes in the tp04 xstruct into your xstruct03:02
cherezYeah, I just took a look at it.03:02
cherezI've got some unit tests to change.03:03
tansellit should be pretty simple to merge03:07
*** nash has quit IRC03:23
*** greenlion has joined #tp03:34
*** Greywhind has quit IRC04:01
*** greenlion has quit IRC04:42
*** jmtan has quit IRC06:51
*** Landon has quit IRC07:35
*** Landon has joined #tp07:38
* llnz wanders off08:19
llnzlater all08:19
*** llnz has quit IRC08:19
*** alanp_ has joined #tp08:48
*** tansell_laptop has quit IRC10:05
*** Erroneous has quit IRC10:05
*** gau_veldt has quit IRC10:05
*** tansell_laptop has joined #tp10:05
*** Erroneous has joined #tp10:05
*** gau_veldt has joined #tp10:05
*** DTRemenak has joined #tp10:13
*** jmtan has joined #tp10:13
CIA-21landon 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 FrameWindows10:26
CIA-21landon tpclient-pyogre * r3999e91de5a5 /src/battleviewer.py: Bound methods to the controls buttons only10:26
tansellLandon, so does that mean we can view a battle now?10:27
Landonsort of a little10:28
Landon1) it only goes forward, 2) no timing between events in rounds10:28
Landonbut it will show you the end result of each round :P10:28
alanp_hey guys10:29
*** Erroneous has quit IRC10:30
tanselltiming between events in rounds?10:31
Landontansell: well right now everything happens at once10:31
tansellLandon, well that is kinda what I would expect?10:32
Landoneveryone fires at once, dies at once, only the last log message is shown10:32
Landonit's confusing10:32
tansellLandon, everyone firing at once is what I think is correct thing to do10:33
tansellbut I guess it should be something like10:33
tansellmove -> fire -> death10:33
tansell(IE everyone moves, then everyone fires, then everyone dies)10:33
tansell(in one round that is)10:33
Landonhm ok10:34
Landonbut, it gets confusing when a laser beam blips out of nowhere too fast to see and kills someone off in the round :P10:34
Landonthat's the sort of thing I need to introduce timing for10:34
tansellwell the laser should take a certain amount of time to charge up, and then fire right?10:36
Landonright10:37
tansell(ie the animation takes a certain amount of time)10:37
tansellsame with the death animation10:37
tansellI think it's important to show if two ships are recorded as firing at the same time, they should fire at the same time10:38
tansellright?10:38
Landonyeah10:38
tansellof 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 much10:40
Landonwell, there are no animations right now10:41
Landonbut 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 two10:42
tansellI think that is just part of the animation10:42
tansellthe first animation you have of course is "nothing happens for 5 seconds"10:42
tansellyou know what I mean?10:42
Landonah ok10:43
tansellI was thinking that eventually you probably want to have "hyperspace in" type animations at the start too :)10:45
tansellanyway I should have been in bed like 1.5 hours ago10:46
tanselland I'm still at work10:46
alanp_hehe10:49
CIA-21landon tpclient-pyogre * r348be1c953d5 / (media/battleviewer/battleviewer.layout src/battleviewer.py): Logs now use the MultiLineEditBox to display11:28
CIA-21landon tpclient-pyogre * r01d6f00f49a6 /src/battleviewer.py: Logs box now scrolls to the bottom automatically when new logs are added11:28
* Landon -> bed now11:29
Landonhey jmtan, I'm going to be home this weekend (leaving tomorrow morning I think)11:29
LandonI might be able to get some work done at home, but probably won't be able to test it11:30
jmtanLandon: alright, sure11:30
*** greenlion has joined #tp12:19
*** epyon has joined #tp14:56
*** alanp_ has quit IRC15:15
*** jmtan has quit IRC15:19
CIA-21epyon tpserver-cpp-refactor * r4d7e83bd7b09 / (Doxyfile tpserver/frame.cpp tpserver/frame.h):16:01
CIA-21Frame update16:01
CIA-21* Removed dangerous frame methods16:01
CIA-21* updated doxyfile to generate full documentation for tpserver only16:01
CIA-21epyon tpserver-cpp-refactor * r237775c7ed32 /tpserver/ (10 files): String packing into frame had completely unneccesary passed char*16:10
*** llnz has joined #tp16:13
llnzepyon: so to pack arbitrary data into the frame, i should put it in a string first, then pack the string?16:22
epyonllnz: basically char* data, and int len is evil16:23
epyonuse std::string everywhere16:23
llnzok16:23
epyonI'll try to do that for the existing code, but it will create a bughell ---16:23
epyon-_-16:23
epyonbut it'll be far more stable afterwards and valgrind-safe :P16:24
CIA-21epyon tpserver-cpp-refactor * r7a758afe2966 /tpserver/ (Makefile.am packable.h): Packable trait class added16:25
llnzi know packData wasn't used yet16:25
epyontriple inheritance... duh16:29
epyonBut this will be fixed afterwards when I move all traits to a single ProtocolObject class16:29
CIA-21epyon tpserver-cpp-refactor * rbf2d10ed0d93 /tpserver/packable.h: Fix virtuality of pack16:39
CIA-21epyon tpserver-cpp-refactor * r13ce4b68ed87 /tpserver/ (board.cpp board.h playeragent.cpp): Board now derives from packable16:40
CIA-21epyon tpserver-cpp-refactor * rf660e3acd99e /tpserver/ (message.cpp message.h): Message now is packable16:40
CIA-21epyon tpserver-cpp-refactor * r5dc915f1812d /tpserver/ (category.cpp category.h playeragent.cpp): Category now is packable17:02
CIA-21epyon tpserver-cpp-refactor * r5905255d6763 /tpserver/ (Makefile.am protocolobject.h): ProtocolObject class added -- currently more of a stub17:02
CIA-21epyon tpserver-cpp-refactor * r9dcc9af9dec4 /tpserver/ (board.cpp board.h): Board made a ProtocolObject17:02
CIA-21epyon tpserver-cpp-refactor * r0cb18d4b6592 /tpserver/ (board.cpp protocolobject.h): ProtocolObject now stores frametype and can pack a headerwq17:02
CIA-21epyon tpserver-cpp-refactor * r422fcad0dc60 /tpserver/ (category.cpp category.h): Category made a ProtocolObject17:24
CIA-21epyon tpserver-cpp-refactor * rdf243d276f27 /tpserver/ (player.cpp player.h playeragent.cpp): Player is now a ProtocolObject17:24
CIA-21epyon tpserver-cpp-refactor * ra095b208eb61 /tpserver/ (frame.cpp frame.h playeragent.cpp property.cpp property.h):17:24
CIA-21Property now a ProtocolObject17:24
CIA-21* also added packIdSet to Frame17:24
*** greenlion has quit IRC17:43
CIA-21epyon tpserver-cpp-refactor * r8ea6fb8d2af6 /tpserver/ (Makefile.am protocolview.h): ProtocolView base class added17:50
CIA-21epyon tpserver-cpp-refactor * rf36cdda64318 /tpserver/ (componentview.cpp componentview.h protocolview.h): ComponentView a ProtocolView + minor refactoring18:33
CIA-21epyon tpserver-cpp-refactor * rea5680cb73b7 /tpserver/ (designview.cpp designview.h playerview.cpp protocolview.h): DesignView a ProtocolView + minor refactoring18:33
CIA-21epyon tpserver-cpp-refactor * r0498311809f1 /tpserver/ (objectview.cpp objectview.h): ObjectView a ProtocolView + minor refactoring18:33
*** Greywhind has joined #tp18:52
CIA-21epyon tpserver-cpp-refactor * r6dee72552b71 /tpserver/ (7 files): ProtocolViews now have proper pack instead of packFrame18:59
CIA-21epyon tpserver-cpp-refactor * r9c883b9a693f /tpserver/ (objectview.cpp objectview.h playerview.cpp): PlayerView updates18:59
CIA-21epyon tpserver-cpp-refactor * r863ddf8ef350 /modules/games/ (minisec/minisec.cpp rfts/rfts.cpp tae/fleetbuilder.cpp): Fixes for the rulesets19:09
epyonduh19:10
*** nash has joined #tp19:16
Greywhindhm19:31
Greywhindanyone know what might cause this:19:31
Greywhindconfig.status: error: cannot find input file: libltdl/Makefile.in19:31
Greywhindwhen compiling tpserver-cpp19:31
Greywhindwait, sorry, new error instead:19:32
Greywhind=== configuring in libltdl (/home/Greywhind/Programming/tp04/tpserver-cpp/libltdl)19:32
Greywhindconfigure: WARNING: no configuration information is in libltdl19:32
llnzummm...19:32
Greywhindi do have libtool-ltdl installed, and libtool-ldtl-devel19:32
llnzmaybe you messed up the libltdl directory, try to revert any changes to it, then run autogen.sh and configure again19:32
Greywhindwell, all i did was update to Fedora 1119:33
Greywhindi'll try doing a git reset --hard19:33
Greywhindnope, after git reset --hard and ./autogen.sh19:34
Greywhind./configure still gives:19:34
Greywhindconfig.status: executing libtool commands19:34
Greywhind=== configuring in libltdl (/home/Greywhind/Programming/tp04/tpserver-c19:34
Greywhindl)19:34
Greywhindconfigure: WARNING: no configuration information is in libltdl19:34
llnzodd19:35
llnzare there change in libltdl? (git status to check)19:35
Greywhindwell, there weren't after the git reset19:36
Greywhindbut now:19:36
Greywhind[Greywhind@localhost tpserver-cpp]$ git status19:36
Greywhind# On branch multiqueue19: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.am19:36
Greywhind#modified:   libltdl/Makefile.in19:36
Greywhind#modified:   libltdl/README19:36
Greywhind#deleted:    libltdl/acinclude.m419:36
Greywhind#deleted:    libltdl/aclocal.m419:37
Greywhind#deleted:    libltdl/config-h.in19:37
Greywhind#deleted:    libltdl/configure19:37
Greywhind#deleted:    libltdl/configure.ac19:37
Greywhind#modified:   libltdl/ltdl.c19:37
Greywhind#modified:   libltdl/ltdl.h19:37
llnzpastebin would have been a good idea (or just the changed lines)19:37
llnzrevert just configure and configure.ac19:38
llnzit appears that your version of libtoolize is not creating them19:38
CIA-21epyon tpserver-cpp-refactor * r72e9e50a1f14 /tpserver/ (3 files): ResouceDescription now a ProtocolObject19:39
CIA-21epyon tpserver-cpp-refactor * rd95285d80826 /tpserver/resourcedescription.h: ResouceDescription -- removed unneccesary declarations19:39
CIA-21epyon tpserver-cpp-refactor * rab706c4a2299 /tpserver/ (7 files): OrderParameter now a Describable object19:39
Greywhindllnz: reverting just those two files seems to fix the problem19:39
Greywhindwait, not quite19:39
Greywhindconfigure: creating ./config.status19:39
Greywhindconfig.status: creating Makefile19:39
Greywhindconfig.status: error: cannot find input file: config-h.in19:39
Greywhindconfigure: error: ./configure failed for libltdl19:39
llnzit might be all the files that were deleted19:40
Greywhindi'll try reverting that one too19:40
llnzalso can you *email* me the output for libtoolize --help please?19:41
Greywhindok19:41
Greywhind[Greywhind@localhost tpserver-cpp]$ make19:41
GreywhindMaking all in libltdl19:41
Greywhindmake[2]: *** No rule to make target `@MAINTAINER_MODE_TRUE@', needed by `Makefile.in'.  Stop.19:41
llnzoh, and libtoolize --version19:42
Greywhindok19:42
llnzmhilmi: i have you email, will reply shortly19:42
llnzs/you/your/19:43
Greywhindllnz: e-mailed19:43
llnzGreywhind: thanks19:43
CIA-21epyon tpserver-cpp-refactor * rf1f65ce31838 / (4 files in 2 dirs): Objects now don't have setID19:59
tansellGreywhind, why in gods name would you run fedora?20:15
llnzhummm...20:17
llnzGreywhind: can you edit autogen.sh and change libtoolize --automake --copy --force --ltdl --recursive to libtoolize --automake --copy --force --ltdl --subproject20:18
llnzthen run autogen.sh and configure20:19
*** mithro has quit IRC20:35
*** mithro has joined #tp20:36
Greywhindllnz: will do20:44
Greywhindtansell: 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 work20:45
Greywhindllnz: that got me to here, still:20:47
Greywhind[Greywhind@localhost tpserver-cpp]$ make20:47
GreywhindMaking all in libltdl20:47
Greywhindmake[2]: *** No rule to make target `@MAINTAINER_MODE_TRUE@', needed by `Makefile.in'.  Stop.20:47
llnzodd, i don't know what that means20:47
llnzthough good about the changing recursive to subproject solved part of the issue20:48
Greywhindyeah20:48
tansell_laptopcherez, all sorted now?21:04
cherezSorted?21:04
CIA-21epyon tpserver-cpp-refactor * r5cbc1490c682 /tpserver/protocolobject.h: getFrameType added to ProtocolObject21:05
CIA-21epyon tpserver-cpp-refactor * r456d38d71084 /tpserver/ (persistence.cpp persistence.h): Persistence has generic View retrieval and save21:05
CIA-21epyon tpserver-cpp-refactor * r96e347204a7f /tpserver/ (playerview.cpp playerview.h): PlayerView now has a generic addVisible21:05
CIA-21epyon tpserver-cpp-refactor * r2d52c005e49e /tpserver/ (playerview.cpp playerview.h protocolobject.h): PlayerView has generic View retrieval21:05
CIA-21epyon tpserver-cpp-refactor * r21a0dd3b4552 /tpserver/ (Makefile.am protocolobject.cpp protocolobject.h): Correction to ProtocolObject linkage21:05
*** epyon has quit IRC21:13
Greywhindllnz: any ideas?21:15
tansell_laptopGreywhind, can you try a fresh checkout?21:21
Greywhindtansell_laptop: i'll try21:21
Greywhindtansell_laptop: still the same22:31
llnzGreywhind: what version of automake?22:35
Greywhindautomake (GNU automake) 1.1122:35
llnzi believe i fun automake 1.9 (or 1.7)22:37
llnzor 1.1022:37
llnzs/fun/run/22:37
*** DTRemenak has quit IRC23:11
*** jmtan has joined #tp23:22
tansellwtf is up with the automake people recently?23:31
tansellthey seem to be breaking everything with each new version23:31
* llnz wanders off23:58
llnzbusy weekend, not sure when I will be around on IRC23:58
*** llnz has quit IRC23:58

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!