Monday, 2010-06-21

*** tpb has joined #tp00:00
*** ChanServ sets mode: +o tpb00:00
*** StupidIncarnate has joined #tp00:15
alanpsup01:23
llnzhi alanp01:36
*** tansell-laptop has joined #tp01:49
alanphey llnz02:31
alanpbig excitement in NZ today?02:31
llnzhehe, yeah02:31
alanp2 points, not bad02:31
llnz:-)02:33
*** StupidIncarnate has quit IRC02:34
llnzhow is it going alanp?02:50
*** StupidIncarnate has joined #tp02:58
alanpnot bad03:07
alanpyou?03:07
*** StupidIncarnate has quit IRC03:08
llnznot too bad03:09
llnzit was good to crush some tpserver-cpp bugs on the weekend03:09
tansellllnz, did you get the one with time going backwards?03:36
llnzyes, not a tpserver-cpp bug, but in libtpclient-py03:37
alanpwas that 97?03:41
CIA-31alanp tpserver-cpp * r39b81eb414e3 /modules/games/mtsec/tpserver-cpp-mtsec-gamedata.xml: - Scheme fixes for colonisation module03:42
CIA-31alanp tpserver-cpp * r4364880ef74c /tpserver/ (11 files): Merge branch 'master' of git+ssh://git.thousandparsec.net/git/tpserver-cpp03:42
alanpwtf is that second commit03:43
tansellllnz, got a link to the fix?03:43
llnzhttp://git.thousandparsec.net/gitweb/gitweb.cgi?p=libtpclient-py.git;a=commit;h=b3ae9a0af1088f15e2d1d2be750e1353db24676803:44
tpb<http://ln-s.net/79OI> (at git.thousandparsec.net)03:44
llnzfour line addition03:44
llnzalanp: that for merging with master (probably a git pull after you have a local commit)03:45
alanpahh03:45
tansellwhy is 0 == noaccess?03:45
llnzbeacuse it's true03:46
llnzit occured to me this morning that it could have been len(allowed order types) == 003:46
llnzin fact len(allowed order types) == 0 and num_orders == 0 would work03:48
tansellOrder QueueAn order queue.03:53
tansell32 bit unsigned integer, the Maximum slots, The maximum number of slots that can be used in this queue.03:53
tansell32 bit unsigned integer, the Queue ID, The ID number of the queue.03:53
tansell32 bit unsigned integer, the Number of Orders, The number of orders in the queue.03:53
tansellIt doesn't really say that the Queue ID == 0 is no-access03:54
llnzthe guest user doesn't have access to any orderqueue - what should the orderqueueid be then?03:54
matthewdShould queue 0 be an empty queue that really exists, perhaps?03:56
llnzmatthewd: why should it exist?03:56
matthewd(So the fact that 0 implies no-access is purely a server-side convention)03:56
llnzah03:56
tansellyeah, that is what I was just thinking03:56
tansellthe problem is actually this03:57
tansell 688                         self.orders[queueid] = (self.objects[objectid].modify_time, subs)03:57
tansellorder queue's don't have their own mod-time03:57
llnzfucked if i know what was causing the problem with that line, the modtime for the getIds list and the object itself both come from the same value03:58
tansellso if two objects reference the same order queue, things go pair shaped03:58
llnzah03:58
llnzaaahhhhhh03:58
* llnz grumbles about wasting all that time on the weekend trying to find the problem03:59
tansellllnz, well - if you hadn't found that fix, I wouldn't have figured out the real cause03:59
tansellllnz, what do you think is the correct fix?04:06
llnzprobably for orderqueues to exist as first class objects04:10
llnzmaybe?04:10
*** StupidIncarnate has joined #tp04:10
llnzor the second don't fetch filter i suggested about with num_orders and order types04:11
tansellllnz, that was my thought04:17
tansellit would dramatically reduce the number of objects we need to get04:17
llnzmaybe not "dramatically"04:33
*** mithro has joined #tp04:35
llnzbbl04:39
*** llnz has quit IRC04:39
*** mithro has quit IRC04:40
*** mithro has joined #tp04:54
*** DTRemenak has quit IRC05:10
*** DTRemenak has joined #tp05:11
*** StupidIncarnate has quit IRC05:23
tansellezod, pong?05:24
tansellalanp, ping?05:24
alanptansell: pong05:24
tansellalanp, do you use tripit?05:24
alanpno05:25
alanpdo i need to?05:25
tansellwhen are you flying back home from the Thousand Parsec hackfest?05:25
alanpDeparture (SJC): August 14, 8:45 AM PDT (morning)05:26
alanpany problem?05:27
tansellLooks like I've book the hotel till the 13th rather then the 14th05:30
alanp:(05:30
alanpwhen does hackfest end05:30
tansellwell, the 13th is the last day of hacking05:31
alanpmight be able to couchsurf05:35
alanpgoing to have to leave for the airport around 430-50005:35
tansellI'm going to see if we can extend the reservation05:35
alanpwtf05:36
alanpthey must have changed the flight time since i bought them05:36
alanpnow it's showing 10am05:37
alanpyep, changed it05:38
alanpshorter plane change now though05:38
tansellalanp, if you put your trips in tripit, you can share them with me :)05:40
alanphehe05:40
alanp1min05:41
*** StupidIncarnate has joined #tp05:42
alanpi think i had the android client at one point but didn't want to send all the e-mails05:42
alanpwhat's your email on tripit?05:47
alanptansell: ^05:48
*** llnz has joined #tp06:02
tansellllnz, which day are you flying out on?06:11
tansell(out of SFO)06:11
llnzhumm...06:39
llnzlet me check06:39
tansellllnz, do you use tripit?06:43
alanptripit privacy concerns worry me, but it seems pretty ok06:44
llnzno, i don't06:46
llnznot good....06:53
tansellllnz, ?06:59
llnzfound it07:02
llnzi fly home the evening of 14 August07:02
*** DTRemenak has quit IRC07:13
*** DTRemenak has joined #tp07:17
tansellllnz, do you want to write up a proposal for how to fix the orderqueue thing?07:34
*** bisc has joined #tp08:01
llnztansell: i guess so08:04
biscmithro, tansell: ping08:17
tansellbisc, pong08:20
bisctansell: a question about creating the controls for sending messages. Wasn't it done earlier (saw winmessageboard.py, but didn't see usage of it)? And if it wasn't -- do we want it as a lightweigh frame or a dialog?08:23
tansellI think the idea was to create a proper message board type system08:24
bisctansell: the board should be a separate window or a panel?08:25
tansellwindow08:26
bisctansell: are these boards now supported by server?08:35
tansellnot sure08:38
bisctansell: hmm. So, I shouldn't work in the way of making message board work on client?08:42
*** null_000 has joined #tp08:47
*** null_000 has quit IRC09:09
tansellbisc, the message window in the client is to view "system" messages really, rather then viewing communication between players09:47
bisctansell: you mean the messages panel?  And the message board is intended for players communication?09:49
tansellyeah09:50
llnzplayer will (eventually) have access to many boards09:51
llnzone is the current system board, the others are shared with another player to allow them to communicate09:52
biscllnz: is this already working or just plans?09:52
biscllnz: or, maybe, partly working.09:53
llnzjust plans, but all protocol support is there09:53
llnzif there were plans to use it I might improve it and get it working09:53
biscllnz: In my opinion -- it would be great to have this functionality. Then I work on client part for this thing.09:56
llnzbisc: how do i test if it's working? i need to client to test it09:56
biscllnz: yes, I understand. Will see what's currently done there and make it formally show all available boards and their messages.09:58
llnzcool09:58
* llnz adds interplayer boards to his todo list09:59
*** tanoku has joined #tp10:01
bisctansell: can you please look at these patches (latest patch sets) when you have time http://codereview.mithis.com/68001 http://codereview.mithis.com/59001 http://codereview.mithis.com/5400210:01
tpbTitle: Issue 68001: Work on update window UI. - Code Review (at codereview.mithis.com)10:01
* llnz wanders off10:14
llnzlater all10:14
*** llnz has quit IRC10:14
tansellbisc, comments added to the last two10:18
bisctansell: replied to your comments. What about the first one?10:25
tansellbisc, if auto-connect is set to true, then the system will connect to the first server in the list10:34
bisctansell: I'm not operating with autoconnect parameter. That's just sorting of server list, and consequently setting the default selected server.10:35
*** Epyon has joined #tp10:36
bisctansell: it makes you have 127.0.0.1 selected by in the combobox if the previous time you've tried connecting to it.10:37
tansellbisc, if you have autoconnect on and then connect to a different server you'll change that server to be autoconnected too10:38
bisctansell: it10:41
bisctansell: yes, I know that. How does it affect my patch? I seem to be working independently with this logic.10:43
tansellif you are setting a new server as the default, shouldn't you ask?10:44
bisctansell: oh, I understand what you're saying.10:45
bisctansell: if the autoconnect is switched on, I should ask the user about changing the server, right?10:46
bisctansell: changing the _default_ server10:46
bisctansell: the code, governing the autoconnect selection, is strange. it checks all servers and finds the one with autoconnect. And if not found sets the autoconnect from the first server. So, I don't understand it again :)10:55
tansellmaybe I'm wrong10:56
bisctansell: lines 785-816 here http://git.thousandparsec.net/gitweb/gitweb.cgi?p=tpclient-pywx.git;a=blob;f=windows/winConnect.py;h=34f362e15834f90e0ca1d5d68560214d8a6bcae2;hb=refs/heads/tp0410:57
tpb<http://ln-s.net/79V2> (at git.thousandparsec.net)10:57
bisctansell: check all servers for autoconnect, then take the first server. So, in case of autoconnect, I don't spoil anything.10:59
tanselllooks like you are correct11:00
bisctansell: in case there's only one server with autoconnect = true. And this is ensured by other code.11:01
bisctansell: * not more than one *11:02
tansell-laptopokay11:12
*** welterde has quit IRC11:16
bisctansell-laptop: and what is your opinion on this patch? You think that my code is misplaced?11:21
tansell-laptopbisc, yes, I think it should be one level out with the if statements11:21
*** welterde has joined #tp11:22
bisctansell-laptop: hmm, why? I don't want to do this to servers not in config, as they're handled separately.11:27
tansell-laptopbecause you want to do something like11:27
tansell-laptopif currentserver != oldserver: move the server to the front of the queue11:27
bisctansell-laptop: I can add this condition ( currentserver!=oldserver), but still I see no reason to remove my code out of "if server in self.config['servers']:"11:30
tansell-laptopbecause that code is logically doing something different11:30
bisctansell-laptop: ah, I see. Ok, will fix that.11:32
biscmithro, tansell-laptop: there're a lot of commits in my repos at github. You possibly want to try or/and pull them.12:11
*** Agon has joined #tp12:19
*** StupidIncarnate has quit IRC13:13
*** bisc has quit IRC13:29
*** peres has joined #tp13:45
*** matthewd has quit IRC13:59
*** peres has quit IRC14:01
*** Epyon has quit IRC14:14
*** ezod has quit IRC14:18
*** Epyon has joined #tp14:19
*** ezod has joined #tp14:30
*** matthewd has joined #tp14:35
*** Epyon has quit IRC14:53
*** glew has joined #tp15:27
*** Agon has quit IRC16:37
*** verhoevenv has joined #tp16:59
*** Epyon has joined #tp17:06
*** JLP has joined #tp17:24
*** JLP has joined #tp17:24
*** JLP_ has quit IRC17:25
*** null_000 has joined #tp17:44
*** Agon has joined #tp18:42
*** tanoku has quit IRC18:50
*** tanoku has joined #tp19:02
*** tanoku has quit IRC19:08
*** tanoku has joined #tp19:10
*** Agon has quit IRC20:25
null_000alanp: ping20:36
*** llnz has joined #tp20:46
llnzmorning all20:48
null_000llnz: evening21:15
llnzhi null_00021:17
llnzafk22:04
*** null_000 has quit IRC22:22
Epyonllnz: ping?22:48
*** verhoevenv has quit IRC22:52
*** verhoevenv has joined #tp23:04
alanpw00p23:16
*** Epyon_ has joined #tp23:28
*** Epyon has quit IRC23:29
*** glew has quit IRC23:29
* llnz is back23:35
llnzhi Epyon_23:35
llnzoh, net split23:35
*** glew has joined #tp23:44
*** verhoevenv has quit IRC23:50

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