Log for #openttdcoop.devzone on 13th December 2011:
Times are UTC Toggle Colours
00:22:20  *** Zuu has quit IRC
01:46:18  <Brot6> Central European Train Set - Feature #3043: freight wagons (Eddi) @
02:20:30  *** welshdragon has quit IRC
02:45:35  <Brot6> Central European Train Set - Feature #3043: freight wagons (oberhuemer) @
03:17:58  <Brot6> Central European Train Set - Feature #3043: freight wagons (oberhuemer) @
06:54:04  *** JVassie has joined #openttdcoop.devzone
07:03:48  <Brot6> NewGRF Meta Language - Revision 1753:7fce7493b930: Add: Support for aircraft range property (planetmaker) @
07:07:23  <Brot6> NewGRF Meta Language - Revision 1754:594072baadb0: Add: Also support the range property as callba... (planetmaker) @
07:24:11  *** JVassie has quit IRC
08:11:59  *** ODM has joined #openttdcoop.devzone
08:12:23  *** Zuu has joined #openttdcoop.devzone
08:35:35  *** Zuu has quit IRC
11:05:56  *** welshdragon has joined #openttdcoop.devzone
12:38:35  *** welshdragon is now known as SnowDragon
12:39:01  <Brot6> Dutch Trains 2.0 - Feature #3336: Old pax coaches (Voyager1) @
12:51:31  *** SnowDragon has quit IRC
12:54:49  *** SnowDragon has joined #openttdcoop.devzone
15:08:36  <Brot6> GRFCodec - Bug #3361 (New): Invalid PNG file produced by grfcodec (petern) @
15:53:36  <Brot6> Central European Train Set - Revision 380:49cafe6d50bd: unify cargo classes over all vehicles, so... (Eddi) @
15:53:36  <Brot6> Central European Train Set - Revision 381:9b9a26dda1cf: change articulated callback to return 0x7... (Eddi) @
15:55:58  <Brot6> Central European Train Set - Feature #3043: freight wagons (Eddi) @
16:01:11  <Brot6> cets: update from r379 to r381 done (220 warnings) -
16:24:44  <Brot6> GRFCodec - Bug #3361 (Closed): Invalid PNG file produced by grfcodec (petern) @
16:24:44  <Brot6> GRFCodec - Revision 847:a9be5e47645b: -Fix: creation of 'corrupt' PNGs [Fixes #3361] (Rubidium) @
16:24:44  <Brot6> GRFCodec - Bug #3361 (Closed): Invalid PNG file produced by grfcodec (Rubidium) @
16:46:17  <Brot6> Central European Train Set - Revision 382:9c26dbde1d9c: enable hgeol extension, to make things mo... (Eddi) @
16:46:17  <Brot6> Dutch Trains 2.0 - Feature #3336: Old pax coaches (foobar) @
16:52:55  <Brot6> cets: update from r381 to r382 done (220 warnings) -
17:09:31  *** frosch123 has joined #openttdcoop.devzone
17:09:52  <Brot6> grfcodec: update from r846 to r847 done -
17:14:11  <Brot6> nml: update from r1752 to r1754 done -
17:25:15  <Brot6> cets: update from r379 to r382 done (220 warnings) -
17:28:18  <Brot6> narvs: compile of r57 still failed (#3353) -
17:29:44  <Brot6> ogfx-industries: compile of r124 still failed (#3354) -
17:37:05  <Brot6> bandit: compile of r26 still failed (#3303) -
17:37:55  *** JVassie has joined #openttdcoop.devzone
17:51:12  <Brot6> dutchtramset: compile of r87 still failed (#3355) -
17:55:56  <Brot6> ogfx-rv: compile of r140 still failed (#3356) -
17:57:50  <Brot6> ogfx-landscape: compile of r110 still failed (#3357) -
18:05:46  <Brot6> airportsplus: compile of r147 still failed (#3358) -
18:06:28  <Brot6> Following repos rebuilds successful without any difference to earlier nightlies builds: ogfx-trains, bros (1 warnings), firs (24 warnings), foobarstramtracks (Diffsize: 37132), transrapidtrackset, 2cctrainset (6 warnings), worldairlinersset, heqs, basecosts, nutracks, water-features, isr (2 warnings), 32bpp-extra (2 warnings), manindu (Diffsize: 2), newgrf_makefile (Diffsize: 12), dutchtrains, rust (Diffsize: 188), ogfx-biggui
18:06:28  <Brot6> (Diffsize: 30), snowlinemod, swisstowns (Diffsize: 43), metrotrackset (Diffsize: 1), dutchroadfurniture (Diffsize: 8632), spanishtowns (Diffsize: 8), frenchtowns (Diffsize: 21), fish, ttrs (7 warnings), ogfx-trees, swedishrails (Diffsize: 737), german-townnames (Diffsize: 51), smts (Diffsize: 8), chips (1 warnings), dach (Diffsize: 10855), belarusiantowns (Diffsize: 64), indonesiantowns (1 warnings) (Diffsize: 29), comic-houses (3
18:06:30  <Brot6> warnings) (Diffsize: 22)
18:49:41  *** SnowDragon has quit IRC
18:54:21  <Yexo> planetmaker / Terkhen: mind if I add myself as developer for opengfx+industries?
18:55:43  *** SnowDragon has joined #openttdcoop.devzone
19:02:55  <Brot6> clientpatches: compile of r23508 still failed (#2964) -
19:04:55  <Brot6> serverpatches: compile of r23508 still failed (#2966) -
19:06:11  <Yexo> regarding #2737: can I just copy the sprites from the FIRS foodmarket?
19:06:12  <Brot6> Yexo: #2737 is "OpenGFX+ Industries - Bug #2737: Snowy sprite for shop - #openttdcoop Development Zone"
19:07:13  <Brot6> 32bpp-ez-patches: compile of r23508 still failed (#2446) -
19:16:39  *** andythenorth has joined #openttdcoop.devzone
19:18:51  <Brot6> Dutch Trains 2.0 - Feature #3359 (New): Templated steamers, per request (Supercheese) @
19:18:51  <Brot6> Dutch Trains 2.0 - Feature #3359: Templated steamers, per request (V453000) @
19:18:51  <Brot6> Dutch Trains 2.0 - Feature #3359: Templated steamers, per request (foobar) @
19:32:15  *** Zuu has joined #openttdcoop.devzone
19:51:50  <Hirundo> planetmaker: You document aircraft range as having a value range 0..2048, shouldn't that be 0..4096 (map size x + y)?
19:53:59  <Yexo> yes
19:54:05  <Yexo> also: time for OpenGFX+Aircraft :p
19:54:14  <michi_cc> Hirundo: Aircarft don't use manahatten distance, so the max usefull distance is 2048*sqrt(2)
19:54:40  <Terkhen> I was thinking that too :)
19:55:29  <Yexo> Terkhen: any comments on my questions above regarding opengfx+industries?
19:55:57  <Terkhen> sorry, I missed that highlight
19:56:04  <Terkhen> go ahead, welcome :)
19:56:30  <Yexo> and the other one regarding the foodmarket?
19:56:47  <Yexo> also: FIRS shows two small trees in front of the building. Do we want those too?
19:57:41  * Terkhen checks the logs
19:57:50  <Yexo> <Yexo> regarding #2737: can I just copy the sprites from the FIRS foodmarket?
19:57:50  <Brot6> Yexo: #2737 is "OpenGFX+ Industries - Bug #2737: Snowy sprite for shop - #openttdcoop Development Zone"
19:58:36  <Terkhen> yes, that was the idea
19:58:49  <Terkhen> the non-snowy ones already use FIRS sprites, and they are credited at the readme
19:59:23  <Terkhen> andythenorth: ^ can we borrow another shop sprite from FIRS? :)
20:06:13  <andythenorth> why not ;)
20:06:58  <Rubidium> because they haven't returned the previously borrowed sprites?
20:07:29  <Terkhen> thank you :)
20:07:36  * Terkhen does not plan to return this one either
20:09:40  <andythenorth> I've given out more than I get back :)
20:09:49  <andythenorth> but it seems to return sometimes in ottd commits
20:10:52  <Terkhen> :P
20:12:13  * andythenorth waves and points at rv-wagons
20:12:24  <andythenorth> although that suggests sprites trade at a high premium :P
20:12:39  <andythenorth> what is the exchange rate on sprites : trunk commits?
20:13:05  <Rubidium> let me check ;)
20:13:13  <planetmaker> hm, might be that we soonish get the VAST NewGRFs here :-)
20:13:26  *** SnowDragon has quit IRC
20:13:30  <Terkhen> andythenorth: depends on the quality of the commits
20:13:55  <planetmaker> Hirundo: yes, should be 4096
20:14:03  <Rubidium> andythenorth: 784 sprites gives 1 commit
20:14:05  <planetmaker> or... sqrt*2048
20:14:27  <andythenorth> Rubidium: is it cheaper if I buy futures?
20:14:34  <andythenorth> can I short sell?
20:14:42  <planetmaker> yes, you can
20:14:48  <planetmaker> but you need to rename to SAC then
20:14:59  <Hirundo> that'd be 2896 *adds*
20:15:57  <Rubidium> rather sqrt*2046
20:16:26  <Hirundo> and 0 = infinite?
20:17:06  <Yexo> planetmaker: could you take a look at the 2nd childsprite in GROUND_AWARE_SPRITELAYOUT ?
20:17:11  <andythenorth> hmm
20:17:18  <Yexo> I think the hide_sprite definition misses some parentheses
20:17:19  <andythenorth> maybe I can corner the market, increasing prices artificially
20:17:34  <planetmaker> can you tell me the filename, please?
20:18:59  <Yexo> templates\spritelayouts_groundaware.pnml
20:19:28  <planetmaker> I keep missing that file. Thanks
20:20:23  <Yexo> first two lines now read; ((climate != tropic) || (climate ==tropic)) && (tile_type==desert) which is true && (tile_type==desert)
20:21:01  <planetmaker> looks very weired
20:21:22  <Hirundo> hmm... some parts of the code seem to include a range != 0 check and others don't
20:21:24  <planetmaker> hm... though
20:21:26  * Hirundo investigates
20:21:37  <Yexo> oh, && has higher priority than ||
20:21:52  <planetmaker> yes
20:22:02  <Yexo> but that still doesn't make it right
20:22:22  <planetmaker> it should ensure proper desert transition...
20:22:46  <Yexo> as it means is read: (line 1) || (line 2) || (rest of lines)
20:23:00  <Yexo> line1: hide if current climate is not tropic
20:23:02  <planetmaker> yes. That's what's supposed to mean
20:23:13  <planetmaker> or if desert in tropical climate
20:23:14  <Yexo> line 2: hide if we're in tropic and it's a desert tile
20:23:26  <planetmaker> line3: hide if no adjacent desert tile
20:23:32  <planetmaker> line3+ actually
20:23:38  <Yexo> which sprite is that?
20:23:43  <planetmaker> desert transition
20:23:47  <Yexo> ah, ok
20:23:51  <Yexo> it finally makes sense
20:23:58  <Yexo> I'd still like some extra parentheses though
20:24:16  <planetmaker> probably it needs most of all a comment :-)
20:24:43  <Yexo> will add that now
20:24:48  <planetmaker> thanks :-)
20:29:26  <Brot6> FIRS Industry Replacement Set - Revision 2651:f97bc7fb2ada: Doc: desert<>grass transition tile in... (yexo) @
20:29:54  <andythenorth> :)
20:30:36  <Brot6> OpenGFX+ Industries - Revision 125:208ec6ad54a5: Fix #3354: use new-style callbacks so it compile... (yexo) @
20:30:36  <Brot6> OpenGFX+ Industries - Revision 126:deb140baedeb: Fix: line endings of docs/license.txt (yexo) @
20:30:36  <Brot6> OpenGFX+ Industries - Bug #3354 (Closed): DevZone compile failed (yexo) @
20:32:38  <Brot6> ogfx-industries: update from r124 to r126 done -
20:37:08  <planetmaker> <-- epic answer, Yexo :-)
20:37:09  <Webster> Title: Transport Tycoon Forums View topic - NewGRFs in scenarios (at
20:37:29  <Hirundo> It seems it is intended that range=0 means infinite, although implementation is buggy (see FS#mostrecent)
20:37:40  * Hirundo documents
20:37:47  <planetmaker> the bug?
20:39:27  <Hirundo> cloning/sharing fails, the code path doesn't include the 'range !=0' check
20:39:54  <Hirundo>
20:42:07  *** SnowDragon has joined #openttdcoop.devzone
20:44:25  <Terkhen> heh :D
20:44:37  <planetmaker> hm
20:48:07  <Hirundo> planetmaker: Changing line 1404 of order_cmd.cpp to "if (first == NULL || v->acache.cached_max_range == 0) return true; should do the trick, methinks
20:48:17  <Hirundo> Don't have the time or compile environment to test, though
20:48:41  * planetmaker tests
20:50:18  *** SnowDragon_ has joined #openttdcoop.devzone
20:50:28  *** SnowDragon_ has quit IRC
20:51:05  <planetmaker> you should get a dev environment again ;-)
20:51:20  *** SnowDragon_ has joined #openttdcoop.devzone
20:51:35  <planetmaker> though that is probably the minor difficulty compared to 'little time' ;-)
20:55:26  *** SnowDragon has quit IRC
20:55:26  *** SnowDragon_ is now known as SnowDragon
21:34:19  <Yexo> planetmaker: what are your plans for SLOPE_AWARE_SPRITELAYOUT ?
21:34:24  <Yexo> it's not used anywhere currently
21:37:33  <Yexo> and were was that script that allowed browsing the opengfx sprites by number?
21:38:43  <Brot6> Japanese Trains - Revision 41:82e666fd2a00: Update D50 (steam) graphics (dandan) @
21:39:36  <Terkhen> Yexo: <--- do you mean this?
21:39:44  <Yexo> yes, I found that one now :)
21:40:47  <Terkhen> ok :)
21:41:15  <planetmaker> the link terkhen gave is what I use, too
21:41:38  <planetmaker> you can also browse the same way the others. except the extra which has no fixed numbers
21:42:08  <planetmaker> wrt the sprite layouts.... let's see.
21:42:15  <Yexo> suggestions for better names?
21:42:17  <planetmaker> Either it's old. Or it's new and "better"
21:42:41  <Yexo> planetmaker: here you added it
21:42:54  <Yexo> but as far as I can see it was never used
21:43:18  <planetmaker> well. The obvious difference to the other sprite layout is "foundations"
21:43:32  <planetmaker> if you want ground-aware stuff with foundations, that's the layout to use
21:43:54  <planetmaker> the other one must fail there as it will wrongly show a sloped sprite
21:44:02  <Yexo> the dairy farm uses GROUND_AWARE_SPRITELAYOUT and that one displays foundations
21:44:05  <planetmaker> thus it's not yet used
21:44:31  <Yexo> <Yexo> it's not used anywhere currently <Yexo> planetmaker: what are your plans for SLOPE_AWARE_SPRITELAYOUT ?
21:44:50  <planetmaker> I forgot :-)
21:45:05  <Yexo> but you do plan to use it, ok :)
21:45:46  <planetmaker> I *think* the benefit was that it can do with a LOT of less calculations
21:45:55  <planetmaker> no slope calculation needed in advance or so
21:46:38  <Yexo> I think you're mixing them up
21:46:52  <Yexo> the slope-aware one (=the one with more calculations) is not used
21:48:57  <planetmaker> Hm... then, I guess I prepared it for use with the farms
21:49:03  <planetmaker> as fields on foundations are... odd
21:50:04  <planetmaker> removing the foundations from farm fields was at least one of the things I was working on
21:50:35  <Yexo> ok
21:51:12  <planetmaker> we could remove all the nfo code, though
21:51:49  <Brot6> Japanese Trains - Revision 42:9b77b8726cfd: Update 8620 (steam) graphics (dandan) @
21:52:42  <planetmaker> what are your plans with firs currently?
21:53:07  <planetmaker> I mean... feel free to tackle the farm fields or so :-)
21:53:16  <Yexo> none, I was looking to copy spritelayouts_groundaware.pnml to opengfx+industries
21:53:28  <planetmaker> I hope I get a bit time after Christmas and then I wanted to look a bit at supply code
21:53:31  <planetmaker> ah
21:53:42  <planetmaker> so you're fixing / improving that :-)
21:53:59  * andythenorth would like to name all the spritelayouts in FIRS
21:54:04  <Yexo> but I have a few commits todo to fix up nml for grf v8
21:54:09  <Yexo> and than a lot of documentation to fix
21:54:21  <planetmaker> he, yeah... we're never out of work
21:54:27  <andythenorth> also the THIS_ID(foo) stuff is just hard to read for me - dunno if that can be changed or not?
21:54:30  <planetmaker> strange as it sounds
21:54:39  <planetmaker> andythenorth: I'd not change that
21:54:49  <planetmaker> it can, of course be changed
21:54:57  <planetmaker> though the templates absolutely require its use
21:55:00  <andythenorth> maybe I just learn to read it
21:55:09  <andythenorth> currently I feel overwhelmed by parentheses
21:55:12  <andythenorth> :D
21:55:16  <planetmaker> it's the only easy way templates for this stuff can work
21:56:38  <Brot6> Japanese Trains - Revision 43:da4c8e6843d1: Update 9750 (steam) graphics (dandan) @
21:56:55  <planetmaker> I'm happy about dan's use of the repo :-)
22:02:25  *** andythenorth has quit IRC
22:12:24  <planetmaker> btw, Yexo, the constants wrt the transition ground tiles sound good
22:15:23  <Brot6> NewGRF Meta Language - Revision 1755:bafc2593f2bd: Add: a few constants for generic groundsprites... (yexo) @
22:15:23  <Brot6> NewGRF Meta Language - Revision 1756:98d3a70a38b3: Add: two constants for the articulated vehicle... (yexo) @
22:15:23  <Brot6> NewGRF Meta Language - Revision 1757:62d882f377ce: Add: new constant CB_RESULT_NO_TEXT for use in... (yexo) @
22:15:27  <Brot6> NewGRF Meta Language - Revision 1758:7250cf83a13f: Fix: line endings (yexo) @
22:15:30  <Brot6> NewGRF Meta Language - Revision 1759:a4e8ba37ac81: Add: backport some constants (yexo) @
22:15:51  *** frosch123 has quit IRC
22:25:13  <Brot6> NewGRF Meta Language - Revision 1760:816a3c5adbf1: Fix: update examples to use the new constants ... (yexo) @
22:44:47  <Yexo> snowline table still needs looking at, for the rest I think I'm done for grf v8 (for both code and documentation)
22:46:02  <planetmaker> what about the granularity of the height variable?
22:46:13  <planetmaker> keep at 8px or switch to real height levels?
22:47:21  <Yexo> for now I've kept them the same (so multiply openttd value by 8)
22:48:30  <Yexo> going to tile-height is probably a good idea, although I'd like to reflect that clearly in the variable names
22:48:54  <Yexo> so *_tile_height and/or *_pixel_height instead of *_height
22:52:11  <Brot6> NewGRF Meta Language - Feature #3224 (Closed): Support for NewGRF v8 (yexo) @
23:00:28  <Brot6> NewGRF Meta Language - Bug #3362 (New): nearby_tile_height is broken (yexo) @
23:09:36  <planetmaker> yes, using tile_height and pixel_height is a good suggestion
23:10:19  <planetmaker> though pixel_height is a bit strange name
23:11:09  <planetmaker> but everything else I can think of now will be bulky and not necessarily clearer
23:11:47  <Yexo> nearby_tile_tile_height is not so nice though
23:12:47  <planetmaker> true
23:12:56  <planetmaker> skip one tile_
23:13:13  <planetmaker> and give the other one the additional suffix _pixels
23:13:34  <planetmaker> not sure we need both, though. Or do we?
23:14:01  <planetmaker> hm. possibly nicer for transition, though
23:14:22  <Yexo> in nml 0.2 nearby_tile_height is in pixels
23:14:29  <planetmaker> yes
23:14:36  <Yexo> changing that to tiles is not nice
23:14:53  <planetmaker> Well... not exactly nice. But I think it's do-able
23:14:59  <Yexo> true
23:17:03  <planetmaker> so ... I suggest to convert the existing names to tile heights. And introduce legacy names with the suffix _pixels
23:17:14  <planetmaker> Then we can add to the transition guide to rename the variables
23:17:18  <planetmaker> which is not difficult
23:17:44  <planetmaker> and we'll have the most often used variables named sensibly
23:17:50  *** JVassie has quit IRC
23:22:44  <Brot6> FIRS Industry Replacement Set - Revision 2652:26d7bddc5874: Codechange: use the new constants for... (yexo) @
23:27:47  *** ODM has quit IRC
23:29:00  *** Zuu has quit IRC
23:31:54  <planetmaker> good night

Powered by YARRSTE version: svn-trunk