Times are UTC Toggle Colours
00:31:22 *** KenjiE20 has quit IRC 06:44:32 *** ODM has joined #openttdcoop.devzone 07:33:52 <Brot6> #openttdcoop NewGRF package - Bug #1009 (New): removed too many obsolete newgrfs (planetmaker) @ http://dev.openttdcoop.org/issues/1009 07:33:52 <Brot6> #openttdcoop NewGRF package - Bug #542: Remove OpenGFX newgrfs (planetmaker) @ http://dev.openttdcoop.org/issues/542#change-2612 07:35:38 <Brot6> #openttdcoop NewGRF package - Bug #550 (Closed): Remove ECS from George (planetmaker) @ http://dev.openttdcoop.org/issues/550#change-2613 09:53:19 *** KenjiE20 has joined #openttdcoop.devzone 13:32:04 <Brot6> NFO Meta Language - Feature #969: Parse #file and #line lines that are written by a preprocessor (yexo) @ http://dev.openttdcoop.org/issues/969#change-2614 13:36:30 <Brot6> NFO Meta Language - Feature #969: Parse #file and #line lines that are written by a preprocessor (yexo) @ http://dev.openttdcoop.org/issues/969#change-2616 14:52:14 <Brot6> NFO Meta Language - Revision 299:4665232e95d9: Feature: support for conditional expressions in ac... (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/4665232e95d9 15:03:21 *** Seberoth has joined #openttdcoop.devzone 16:12:06 *** frosch123 has joined #openttdcoop.devzone 16:15:48 <Brot6> NFO Meta Language - Revision 300:af1ae084ac48: Change: allow expressions (instead of only IDs) as... (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/af1ae084ac48 16:18:58 <Brot6> nml: update from r292 to r300 done - http://bundles.openttdcoop.org/nml/nightlies/r300 16:19:04 <Brot6> Following repos didn't need a nightlies update: 2cctrainset (r545), 32bpp-extra (r36), airportsplus (r50), bros (r12), comic-houses (r70), firs (r910), fish (r375), heqs (r320), newgrf_makefile (ERROR r99), nmts (r16), nutracks (r69), ogfxplus (r27), opengfx (r459), openmsx (r57), opensfx (r94), snowlinemod (r12), swedishrails (r25), test (ERROR r47), worldairlinersset (r643) 17:26:11 *** Seberoth has quit IRC 17:27:31 *** Seberoth has joined #openttdcoop.devzone 17:32:38 *** Alberth has joined #openttdcoop.devzone 17:46:15 <Brot6> NFO Meta Language - Revision 301:3778d42fa25b: Fix: print action3 mappings in the same order as t... (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/3778d42fa25b 17:59:34 <Brot6> NFO Meta Language - Feature #969: Parse #file and #line lines that are written by a preprocessor (Alberth) @ http://dev.openttdcoop.org/issues/969#change-2617 18:06:48 <Brot6> NFO Meta Language - Feature #995: Handle errors more gracefully (Alberth) @ http://dev.openttdcoop.org/issues/995#change-2618 18:10:31 <Brot6> NFO Meta Language - Feature #995: Handle errors more gracefully (yexo) @ http://dev.openttdcoop.org/issues/995#change-2619 18:13:23 <Alberth> hai 18:14:18 <Alberth> can you explain the warning thingie in #1008? where/when would such a warning be given? 18:15:21 <Yexo> nml checks the version where all used features are supported and determines you need at least openttd r19000 for a feature that is used 18:15:38 * Alberth nods 18:15:40 <Yexo> when the nml doesn't have a "minimum_openttd_version(1, 0, 0, 19000)" or higher statement then a warning is given 18:16:14 <Alberth> ah, to the nml file author. that makes sense 18:16:47 <Yexo> yes, as planetmaker said "But in order to elaborate on it: I don't think NML should go and decide for the programmer which minimum version requirement a newgrf should have." 18:18:00 <Alberth> I read that differently, as in 'nml should not bother about minimum version', and then I got lost. 18:18:09 <Alberth> thanks for the clarification 18:20:35 <Alberth> About #995 and #969, I think they are seperate issues, except that they both start with position reporting by ply. The only thing we should synchronize imho is that a solution with positions for one bug is also usable by the other bug. 18:21:21 <Alberth> ie #995 can perfectly be done with a position-class containing only a line number or so. 18:22:32 <Yexo> yes, but if the Position-class is done then #969 is done with the line_directives.diff patch I added there 18:23:26 <Hirundo> http://dev.openttdcoop.org/projects/nml/repository/revisions/4665232e95d9/diff/nml/actions/actionD.py <- Is this really easier / better than using action9? 18:25:17 <Yexo> Hirundo: action7/9 can't compare parameter with eachother 18:25:33 <Hirundo> action6 should fix that 18:25:36 <Yexo> so it'd be action6 to write param2 value, then action7/9 to compare param1 with written value 18:25:45 <Yexo> then an actionD to set the target to 1 18:25:54 <Yexo> before that all you need an actionD to set the target to 0 18:26:47 <Yexo> you're right, here action7/9 might be better but we need code very much like this anyway to support the same operators in action2 code 18:27:44 <Hirundo> Action2 has op 12 18:28:48 <Yexo> hmm, I forgot about that one 18:29:43 <Yexo> ok, I do agree that using action7/9 would be better 18:30:02 <Hirundo> though this works, and working code is better than no code whatsoever 18:30:51 <Yexo> using action7/9 will introduce yet another cyclic dependency 18:32:44 <Hirundo> Could fix that, by splitting the business logic of actionD/7/9 (get_tmp_parameteter) to a new file that imports Action7/9/D 18:33:35 <Yexo> yep, indeed 18:39:35 <Yexo> Hirundo: how do you plan to solve #974 18:40:57 <Hirundo> I don't have a complete plan yet 18:40:59 <Hirundo> first step, which is mostly finished, is to count the bytes inside the Output classes 19:17:10 <Alberth> regression test fails 19:17:48 <Alberth> nml/actions/action3.py", line 63, in parse_graphics_block 19:18:01 <Alberth> TypeError: 'NoneType' object is not iterable 19:25:55 <Yexo> fixed 19:26:53 <Brot6> NFO Meta Language - Revision 302:3e01cae1bc33: Fix (r301): no graphics blocks should be an empty ... (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/3e01cae1bc33 19:48:29 <Brot6> NFO Meta Language - Revision 303:a1e7d5a70f2c: Fix (r294): One reduce_expr() was left. (Alberth) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/a1e7d5a70f2c 19:57:28 <Brot6> NFO Meta Language - Revision 304:112ababa53e9: Codechange: Moving some classes from ast to action1. (Alberth) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/112ababa53e9 19:57:28 <Brot6> NFO Meta Language - Revision 305:d3b6a7e81314: Codechange: Moving ParameterAssignment from ast to... (Alberth) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/d3b6a7e81314 19:57:28 <Brot6> NFO Meta Language - Revision 306:4e0fa08f10cf: Codechange: Moving SwitchRange class from ast to a... (Alberth) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/4e0fa08f10cf 19:57:32 <Brot6> NFO Meta Language - Revision 307:42e3f83f34d3: Codechange: Move a few classes from ast to real_sp... (Alberth) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/42e3f83f34d3 19:57:53 <PeterT> why does Brot6 say 3 revisions in the same second? 19:58:20 <Alberth> last 4 all got pushed together 19:58:50 <Alberth> so the real question is, why is the last one 4 seconds late? 19:59:14 <PeterT> didn't know you could push 4 different revisions together ;-) 19:59:25 <PeterT> err...wrong emoticon 19:59:27 <PeterT> O_o 19:59:56 <Alberth> with mercurial and git, you can, since you first commit locally, then push all changes that are not at the server 20:00:35 <PeterT> ah, git ,3 20:00:37 <PeterT> *<3 20:00:48 <Alberth> mercurial, actually :p 20:01:11 <Alberth> git has too many settings 20:01:15 <PeterT> yeah 20:01:21 <PeterT> but git's branches, wow. 20:01:32 <Alberth> mercurial also has branches 20:02:04 <Alberth> I think they are more or less equal 20:10:33 <PeterT> either way, they are both slow with cloning 20:10:37 <Brot6> NFO Meta Language - Revision 308:be8dbdbd084d: Codechange: Explicitly import expression module in... (Alberth) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/be8dbdbd084d 20:11:59 <Alberth> jay, we got a 6.42/10 from pylint for action2layout.py :D 20:26:03 <Brot6> FIRS Industry Replacement Set - Feature #1010 (New): Clustering for mines (andythenorth) @ http://dev.openttdcoop.org/issues/1010 20:26:18 <planetmaker> good evening :-) 20:26:29 <andythenorth> hi hi 20:27:07 <planetmaker> uh... busy NML evening :-) 20:27:07 <Yexo> Alberth: yes, and -7.43/10 for main.py (yes, a negative score) 20:27:13 <Yexo> evening planetmaker 20:27:32 <Brot6> FIRS Industry Replacement Set - Feature #1011 (New): Clustering for forests (andythenorth) @ http://dev.openttdcoop.org/issues/1011 20:27:44 * planetmaker calles 'update_repos' :-) 20:27:59 <Alberth> planetmaker: our leader has made lots of decisions :) 20:28:11 <planetmaker> good leader we have then :-) 20:29:05 * Alberth agrees 20:29:14 <Brot6> FIRS Industry Replacement Set - Feature #1012 (New): Oil wells should be oil field - high product... (andythenorth) @ http://dev.openttdcoop.org/issues/1012 20:29:46 <Yexo> planetmaker: did you already write some constants for those railtype numbers? if not I'll do that now 20:30:04 <planetmaker> I did, but I had to leave before I could commit 20:30:11 <planetmaker> let me get you the paste for review 20:31:41 <planetmaker> http://paste.openttd.org/225945 <-- that worked for me, but I haven't made sure enough it has no side effects 20:32:15 <Alberth> Yexo: I tried making main.py better last weekend, but failed due to the imports 20:32:40 <Brot6> NFO Meta Language - Revision 309:167b9e182367: Codechange: Explicit module import for several act... (Alberth) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/167b9e182367 20:32:58 <Yexo> planetmaker: that works, but that breaks using cargo labels in vehicle action3's 20:33:08 <planetmaker> I feared something like that 20:33:19 <Yexo> but with that list of IDs it's easy to implement 20:33:42 <planetmaker> giving it another name doesn't do the job. How is it proper? 20:34:48 <Yexo> writing a function that returns the proper list (either cargo_number for vehicle/station features or the list in that pastebin for railtypes) and use that instead of always [cargo_numbers] 20:36:27 * andythenorth reads some nml source 20:37:40 <planetmaker> hm... I guess you know already how to do that, Yexo :-) 20:37:59 <Alberth> good night all 20:38:15 <planetmaker> g'night Alberth 20:38:52 *** Alberth has left #openttdcoop.devzone 20:39:54 <Brot6> FIRS Industry Replacement Set - Feature #977 (New): Rework Aluminium Plant (andythenorth) @ http://dev.openttdcoop.org/issues/977#change-2620 20:40:15 <Yexo> planetmaker: thanks for the list with constants :) 20:40:36 <planetmaker> that's the no-brainer :-P 20:41:08 <Brot6> NFO Meta Language - Revision 310:7a4893da6ce5: Feature: constants for use in railtype action3 (ba... (yexo) @ http://dev.openttdcoop.org/projects/nml/repository/revisions/7a4893da6ce5 21:02:21 *** frosch123 has quit IRC 22:24:29 <Brot6> Swedish Rails - Revision 26:8a270c3fa811: Feature: Normal and electrical rails use railtypes (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/8a270c3fa811 22:31:32 *** ODM has quit IRC 22:42:46 <Brot6> Swedish Rails - Revision 27:7454303aab67: Fix: Don't forget to actually add the source files (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/7454303aab67 23:00:44 <Brot6> Swedish Rails - Revision 28:380e0e6eec51: Fix: Add simple level crossings (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/380e0e6eec51 23:24:24 <Brot6> Swedish Rails - Revision 29:a61806623969: Doc: Update repo URL and some minor improvements (planetmaker) @ http://dev.openttdcoop.org/projects/swedishrails/repository/revisions/a61806623969 23:27:36 *** Seberoth has quit IRC 23:28:55 <Brot6> Swedish Rails - Feature #989 (Closed): railtypes support for OpenTTD (planetmaker) @ http://dev.openttdcoop.org/issues/989#change-2621 23:28:55 <Brot6> Swedish Rails - Feature #988 (Closed): support for tropical climate (planetmaker) @ http://dev.openttdcoop.org/issues/988#change-2622 23:30:32 *** KenjiE20 has quit IRC 23:30:52 <Brot6> Swedish Rails - Feature #987 (Closed): Improved snowy TTD track sprites (planetmaker) @ http://dev.openttdcoop.org/issues/987#change-2623 23:33:15 <Brot6> Swedish Rails - Feature #1013 (New): rail type level crossing sprites (planetmaker) @ http://dev.openttdcoop.org/issues/1013 23:35:42 <Brot6> Swedish Rails - Feature #1014 (New): ttdp - style level crossings (planetmaker) @ http://dev.openttdcoop.org/issues/1014 23:35:42 <Brot6> Swedish Rails - Bug #1015 (New): tunnel sprites with OpenGFX-style ground sprites (planetmaker) @ http://dev.openttdcoop.org/issues/1015 23:37:23 <Brot6> Swedish Rails - Feature #1016 (New): depot sprites (planetmaker) @ http://dev.openttdcoop.org/issues/1016