Config
Log for #openttdcoop.devzone on 15th April 2010:
Times are UTC Toggle Colours
00:16:36  <PeterT> Ammler: Issue created
00:16:44  <PeterT> http://dev.openttdcoop.org:81/issues/896
00:17:51  <PeterT> Ammler: damn, there is a mistake
00:18:04  <PeterT> can you delete join-part-no-op.diff please?
00:22:58  <Ammler> deleted
00:23:27  <Ammler> please do also note a usage example... :-)
00:23:38  <PeterT> what about it?
00:25:09  <Ammler> well, when do you use it?
00:25:43  <PeterT> I don't know
00:25:50  <PeterT> if you want a seperate testing channel
00:25:54  <PeterT> like #openttdcoop.bots
00:26:00  *** KenjiE20 has quit IRC
00:28:37  <Webster> Latest update from devactivity: Autopilot - Patch #896: Join and Part commands <http://dev.openttdcoop.org:81/issues/896#change-2421" target="_blank">http://dev.openttdcoop.org:81/issues/896#change-2421> || Autopilot - Patch #896 (New): Join and Part commands <http://dev.openttdcoop.org:81/issues/896>
00:31:53  *** SN1 has joined #openttdcoop.devzone
00:31:59  <PeterT> like this^
00:32:07  <PeterT> !part #openttdcoop.devzone
00:32:08  *** SN1 has left #openttdcoop.devzone
06:02:23  *** ODM has joined #openttdcoop.devzone
07:59:53  *** ODM has quit IRC
08:30:13  *** Doorslammer has joined #openttdcoop.devzone
08:53:58  <Webster> Latest update from devactivity: NFO Meta Language - Revision 67: Fix: Action6 was used incorrectly when setting the severity of a... <http://dev.openttdcoop.org:81/projects/nml/repository/revisions/67> || NFO Meta Language - Revision 66: Codechange: Use a param_list when parsing an error_block. <http://dev.openttdcoop.org:81/projects/nml/repository/revisions/66>
09:04:13  <planetmaker> Ammler, so... the old OpenGFX newgrfs. Do they go to legacy or obsolete?
09:04:20  <planetmaker> I'd put them now rather in obsolete?
09:04:46  <Ammler> hmm, shouldn't be needed at all
09:05:19  <Ammler> but why not discuss that in the ticket about?
09:05:48  <planetmaker> well... we don't want to supply them by default anymore with our grfpack, I think
09:05:49  <Ammler> http://dev.openttdcoop.org:81/issues/542
09:42:09  <Webster> Latest update from devactivity: #openttdcoop NewGRF package - Bug #542: Remove OpenGFX newgrfs <http://dev.openttdcoop.org:81/issues/542#change-2423> || #openttdcoop NewGRF package - Bug #542: Remove OpenGFX newgrfs <http://dev.openttdcoop.org:81/issues/542#change-2422>
10:27:42  <Hirundo> Yexo: Is it possible / recommended to insert the same action6 multiple times in the action list?
10:44:44  *** KenjiE20 has joined #openttdcoop.devzone
11:04:41  <planetmaker> Ammler, is the grfpack's repo not linked to redmine on purpose (mb)?
11:06:31  <Ammler> yeah :-(
11:06:45  <Ammler> it isn't public at all
11:07:07  <planetmaker> hm... could it be made visible only to loged in users?
11:08:02  *** ODM has joined #openttdcoop.devzone
11:09:47  <planetmaker> hm... only a general "public" option...
11:11:34  <planetmaker> hm... could we hack the system?
11:11:55  <planetmaker> Not allowing public acess to those files? Or does it work via a DB access, so that it'd work regardless?
11:16:57  <Ammler> well, we could close the project for members only
11:17:06  <Ammler> ah
11:17:12  <Ammler> you like the annoucements here?
11:19:20  <planetmaker> I liked a link to revisions actually
11:19:36  <planetmaker> but it's not really that important
11:19:44  <Ammler> trac is still there
11:20:01  <Ammler> or should
11:20:56  <Ammler> http://trac.openttdcoop.org/browser/grfpack
11:34:26  <Webster> Latest update from devactivity: #openttdcoop NewGRF package - Bug #544 (Confirmed): Remove (e)GRVTS <http://dev.openttdcoop.org:81/issues/544#change-2424>
11:37:41  <Yexo> Hirundo: it's possible, but it won't have the desired effect probably
11:39:06  <Hirundo> why not?
11:41:07  <Hirundo> desired effect = setting the severity in all ActionBs
11:41:48  <Yexo> ah, in that case it should work
11:46:49  <Yexo> Hirundo: currently your code assumes that every ID will be mapped to a constant number
11:47:05  <Yexo> in case of variables however it'll be mapped to something of the Variable class
11:48:14  <Hirundo> I had planned to pass an additional (lambda?) function that maps the value from the dictionary to an AST item, but I did not introduce that (yet)
11:48:25  <Yexo> ok
12:05:50  <Webster> Latest update from devactivity: NFO Meta Language - Revision 69: Feature: Support parameters for actionB error messages. <http://dev.openttdcoop.org:81/projects/nml/repository/revisions/69> || NFO Meta Language - Revision 68: Fix: Fix action6 for error messages, this time done properly (I ... <http://dev.openttdcoop.org:81/projects/nml/repository/revisions/68>
13:29:26  *** ODM has quit IRC
13:46:16  * Hirundo chases the nml equivalent of a memory leak: a free parameter leak
14:03:11  <Yexo> a missing "free_parameters_backup = free_parameters[:]" in one of the functions called from ast.py?
14:10:59  <Hirundo> No, at least not in the functions I'm calling now
14:11:41  <Hirundo> It seems the parameters somehow leak in-between two blocks
14:12:16  <Hirundo> That is, at the end of block one the first free param is 0x40 (as it should), but at the beginning of the next block it is 0x41 or 0x42 :S
14:15:59  <Yexo> can you pastebin an exampel nml tha treprocudes it?
14:16:22  <Hirundo> I'll try to reproduce with 'trunk'
14:19:31  *** Yexo has quit IRC
14:19:47  *** Yexo has joined #openttdcoop.devzone
14:20:57  <Hirundo> http://paste.openttd.org/225572
14:23:02  <Hirundo> Yexo: Question is; why does it use parameter 41 for the 2nd statement?
14:23:20  <Yexo> no idea, trying to find that out now
14:32:11  <Yexo> the "free_parameters = free_parameters_backup" at the end of parse_error_block doesn't work for some reason
14:34:32  <Hirundo> to me, it reports that it does work
14:36:56  <Hirundo> http://paste.openttd.org/225574 <- result of debug statements like this: print "error entry", to_hex(free_parameters[-1])
14:37:58  <Yexo> Hirundo: yes, now try printing the length also in main.py:36
14:38:03  <Yexo> so just after each get_action_list
14:38:22  <Yexo> you'll notice that also error exit prints 40, in the main loop it'll be 41 again
14:39:13  <Hirundo> yes
14:41:09  <Webster> Latest update from devactivity: MiniGRFs - Revision 10: [basecosts] Release Presets 1.4 <http://dev.openttdcoop.org:81/projects/minigrfs/repository/revisions/10>
14:42:04  <Yexo> replace "free_parameters = free_parameters_backup" with "free_parameters.extend([item for item in free_parameters_backup if not item in free_parameters])" and it does work
14:45:05  <Hirundo> Yes, but why ? what piece of python's black box magic am I missing?
14:45:27  <Yexo> it might have something to do with setting a global variable defined in another module
14:45:45  <Yexo> the new line doesn't set the global variable, it changes it
14:47:17  * Hirundo sees the light
14:47:42  <Yexo> ok if I commit that?
14:47:50  <Hirundo> yes
14:48:02  <Yexo> there might be a better solution, this is my first python project
14:48:39  <Hirundo> The assignment is valid only in the current module, I think
14:49:09  <Hirundo> So the second action D statement 'sees' the value left by the  actionD that was part of the error block, i.e. 41
14:50:47  <Yexo> yes, something like that
14:50:51  <Yexo> not 100% how it works
14:53:16  <Hirundo> http://effbot.org/zone/import-confusion.htm <- regarding from-import
14:53:17  <Webster> Title: Importing Python Modules (at effbot.org)
14:56:39  <Webster> Latest update from devactivity: MiniGRFs - basecosts-presets.grf <http://dev.openttdcoop.org:81/attachments/download/657/basecosts-presets.grf>
14:57:33  <Ammler> planetmaker: I suggest making a grfpack 8.0 beta and make some tests with the archive
14:57:46  <Ammler> so we see if we removed too many
14:58:15  <Ammler> then we release 8.0 some weeks later
14:58:37  <Ammler> which should then hold for a year again :-)
14:58:53  <Ammler> or longer
14:59:33  <Ammler> meh, don't like beta, we simply release 8.0 and if it doesn't work, fix and release 8.1
15:00:58  <Yexo> Hirundo: ah, "And if name in X is changed to point to some other object, your module won’t notice" <- that was the problem
15:02:22  <planetmaker> Ammler, but we still should test it before we call it a release
15:02:30  <Hirundo> OK, improved ActionE coming in
15:04:39  <Ammler> planetmaker: we most probably can't
15:04:51  <Ammler> or will you try it on the 150 games?
15:05:01  <planetmaker> Ammler, how can't we? No. But on 'selected'
15:05:07  <Ammler> of course
15:05:27  <Ammler> we could for example on those which failed last time :-)
15:06:22  <Ammler> http://trac.openttdcoop.org/ticket/10
15:07:23  <planetmaker> yep
15:07:28  <planetmaker> possibly some newer, too
15:09:00  <Ammler> we should ask Zephyris to upload the older versions to bananas
15:12:06  <Webster> Latest update from devactivity: NFO Meta Language - Revision 70: Fix: resetting the list with available parameters/labels didn't ... <http://dev.openttdcoop.org:81/projects/nml/repository/revisions/70>
15:18:41  <Yexo> Hirundo: are you working (or planning to work on) adding a lambda function to the id_list passed to reduce_expr? if not I'll give it a go
15:19:43  <Hirundo> You can give it a go, it should be a two line change I think
15:21:00  <Hirundo> hmm I have already done it, could not resist :)
15:21:45  <Yexo> ok :)
15:22:11  <Yexo> push it then, so I can work on varaction2 support
15:23:08  <Hirundo> Correction, it's 4 lines not 2
15:24:16  <Hirundo> Yexo: done
15:27:20  <Webster> Latest update from devactivity: Nutracks - Bug #865: Track costs are inconsistent and exploitable <http://dev.openttdcoop.org:81/issues/865#change-2425> || NFO Meta Language - Revision 71: Feature: Make 'deactivate' accept more than one parameter, and a... <http://dev.openttdcoop.org:81/projects/nml/repository/revisions/71>
15:30:41  <Yexo> Hirundo: wouldn't a function per id_dict be better?
15:31:01  <Yexo> so make id_dicts an array of (id_dict, func) tuples
15:31:56  <Hirundo> Is it still possible to specify a default that way?
15:33:01  <Yexo> func = lambda x: ConstantNumerci(x) if not isinstance(id_dict, tuple) else id_dict[1]
15:34:54  <Hirundo> ok, go ahead
15:38:37  <Ammler> funny thing, andythenorth does also upload the release to tt-forums, so obviously his release tasks are bit more work... ;-)
15:38:59  <planetmaker> hehe :-)
15:41:05  <Ammler> my new backup script failed, somehow it synced the whole system :-/
15:42:23  <Webster> Latest update from devactivity: NFO Meta Language - Revision 72: Feature: lambda functions in reduce_expr. <http://dev.openttdcoop.org:81/projects/nml/repository/revisions/72>
15:57:28  <Webster> Latest update from devactivity: NFO Meta Language - Revision 73: Change: reduce_expr now accepts a list of (dict, func) couples s... <http://dev.openttdcoop.org:81/projects/nml/repository/revisions/73>
16:05:47  *** ODM has joined #openttdcoop.devzone
16:11:14  *** frosch123 has joined #openttdcoop.devzone
16:59:38  <Webster> Latest update from devactivity: NFO Meta Language - Revision 74: Feature: support references variables by name and make a start w... <http://dev.openttdcoop.org:81/projects/nml/repository/revisions/74>
16:59:43  <Yexo> Hirundo: I'd added constants for feature numbers now (feature numbers are now constant expressions), but I don't have a testcase for PropertyBlock yet
16:59:50  <Yexo> I hope I've fixed all cases in there
17:14:43  <Webster> Latest update from devactivity: NFO Meta Language - Revision 75: Feature: feature numbers are now parsed as constant expressions,... <http://dev.openttdcoop.org:81/projects/nml/repository/revisions/75>
17:41:32  *** frosch123 has quit IRC
17:46:37  *** Doorslammer has quit IRC
17:49:18  *** Doorslammer has joined #openttdcoop.devzone
18:07:37  *** Seberoth has joined #openttdcoop.devzone
18:08:15  *** Seberoth has quit IRC
18:10:44  *** Seberoth has joined #openttdcoop.devzone
19:20:19  <Webster> Latest update from devactivity: NFO Meta Language - Revision 76: Feature: add list of global varaction2 variables <http://dev.openttdcoop.org:81/projects/nml/repository/revisions/76>
19:22:12  *** Seberoth has quit IRC
19:25:55  *** Doorslammer has quit IRC
19:43:19  *** Seberoth has joined #openttdcoop.devzone
19:52:23  <Webster> Latest update from devactivity: NFO Meta Language - Revision 77: Fix: Varact2 variable 0x18 is callback info 2, not 1. <http://dev.openttdcoop.org:81/projects/nml/repository/revisions/77>
20:23:12  <Webster> Latest update from devactivity: NFO Meta Language - Revision 78: Feature: optimize expressions with a Variable and a constant mor... <http://dev.openttdcoop.org:81/projects/nml/repository/revisions/78>
20:45:23  *** frosch123 has joined #openttdcoop.devzone
21:09:23  <Webster> Latest update from devactivity: World Airliners Set - Feature #895: Standard helicopters support <http://dev.openttdcoop.org:81/issues/895#change-2426>
21:26:25  <Ammler> planetmaker: you really should make that tar :-)
21:26:57  <Ammler> so people can use official zip for such things
21:27:52  <planetmaker> uh?
21:28:21  <Ammler> well, yexo or whoever suggested him to use bananas tar...
21:28:37  <Ammler> I would recommend the official zip
21:28:49  <planetmaker> yes. Ah
21:28:56  <Yexo> Ammler: I suggested the tar because that can be included in the .ipa bundle directly, as openttd can read inside a tar
21:28:57  <planetmaker> Well, he needs a format which can be read
21:29:00  <Yexo> with a zip that wouldn't work
21:29:09  <planetmaker> ^
21:29:19  <Ammler> Yexo: but maybe you can unzip a zip before?
21:29:36  <Yexo> yes, but then he'd have to extract the zip, and the risk he then removes files is higher
21:29:38  <Ammler> we do that on suse
21:29:47  <Yexo> if he includes the tar ocmpletely you can be sure all needed files are there
21:30:11  <Ammler> yes, that is why I think, we should have again a tar in the zip
21:30:12  <Yexo> same as he included a lot of AI tar, he didn't extract those, so the license files / readme files are included for them
21:30:24  <Yexo> why a tar in the zip?
21:30:28  <Yexo> why not a single tar only?
21:30:43  <Ammler> becasue windows can't unpack that
21:30:56  <Ammler> and so the readme is badly accessable
21:32:22  <Ammler> I do also wonder, you guys seems more focused to the license file than the readme
21:33:13  <Yexo> I'm not doing that intentionally, if I did so it was by accident
21:33:47  <Yexo> if you insisnt on having the tar + readme + licence in a zip then I'd inlcude the readme/license file twice, once in the tar and also in the zip
21:34:18  <planetmaker> Yexo, it's not done yet. But it has one advantage:
21:34:29  <planetmaker> people might download the zip file - and unzip it.
21:34:45  <planetmaker> then they have the text files to read - and if they re-distribute just the tar, they're still fine
21:34:55  *** frosch123 has quit IRC
21:35:06  <planetmaker> tar is a mostly unknown format by many non *nix people
21:35:10  <Yexo> planetmaker: yes, that's what I was saying ;)
21:35:17  <planetmaker> oh, ok :-P
21:35:19  <Rubidium> once upon a time... there was a zip with a readme, license and a tar (with a readme, license, grfs and obg)
21:35:29  <Ammler> mostly like we had before the Makefile upgrade
21:35:34  <planetmaker> and then I changed it, I guess :-)
21:35:37  <planetmaker> bad me :-)
21:35:40  <Ammler> :-)
21:35:46  <planetmaker> change request dully noted ;-)
21:36:19  <Ammler> well, new advantage is a common used Makefile framework
21:36:27  <Ammler> you can't have everything at once ;-)
21:36:33  <planetmaker> ah, well
21:36:50  <Rubidium> he should've just unzipped into a separate directory and there was not problem either
21:36:50  <planetmaker> though I'm quite sure I didn't touch the bundle* rules.
21:38:34  <Ammler> Rubidium: yes, same problem was with the suse guys...
21:38:51  <Ammler> but that is fixed now :-)
21:39:55  <Ammler> the only bad thing from my side was --enable-strip
22:12:22  *** ODM has quit IRC
22:42:36  <Webster> Latest update from devactivity: NFO Meta Language - Revision 80: Feature: support *,/,% operators for both varaction2 and actionD <http://dev.openttdcoop.org:81/projects/nml/repository/revisions/80> || NFO Meta Language - Revision 79: Feature: optimize some more combinations of a variable and a con... <http://dev.openttdcoop.org:81/projects/nml/repository/revisions/79>
22:43:48  *** Seberoth has quit IRC
22:47:10  *** Seberoth has joined #openttdcoop.devzone
23:38:54  *** Seberoth has quit IRC
23:48:14  *** KenjiE20 has quit IRC

Powered by YARRSTE version: svn-trunk