Log for #openttdcoop.devzone on 18th June 2011:
Times are UTC Toggle Colours
01:34:07  *** Lakie has quit IRC
06:26:31  *** andythenorth has joined #openttdcoop.devzone
06:38:40  *** ODM has joined #openttdcoop.devzone
08:01:45  <Yexo> planetmaker: not sure if you remember the history, but dimme was the one who made the modular airports minigame
08:01:58  <Yexo> I don't think he's interested in the graphics, just in the statemachines
08:02:19  <planetmaker> yes, iirc he was that person
08:03:23  <planetmaker> I only try to get people play with what we have :-) If he doesn't catch it, maybe someone else
08:04:50  *** frosch123 has joined #openttdcoop.devzone
08:14:53  <planetmaker> ha... saving by layer name works :-)
08:15:49  <Terkhen> :)
08:17:00  <planetmaker> now only make it nicely usable ;-)
08:29:50  <andythenorth> planetmaker: that would be win
08:30:13  <planetmaker> definitely
08:30:26  <planetmaker> by number is a pain if a layer is added
08:30:39  <planetmaker> now it just needs a unique layer name
08:30:51  <planetmaker> s/a//
08:30:58  <planetmaker> s/name/names/ ;-)
08:55:04  <Yexo> frosch123: when skipping the groundsprite via the advanced sprite layout, will all 'extra' ground sprites also be skipped?
08:55:27  <Yexo> I mean all childsprites directly following the groundsprite before the first parent sprite
09:03:39  <Brot6> CHIPS Station Set - Revision 142:3e6027faf7a8: Feature: split menu to 'stations' and 'tiles' classes (andythenorth) @
09:03:39  <Brot6> CHIPS Station Set - Feature #2690 (Closed): Decide whether to have one or two classes of station (andythenorth) @
09:13:29  <Brot6> CHIPS Station Set - Feature #2689: Remove deprecated clay tiles (andythenorth) @
09:13:30  <Brot6> CHIPS Station Set - Feature #2493: Parcels office (andythenorth) @
09:16:06  *** andythenorth has left #openttdcoop.devzone
09:17:18  <frosch123> Yexo: no
09:17:37  <Yexo> ok, thanks
09:49:17  <Brot6> OpenGFX+ Trains - Revision 244:2b1394232927: -Feature: [Makefile] Allow to use xcf as direct sour... (planetmaker) @
09:49:17  <Brot6> OpenGFX+ Trains - Bug #2654 (Closed): container colours upon train reversal (planetmaker) @
09:57:28  *** andythenorth has joined #openttdcoop.devzone
10:22:28  *** JVassie has joined #openttdcoop.devzone
11:08:38  <Hirundo> Nice work on NML, Yexo
11:09:21  <Yexo> almost finished work on #2739 bit 0
11:09:21  <Brot6> Yexo: #2739 is "NewGRF Meta Language - Feature #2739: Advanced sprite layouts - #openttdcoop Development Zone"
11:11:03  <JVassie> oooh
11:11:30  <planetmaker> sometimes I really appreciate Brot's auto-quote of issues :-)
11:17:45  <Terkhen> :)
11:18:23  <Brot6> NewGRF Meta Language - Revision 1416:517aae5b906a: Feature #2739: hide_sprite property in tilelay... (yexo) @
11:30:02  <Brot6> OpenGFX+ Trains - Bug #2734 (Rejected): DevZone compile failed (planetmaker) @
11:55:27  <Ammler> IMO, we could release a 0.6.5-beta and upload to bananas with version restrictions
11:55:39  <Brot6> NewGRF Meta Language - Revision 1417:228652066b28: Fix: various bugs in hide_sprite implementation (yexo) @
11:55:45  <Ammler> or call it 0.6.5-nml :-)
11:56:04  <Yexo> Ammler: andy wanted to do more testing today before uploading it
11:56:08  <Ammler> (speaking about firs)
11:56:26  <Yexo> planetmaker: This adds the hide_sprite property to the airport fence defines
11:56:38  <Yexo> I couldn't find a good way to add those fence defines to every spritelayout
11:57:11  <Ammler> well, for andy it is not possible to upload a test version to bananas, which is wrong
11:58:01  <Ammler> I don't think, if you call the version beta, it does harm firs reputation ;-)
11:58:05  <Yexo> I'm not starting that discussion again, it's pointless. Uploading a new version with version restrictions would make the old version unavailable, which is bad
11:58:15  <Yexo> nobody cares about "beta", except developers
11:58:40  <Yexo> and there is nothing wrong with making reasonable sure a version works before uploading it
11:58:53  <Ammler> Yexo: I wasn't aware, bananas is broken, never mind then
11:59:09  <Yexo>
12:01:06  <frosch123> [13:56] <Yexo> planetmaker: This adds the hide_sprite property to the airport fence defines <- does that mean, it works? :)
12:01:16  <Yexo> yes :)
12:02:17  <Yexo> frosch123: I was confused for a while about bit 0: "the "skip bounding box"-register uses value 1 to draw the sprite, and 0 to skip it, "
12:02:29  <Yexo> I had implemented it as 1, skips, 0 draws
12:02:48  <Ammler> well, the other alternative is to make a 2nd testing entry
12:02:48  <Yexo> that was also due to my naming "hide_sprite"
12:03:50  <frosch123> :)
12:05:28  <Ammler> (like LumberMill and LumberMill PS)
12:06:04  <planetmaker> I'll give it a shot, Yexo and try to enable it :-) Thanks a bunch
12:13:57  <Yexo> frosch123: did you plan on making nforenum support the advanced sprite layouts? if not I can work on that now
12:15:24  <Brot6> CHIPS Station Set - Revision 143:bd4c9ef58294: Change: replace Clay tiles with other tiles (andythenorth) @
12:16:48  <frosch123> Yexo:
12:16:56  <frosch123> action 0 support is finished, but completely untested
12:17:09  <frosch123> action2 is basically unstarted
12:17:15  <frosch123> feel free to pick it up
12:17:21  <frosch123> i am doing wiki today :)
12:17:35  <Yexo> ok, I will try to finish action2 support
12:18:22  <frosch123> i guess best is to revert changes in src/act123.cpp :)
12:19:12  <frosch123> hmm, i guess i forgot to bump the action data format version
12:19:18  <Yexo> why is that? it looks like a good start
12:19:20  <frosch123> there are new syntax elements
12:19:38  <Hirundo> Yexo: It might make sense, to remove the difference between 'ttdsprite' and 'sprite' in spritelayouts
12:20:04  <Yexo> makes sense
12:20:05  <Hirundo> i.e. make 'ttdsprite' an alias of 'sprite', and emit a warning upon its use
12:20:19  <frosch123> Yexo: the whole offset issue is quite complicated there
12:21:19  <Brot6> FIRS Industry Replacement Set - Revision 2062:18c9ff60fb21: Fix: mistake with snow for Builders Yard (andythenorth) @
12:21:36  * andythenorth has to remember to be in right branch :P
12:23:14  <Yexo> Hirundo: currently we have "sprite: spriteset_name;", what about "sprite: spriteset_name(offset);" for the advanced layout?
12:23:24  <Yexo> where offset is any varaction2 expression
12:23:38  <Hirundo> There is already such thing implemented, for constants only
12:23:39  <Yexo> indicating the offset within the spriteset, ie bit 1
12:23:48  <Yexo> there is? good :)
12:23:50  <Yexo> that makes it easy
12:23:57  <Hirundo> Though I agree, it might be documented quite poorly
12:24:34  <Yexo> ah indeed, SpriteGroupRef can have params already
12:24:36  <Hirundo> There's also the possibility of adding labels to spritesets
12:27:04  <Hirundo> Are you gonna work on that now, or do you have any pending changes in action2layout?
12:27:15  <Yexo> I have no pending changes and I'm going to work on nforenum first
12:27:40  <Yexo> and before I do any more work on nml I'm going to document some of the new code
12:30:33  <Brot6> FIRS Industry Replacement Set - Revision 2063:d1ce3e29b605: Fix: Plastic Plant was using broken g... (andythenorth) @
12:30:33  <Brot6> FIRS Industry Replacement Set - Revision 2064:d0d1ccf81c27: Cleanup: remove unneeded pcx and temp... (andythenorth) @
12:31:40  <Brot6> FIRS Industry Replacement Set - Bug #2747 (Closed): Plastic Plant snow sprites out of date (andythenorth) @
12:31:40  <Brot6> FIRS Industry Replacement Set - Feature #1786 (Closed): Builders Yard needs snow graphics (andythenorth) @
12:32:07  <planetmaker> hm... it would have helped, if I had pulled the CBF rename version straight away :-P
12:34:25  <Brot6> FIRS Industry Replacement Set - Revision 2065:038a50abad57: Cleanup: remove unneeded pcx files (andythenorth) @
12:35:54  <planetmaker> works like a charm, Yexo :-)
12:36:09  <Yexo> great :)
12:36:30  <planetmaker> I guess I'll add fences to all airports and we could then release it as 0.2.0 or so
12:36:40  <planetmaker> re-written in NML and with one rotation for the small one
12:36:56  <Yexo> sure thing
12:37:07  <Yexo> only thing to watch out for is fences at the end of runways
12:37:18  <planetmaker> that's already taken care of. There'll never one
12:37:28  <planetmaker> +bw
12:37:31  <planetmaker> +be :-P
12:37:50  <planetmaker> though there are default airports where there is actually
12:37:57  <planetmaker> but I don't want to copy that detail
12:38:21  <Yexo> indeed, imo that's a bug of the default airports
12:44:28  <frosch123> all fences are ground/child sprites, right?
12:44:40  <planetmaker> yes
12:45:03  <frosch123> fine :) bounding boxes for them result in the classical foundation glitch
12:45:17  <planetmaker> actually... the fence is a building sprite. But...
12:45:54  <frosch123> so, you readd that bug :p
12:46:00  <planetmaker> hm?
12:46:52  <frosch123> r21272 | frosch | 2010-11-20 15:45:52 +0100 (Sa, 20 Nov 2010) | 1 line
12:46:53  <planetmaker> also with fences as building?
12:46:53  <frosch123> -Change [FS#119-ish]: Draw airport fences as groundsprites on runway/taxiway tiles. This makes the usual foundation-glitch a lot less likely to happen.
12:47:18  <planetmaker> well, it's easy to define them as child sprite
12:47:28  <frosch123> if the fence bounding box is above the foundation, it will force the foundation above the plaint in most cases
12:47:46  <frosch123> what do you mean with 'fences as buildings' ?
12:49:02  <planetmaker> currently I define them as building sprite
12:49:25  <planetmaker> within the sprite layout
12:49:28  <planetmaker> of a tile
12:50:33  <Yexo> planetmaker: yes, that is exactly what frosch was talking about
12:50:48  <Yexo> to make them childsprites make sure they're defined directly after the groundsprite
12:51:34  <planetmaker> hm, ok. also feasible, but currently they're not that, indeed
12:51:42  <planetmaker> should be easy to change :-)
12:53:34  <planetmaker> so, thanks for the hint, frosch123 :-)
12:53:41  <Brot6> FIRS Industry Replacement Set - Revision 2066:544788e66e75: Change: partial snow support for Lime... (andythenorth) @
12:54:45  <planetmaker> andythenorth: I still think you should use the default ttd snowy ground tile and draw the other stuff on top of it. That'll make it blend in with both base sets
12:54:59  <andythenorth> that may be possible soon
12:55:07  <planetmaker> it's also possible now
12:55:20  <andythenorth> not without throwing away the current templating
12:55:42  <planetmaker> it needs slight modification to use always the ground sprite and an overlay
12:56:01  <andythenorth> if it's possible, I miss how :)
12:56:10  <andythenorth> it's one of the cases advanced was invented for
12:56:28  <andythenorth> even if it is solvable by some complex means, advanced layouts is about to solve it (hopefully)
12:59:39  <Yexo> frosch123: This works for the action2's I have so far
13:00:00  <Yexo> I didn't test any action0's, do you have a testcase for that already?
13:00:26  <frosch123> no :p
13:00:30  <Yexo> only update other than act123.cpp is updating the NDF_HEADER to a new version
13:00:49  <planetmaker> andythenorth: I'd simply add in each tile layout: use of the default ground sprite. use of a ground child sprite. And then optionally buildings
13:01:07  <andythenorth> how do you specify the ground child sprite?
13:01:11  <planetmaker> but... well... I just mean: don't bother to draw snowy ground. You'll have that for free
13:01:20  <planetmaker> I can only tell you in NML ;-)
13:01:22  <Brot6> FIRS Industry Replacement Set - advanced_action2_test_sheepfarm.patch (andythenorth) @
13:01:25  <andythenorth> hmm
13:01:27  <planetmaker> but that's what I just do with airports.
13:01:29  <andythenorth> that sounds workable and useful
13:01:39  <andythenorth> for certain cases
13:01:44  <andythenorth> like farm fields
13:01:46  <frosch123> Yexo: flat 0x08 has no data, thus no additional offset
13:01:47  <planetmaker> for all ;-)
13:01:58  <planetmaker> stuff which modifies the default ground is then just drawn as overlay
13:02:02  <andythenorth> I don't see how that works
13:02:13  <planetmaker> of course it doesn't make sense for concrete - unless you use default concrete
13:02:24  <andythenorth> exactly :P
13:02:43  <planetmaker> well... just define a tile layout. Add a ground sprite. Add a child sprite, add a building sprite. Done
13:02:49  <planetmaker> The child sprite is partially transparent
13:03:03  <planetmaker> and adds only what differs from the default ground tile
13:03:13  * andythenorth still doesn't understand :P
13:03:28  <planetmaker> uhm... you know what a tile layout is?
13:03:35  <planetmaker> you can add infinitely many sprites to a tile
13:03:39  <andythenorth> I don't use child sprites
13:03:42  <planetmaker> and arrange them there.
13:03:54  <planetmaker> Thus you could add a stack of wood as child sprite to the ground etc
13:03:56  <andythenorth> I just use sprites with their own bounding box afaik
13:03:57  <planetmaker> oh... you don't?
13:03:59  <planetmaker> You should!
13:04:06  <andythenorth> I don't see how it solves the case though
13:04:14  <andythenorth> a base set sprite number still has to be specific
13:04:16  <planetmaker> default ground: ttdsprite
13:04:16  <andythenorth> specified /s
13:04:20  <planetmaker> addon: yours
13:04:33  <planetmaker> yes. But the base set number of course is known
13:04:37  <andythenorth> ok I get it
13:04:42  <planetmaker> and 95% of your cases is flat
13:04:49  <andythenorth> it's almost what I'm doing now, but change a certain order
13:05:08  <planetmaker> i.e. don't provide ground yourself, just ground modifications
13:05:17  <andythenorth> it would work
13:05:33  <planetmaker> and add a lot compatibility wrt other graphics and also landscape newgrfs
13:06:00  <andythenorth> I'll consider it
13:06:17  <planetmaker> how I hate that word "consider" in this context
13:06:19  <planetmaker> ;-)
13:06:29  <andythenorth> we can't string concatenate in CPP can we ?
13:06:35  <planetmaker> we can
13:06:38  <andythenorth> we can?
13:06:41  <andythenorth> hmm
13:06:42  <planetmaker> blub ## blah
13:06:59  <planetmaker> at least that's how it works in macros
13:07:06  <andythenorth> it would mean reworking the templates for all industries
13:07:10  <planetmaker> #define MACRO(a, b) a ## b
13:07:24  <planetmaker> concatenates a and b
13:07:36  <andythenorth> we'd need (a,b,c)
13:07:37  <planetmaker> when you use MACRO(a, b)
13:08:00  <planetmaker> please elaborate a bit more
13:08:23  <andythenorth> current templating requires
13:08:27  <andythenorth>  #define GRAPHICS  fooindustry.png
13:08:29  <andythenorth> or
13:08:34  <andythenorth>  #define GRAPHICS fooindustry_snow.png
13:08:56  <andythenorth> to make this method work we also need groundtiles.png
13:09:00  <planetmaker> you mean conditionally adding _snow?
13:09:02  <planetmaker> feasible
13:09:03  <andythenorth> and groundtiles_snow.png
13:09:25  <andythenorth> or I have to copy ground tile into every industry png, which I dislike
13:09:40  <andythenorth> already I am doing that for snow, but it's temporary I hope :P
13:10:11  <andythenorth> I don't know if templating will survive transition to nfo
13:10:12  <planetmaker> :-) well... I guess we can fix that when we have NML...
13:10:18  <andythenorth> nml / nfo /s
13:10:25  <planetmaker> the cpp templates for that can be copied from ogfx+airports
13:10:46  <planetmaker> even the conditional fence code now ;-)
13:10:54  * andythenorth wonders if it will be scriptable to migrate existing pnfo
13:10:58  <planetmaker> like draw fences on the outside automatically ;-)
13:11:00  <andythenorth> otherwise it is very painful
13:11:16  <planetmaker> that's hardly scriptable
13:11:29  <planetmaker> that's what the grf2nml does... and then it needs tidying
13:11:47  <planetmaker> but... you'll have help with that
13:11:52  <Yexo> * andythenorth wonders if it will be scriptable to migrate existing pnfo <- definitely not
13:12:01  * planetmaker thinks that 3 people will help andy there
13:12:13  <andythenorth> it's a big job you guys are taking on :P
13:12:26  <andythenorth> you are brave indeed :)
13:12:28  <Yexo> the sooner we can start the better :)
13:12:33  <frosch123> Yexo: diff looks fine
13:12:34  <planetmaker> ^ :-)
13:12:36  <andythenorth> I am quite happy to not be sole maintainer in future
13:12:49  <planetmaker> see. Only to stop that moaning it's worth it :-P
13:13:02  <Yexo> <- frosch123: can you explain that warning?
13:13:06  <Yexo> I can't find it anywhere
13:15:04  <frosch123> if(!hasGround)IssueMessage(WARNING2,NO_GROUNDSPRITE,NONTRANS);
13:15:22  <frosch123> the example has \dx0 for the groundsprite
13:16:27  <Yexo> what is NONTRANS doing there instead of GROUND?
13:17:03  <frosch123> the layout only contains a single sprite, the groundsprite, and that one is 0
13:17:09  <frosch123> so nforenum sees no sprite at all
13:17:15  <Yexo> ah, ok
13:17:23  <frosch123> maybe just disable that test if there are flags?
13:18:09  <frosch123> Yexo: btw. your example is not valid
13:18:24  <frosch123> 40 does not use the syntax of the basic layout
13:18:35  <Yexo> ah, so it needs a zoffs
13:19:04  <frosch123> yes,
13:19:49  <Yexo> ok, it seems to work
13:22:48  <Yexo> frosch123: there is a bug in your action0 code though
13:22:55  <frosch123> :)
13:22:55  <Brot6> FIRS Industry Replacement Set - Revision 2067:f23201850384: Change: convert more things to png an... (andythenorth) @
13:23:05  <frosch123> what?
13:23:33  <Yexo> it spawns a lot of errors about invalid properties
13:23:40  <Yexo> on nfo lines it previously found correct
13:23:54  <Brot6> FIRS Industry Replacement Set - Revision 2068:7074bd71ab1d: Fix: missing files (andythenorth) @
13:23:56  <frosch123> ah, you mean it breaks something :(
13:25:26  <Brot6> FIRS Industry Replacement Set - Revision 2069:35650cf8b5f3: Cleanup: remove further unneeded pcx ... (andythenorth) @
13:31:18  <frosch123> Yexo: you can make the groundsprite test more strict, not all flags are valid there
13:31:31  * andythenorth should automate this pcx->png conversion :P
13:31:41  <andythenorth> but can't be bothered to handle edge cases
13:32:03  <Brot6> FIRS Industry Replacement Set - Revision 2070:9d90f5ffdc70: Cleanup: convert further pcx to png (andythenorth) @
13:32:14  <frosch123> (i.e. xoffs and yoffs are not valid)
13:32:22  <Yexo> frosch123: bits 0, 1, and 2 are valid, 3 is valid but doesn't eat a byte, rest is invalid?
13:32:38  <frosch123> yes, FFF0
13:33:05  <Yexo> This works for all actions2's I've tested
13:33:17  <Yexo> I've left out the action0 code as I don't fully understand it and it breaks things
13:36:06  <frosch123> ground_flags&0x03 <- what is that for?
13:36:19  <frosch123> you mean 0x02 ?
13:36:27  <Yexo> no, I mean 0x03
13:36:35  <Yexo> it checks for ground != 0, so both sprite and palette
13:36:58  <frosch123> so 0x06? :p
13:37:19  <Yexo> yes
13:38:53  <frosch123>  						if(!CheckSpriteNum(building,off+1,act123::Instance().act1,feature,mismatch,hasGround)&& <- that +1 is too much now?
13:39:48  <frosch123> 						if(!building)IssueMessage(ERROR,NO_BUILDING_SPRITE,off+1); <- same
13:39:55  <Yexo> yep, indeed
13:40:08  <Yexo> if(!building)IssueMessage(ERROR,NO_BUILDING_SPRITE,off); <- and that needs a check for the flags too
13:40:25  <frosch123> +							uint x=data.ExtractByte(curoff++),y=data.ExtractByte(curoff++); <- i guess excution order is not well defined there
13:40:53  <Yexo> also fixed
13:41:11  <Yexo> that however was original code
13:41:23  <frosch123> it had no ++ before
13:41:36  <Yexo> right
13:42:31  <frosch123>  						if(++i!=nument1)data.SetEol(off,2); <- no idea, whether that needs a - 1 now
13:42:59  <frosch123> looks fine :)
13:43:06  <frosch123> i should test action 0 better
13:49:33  <Yexo> fixed that and another SetEol line
13:49:45  <Yexo> shall I commit? or do you want to fix action0 and this in one go?
13:50:11  <frosch123> go ahead and commit :)
13:50:16  <Brot6> OpenGFX+ Airports - Revision 94:e9bbeee37fc7: Feature: Automatically draw fences around airport t... (planetmaker) @
13:51:16  <planetmaker> and now building -> childsprite... that needs some serious offset fixes
13:51:42  <Brot6> GRFCodec - Revision 832:08511c778a07: Feature: support the advanced sprite layout in action2 (bas... (yexo) @
13:52:51  <Yexo> Hirundo: to avoid double work: are you working on nml and if so, in which files?
13:53:27  <Hirundo> I'm working in action2layout currently
13:53:28  <frosch123> planetmaker: you can compute them
13:53:32  <Hirundo> I'll see if I can commit shortly
13:53:40  <Yexo> ok
13:54:11  <frosch123> planetmaker: xoffs = 2 * (dy - dx), yoffs = dx + dy - dz
13:55:35  <planetmaker> dx and dy of the ground sprite or the child sprite?
13:58:16  <Hirundo> Yexo: I'll commit now to avoid merging conflicts later, as I got to leave right now
13:58:33  <Yexo> fine by me
13:58:34  <Yexo> have fun :)
13:58:35  <Hirundo> Mind that the code is tested to the point that it passes regression
13:58:42  <Hirundo> i.e. pretty badly :)
14:00:14  <Hirundo> hmm... I missed a commit already and need to merge
14:00:23  <Hirundo> Just continue your work, I'll clean up the mess later
14:00:32  <Hirundo> goodbye for now
14:05:36  <frosch123> planetmaker: dx, dy,dz of the boudingbox
14:05:48  <frosch123> xoffs, yoffs of the equivalent childsprite offset
14:07:42  <planetmaker> but... ground tiles have no bounding box?
14:09:31  <planetmaker> i.e. I'd need to move the fences in the SE and SW left. By about 31 px. But the offset can only be positive...
14:12:55  <Yexo> try 15, 0, 0 for SW and 0, 15, 0 for SE
14:13:12  <frosch123> the groundsprite-childsprites offset from the same position as the groundsprite and as building sprites
14:13:22  <frosch123> so, no 31 etc needed
14:14:00  <frosch123> just put your current dx, dy, dz offsets into those formulas and get the new xoffs, yoffs
14:14:16  <Yexo> <Yexo> try 15, 0, 0 for SW and 0, 15, 0 for SE <- those values are from the openttd source code
14:14:39  <planetmaker> uhm... Yexo ? I only have xoffset and yoffset at my disposal
14:14:44  <planetmaker> or I need to use a building sprite
14:14:48  <Yexo> so leave out the last one
14:14:53  <planetmaker> But not using that was the whole issue, right?
14:15:00  <Yexo> yes
14:15:11  <planetmaker> and a positive x offset... is the wrong direction
14:15:20  <Yexo> so xoff=15, yoff=0 for SW, xoff=0, yoff=15 for SE
14:17:33  <planetmaker> <-- see yourself. Applies to head
14:17:43  <frosch123> planetmaker:
14:18:07  <frosch123> (i am not sure whether xoffset and yoffset is the right syntax in childsprite
14:18:14  <planetmaker> it is
14:18:28  <planetmaker> but that gives wrong results for me
14:19:21  <frosch123> you are not using the formula
14:19:40  <planetmaker> 15 is 0x0F
14:19:51  <frosch123> 		xoffset: 2 * (0x0F - 0x00);\
14:19:53  <frosch123> 		yoffset: 0x0F + 0x00 + 0x00;\
14:19:56  <frosch123> ^^ in my example
14:20:05  <frosch123> +		xoffset: 0; \
14:20:07  <frosch123> +		yoffset: 15; \
14:20:10  <frosch123> ^^ in your code
14:20:44  <Yexo> ah, I missed a macro
14:32:35  <planetmaker> well, I still don't get it
14:32:42  <planetmaker> or something's bugged
14:34:31  <planetmaker> hm...
14:37:00  <Yexo> planetmaker: xoffset:30, yoffset:15 works fine for the SE fences
14:39:45  <Yexo> frosch123: for the SE fence dx=15, dy=0
14:40:07  <Yexo> that means that the formula gives xoffset: 2*(0-15) which is -30
14:40:13  <Yexo> but negative values are not allowed
14:41:30  <Yexo> planetmaker: it seems to be a bug in nml
14:41:40  <Yexo> nfo specs has <xofs> and <yofs> as signed byte
14:42:01  <planetmaker> he
14:42:32  <planetmaker> and I've been playing with the SW fence ;-)
14:44:28  <Yexo> sorry for that, I'm not reading the specs properly
14:44:41  <Yexo> that was for sprites defining a new bounding box
14:44:53  <Yexo> for child sprites it just says "x offset from the top left corner of the previous sprite"
14:45:07  <Yexo> which makes sense to be unsigned, as it is in openttd
14:45:26  <Yexo> so currently it's not possible to fix this while using the default fence sprites
14:45:34  <Yexo> we'll have to include the fences in the grf
14:47:46  *** Lakie has joined #openttdcoop.devzone
14:49:04  <frosch123> ah, too bad.
14:49:28  <frosch123> the signed/unsigned thingie is a difference in default- and station spritelayouts to other spritelayouts
14:51:34  <frosch123> hmm, no, they are signed for action2
14:51:39  <frosch123> it should work
14:51:40  <Yexo> frosch123: on the station action0 page it's not defined whether they signed or unsigned
14:51:49  <Yexo> frosch123: but not for sprites sharing their bounding box
14:53:33  <frosch123> yeah, child_offset_is_unsigned is indeed true for action2 spritelayouts
14:54:04  <planetmaker> so... keep it as building sprite?
14:54:17  <Yexo> no, include the fence as spriteset in the grf
14:54:32  <planetmaker> ok, I'll try
15:46:40  <Brot6> OpenGFX+ Airports - Revision 95:062694f20612: Change: Define our own fences thus that glitches wi... (planetmaker) @
15:47:32  *** ODM has quit IRC
15:59:24  <Brot6> OpenGFX+ Airports - Revision 96:f5b2736da5b4: Change: Apply r95 also to the small airport (planetmaker) @
16:23:46  <Brot6> OpenGFX+ Airports - Revision 97:d32b0898e253: Chage: Add a bit more fence to the small airport (planetmaker) @
16:36:34  *** andythenorth is now known as Guest5108
16:36:34  *** andythenorth has joined #openttdcoop.devzone
16:39:45  <Brot6> OpenGFX+ Airports - Revision 98:89977e431db9: Doc: Add readme and changelog (planetmaker) @
16:56:10  <Brot6> OpenGFX+ Airports - Revision 99:9648b1235b99: Added tag 0.2.0 for changeset 89977e431db9 (planetmaker) @
16:57:10  <Brot6> OpenGFX+ Airports - Bug #2752 (New): DevZone compile failed (compiler) @
17:01:33  <planetmaker> :-( Why, Ammler ?
17:01:52  <planetmaker> Is it (again) the last nightly of NML which triggers this?
17:02:48  <planetmaker> I don't see a real error in any of the logs... but maybe I'm blind
17:08:08  <Brot6> grfcodec: update from r831 to r832 done -
17:09:29  <Yexo> planetmaker: you've comitted the tag before the nightly run from nml
17:09:39  <Yexo> hence it uses the nml nightly from yesterday, which will indeed fail
17:10:07  <planetmaker> meh... but even then I don't see in the logs where it fails. That bothers me even more
17:10:17  <Yexo> that is indeed wrong
17:11:13  <Brot6> nml: update from r1414 to r1417 done -
17:11:19  <planetmaker> :-D
17:12:14  <Yexo> Ammler: I did some changes to the nml output recently, all errors / warnings now appear on stderr
17:12:18  <Yexo> previously some ended up on stdout
17:12:32  <Yexo> could that be the reason the error log doesn't contain the useful error messages?
17:14:02  <planetmaker> hm, possibly, yes.
17:18:03  <Brot6> airportsplus: update from r93 to r99 done -
17:18:23  <Brot6> chips: update from r141 to r143 done (1 errors) -
17:19:30  <Brot6> firs: update from r2060 to r2070 done -
17:20:00  <Brot6> FIRS Industry Replacement Set - Revision 2071:30e5e2bfe6dd: Feature: snow for Lime Kiln (andythenorth) @
17:20:00  <Brot6> FIRS Industry Replacement Set - Feature #1794 (Closed): Lime Kiln needs snow graphics (andythenorth) @
17:20:08  <Brot6> ogfx-trains: update from r242 to r244 done -
17:20:11  <Yexo> will the cf automatically rebuild airportsplus 0.2.0 or does that need a manual prod?
17:20:18  <Brot6> Following repos didn't need a nightlies update: 2cctrainset (r750), 32bpp-extra (r40), ai-admiralai (r75), ai-aroai (r39), ailib-common (r21), ailib-direction (r17), ailib-list (r32), ailib-string (r29), ailib-tile (r16), basecosts (r25), belarusiantowns (r8), bros (r52), comic-houses (r71), fish (r653), frenchtowns (r6), german-townnames (r34), grfcodec (r832), grfpack (r279), heqs (r605), indonesiantowns (r41), manindu (r7),
17:20:18  <Brot6> metrotrackset (r56), narvs (r37), newgrf_makefile (r294), nml (r1417), nutracks (r202), ogfx-industries (r119), ogfx-landscape (r69), ogfx-rv (r107), ogfx-trees (r50), opengfx (r678), openmsx (r97), opensfx (r97), smts (r19), snowlinemod (r49), spanishtowns (r10), sub-landscape (ERROR r66), sub-opengfx (ERROR r666), swedishrails (r202), swisstowns (r22), transrapidtrackset (r15), ttdviewer (r34), ttrs (r36), worldairlinersset (r672)
17:20:22  <planetmaker> I'm not sure. Let's see ;-)
17:24:36  <planetmaker> Yexo: AirportsPlus / climate dependent airports / OpenGFX+ Airports has no release thread yet?
17:24:55  <Yexo> not sure, quite possible
17:27:00  <planetmaker> oh... I didn't see zero.eight's very constructive and helpful postings...
17:27:12  <planetmaker> well :-) And he should be credited for the snowy versions of the graphics
17:27:20  <Yexo> yes, isn't he?
17:27:58  <Brot6> ogfx-industries: rebuild of r119 done (Diffsize: 32645) (DiffDiffsize: 32691) -
17:28:10  <planetmaker> no, he isn't. I wasn't aware of him doing that
17:28:18  <planetmaker> and there was no pre-existing readme in 0.1.0
17:28:40  * andythenorth climbs a FIRS ticket mountain slowly
17:28:41  <Brot6> ogfx-landscape: rebuild of r69 done (Diffsize: 1176) (DiffDiffsize: 1168) -
17:29:03  <planetmaker> hm... I could add the release posting just to the dev thread... can be discussed much better IMHO
17:29:40  <planetmaker> could you change the first posting's title to OpenGFX+ Airports please, Yexo ?
17:29:46  <planetmaker>
17:29:47  <Webster> Title: Transport Tycoon Forums View topic - [OTTD] Climate dependant airports (at
17:30:00  <Yexo> sure
17:30:06  <Yexo> keep the [OTTD] or not?
17:30:25  <planetmaker> well... I don't mind either way
17:30:43  <Yexo> added a link to the forum topic here:
17:31:22  <Yexo> seems no rebuild will be done
17:31:33  <Yexo> I'd say credit zero.eight and just release 0.2.1
17:32:05  <planetmaker> 0.2.0a ;-)
17:32:28  <Yexo> I don't like messing that much with version numbers :)
17:32:35  <Brot6> FIRS Industry Replacement Set - Feature #2753 (New): Stockyard snow sprites (andythenorth) @
17:32:41  * planetmaker neither. Was rather meant as joke :-)
17:32:48  <Yexo> ;)
17:33:11  <Brot6> sub-landscape: compile of r66 still failed (#2616) -
17:33:31  <Brot6> FIRS Industry Replacement Set - Feature #2754 (New): Recycling Depot snow sprites (andythenorth) @
17:33:55  <andythenorth> it's ticket closed, two tickets opened in FIRS land :P
17:34:42  <Brot6> sub-opengfx: compile of r666 still failed (#2586) -
17:36:20  <Brot6> swedishrails: rebuild of r202 done (Diffsize: 3304) (DiffDiffsize: 3185) -
17:37:34  <Brot6> OpenGFX+ Airports - Revision 100:2a8040aa4ce1: Fix: Changelog and credits needed an update (planetmaker) @
17:38:26  <Brot6> Following repos rebuilds successful without any difference to earlier nightlies builds: 2cctrainset (6 errors), 32bpp-extra (2 errors), basecosts, belarusiantowns (Diffsize: 30), bros (1 errors), comic-houses (3 errors) (Diffsize: 22), fish, frenchtowns, german-townnames, heqs, indonesiantowns (1 errors) (Diffsize: 1), manindu (Diffsize: 1), metrotrackset (Diffsize: 1), narvs (Diffsize: 3015), newgrf_makefile, nutracks (9 errors),
17:38:26  <Brot6> ogfx-rv (Diffsize: 4590), ogfx-trees, opengfx (Diffsize: 7), smts (Diffsize: 8), snowlinemod, spanishtowns (Diffsize: 1), swisstowns, transrapidtrackset (Diffsize: 12), ttrs (7 errors) (Diffsize: 1324), worldairlinersset
17:39:24  <Brot6> FIRS Industry Replacement Set - Revision 2072:64b16b77667d: Fix: windmill using old graphics in s... (andythenorth) @
17:40:29  <Brot6> FIRS Industry Replacement Set - Feature #2755 (New): Grain Mill snow sprites need finishing (andythenorth) @
17:42:03  <Brot6> repository /home/hg/bmss registered in Redmine with url /home/hg/bmss
17:42:03  <Brot6> repository /home/hg/bmss created
17:42:43  <planetmaker> bmss?
17:43:37  <Yexo> british modular station set
17:45:20  <Ammler> planetmaker: / Yexo, did the nml issue solve itself?
17:45:26  <Ammler> (didn't read everything back)
17:45:31  <Yexo> Ammler: yes, somewhat
17:45:33  <planetmaker> not really.
17:45:36  <Ammler> :-P
17:45:37  <Yexo> tag was pushed before the nml nightly
17:45:39  <planetmaker> :-D
17:45:45  <Yexo> so it used nml r1414, which failed
17:45:47  <planetmaker> Ammler: still: NML errors are not logged anymore
17:45:53  <Yexo> the nightly compiled fine using nml r1417
17:45:59  <Yexo> however the errors ^^
17:46:00  <Ammler> planetmaker: nowhere?
17:46:07  <Ammler> there are different log files
17:46:13  <planetmaker> nowhere I see. Look at ogfx+airports 0.2.0
17:46:14  <Ammler> or just in the wrong
17:46:43  <Ammler> hmm, which is in the error dir
17:46:48  <Yexo> Ammler: see
17:47:00  <Ammler> planetmaker: retrigger building?
17:47:20  <Ammler> ./ -f -releases airportsplus
17:48:00  <planetmaker> hm... do you really think this is worth a new release, Yexo if we credit him in the forums?
17:48:20  <planetmaker> and of course right now in the nightlies?
17:48:24  <Yexo> planetmaker: yes, especially since the old version isn't published anywhere
17:48:27  <Brot6> airportsplus: compile of 0.2.0 still failed (#2752) -
17:48:48  <Ammler> wow, that was fast :-o
17:49:01  <Yexo> Server returned an error: HTTP Error 404: Not Found
17:49:05  <planetmaker> ^
17:49:38  <Brot6> airportsplus: compile of 0.2.0 still failed (#2752) -
17:58:49  <Brot6> airportsplus: compile of 0.2.0 still failed (#2752) -
17:59:02  <Ammler> majonaise
18:01:17  <Brot6> airportsplus: compile of 0.2.0 still failed (#2752) -
18:02:18  <Brot6> airportsplus: [releases] openSUSE API not reachable, sleep an hour and try again...
18:02:49  <Ammler> added another test to the api
18:10:34  <planetmaker> hm... so it won't compile anything right now?
18:12:54  <Ammler> not with devzone v1 :-)
18:13:27  <Ammler> s/:-)/:-(/
18:14:00  <planetmaker> meh
18:33:14  <Brot6> NewGRF Meta Language - Feature #2746: station support (yexo) @
18:33:43  <Yexo> JVassie: I'm going to postpone starting work on bmss a bit, to see how far we can get with nml support for stations in the near future
18:34:27  <Brot6> FIRS Industry Replacement Set - Feature #2756 (New): Rework Lumber large shed (andythenorth) @
18:47:00  <planetmaker> hm... I should have added a version check... OpenGFX+ Airports of course fails with stable OpenTTD versions now :S
18:47:28  <Yexo> planetmaker: extra reason for 0.2.1
18:48:01  <Yexo> Hirundo: before continuing work on making ttdsprite obsolete, please read #2746 first. If we go for that solution we may not want to make ttdsprite obsolete
18:48:01  <Brot6> Yexo: Hirundo: #2746 is "NewGRF Meta Language - Feature #2746: station support - #openttdcoop Development Zone"
18:53:21  <Yexo> planetmaker: the intercontinental graphics are a step backwards from 0.1.0
18:53:51  <Yexo> see the tiles that don't belong to the airport but are inside it? those are fenced, leaving a grass triangle without fences
18:54:21  <Yexo> compare to where the fencing is different
18:59:24  <planetmaker> Shall that be changed? Easy to do
18:59:41  <planetmaker> I left them out of the layout deliberately
19:00:03  <Yexo> that tile? that's ok, but imo it would look nicer if the tile next to it got a diagonal fence
19:00:16  <Yexo> so the empty tile + diagonal tile are 'together'
19:00:34  <planetmaker> that require a change to that tile. And to the autofencing an exception
19:00:45  <planetmaker> Rather I add the tile to the airport so that there's no fence
19:01:00  <Yexo> that's also better than current
19:01:19  <planetmaker> and the other two 'islands'?
19:02:13  *** ODM has joined #openttdcoop.devzone
19:03:37  <Brot6> airportsplus: update from 0.1.0 to 0.2.0 done -
19:04:29  <Brot6> NewGRF Meta Language - Bug #2740 (Closed): Can't compile very big town names file (yexo) @
19:16:52  <Brot6> OpenGFX+ Airports - Revision 101:e7e4b671aa86: Fix: Update the minimum required version and updat... (planetmaker) @
19:23:15  <Brot6> FIRS Industry Replacement Set - Revision 2073:c700e9b39f4d: Change: framework for snow sprites fo... (andythenorth) @
19:26:10  <Brot6> FIRS Industry Replacement Set - Feature #2757 (New): Forest greeble tile needs snow graphics (andythenorth) @
19:26:10  <Brot6> FIRS Industry Replacement Set - Revision 2074:8fec86bd5639: Change: convert Forest to png (andythenorth) @
19:27:16  <Brot6> OpenGFX+ Airports - Revision 102:ce0de810a333: Change: don't fence the oblique grass tiles (planetmaker) @
19:28:09  <Yexo> planetmaker: doesn't compile due to missing semicolon
19:28:16  <Brot6> NewGRF Meta Language - Revision 1418:0112987af978: Add #2544: very basic man file (yexo) @
19:28:17  <planetmaker> I just noticed :S
19:29:21  <Yexo> Ammler: wrt #2544: The only issue left is the naming. Since even big packages like PIL doesn't do that automatically it's imo not something nml should fix
19:29:21  <Brot6> Yexo: Ammler: #2544 is "NewGRF Meta Language - Feature #2544: rpm - #openttdcoop Development Zone"
19:29:29  <Brot6> OpenGFX+ Airports - Revision 103:e5b56d4ef503: Fix (r102): Syntax error (planetmaker) @
19:29:33  <Yexo> if you want to get rid of that you'll have to fix it during packaging
19:31:32  <Brot6> FIRS Industry Replacement Set - Revision 2075:c8eaa9b927a2: Fix: Yet Another Missing File ® (andythenorth) @
19:32:36  <planetmaker> <-- Yexo ?
19:32:55  <Yexo> testing it myself now too :)
19:33:00  <Yexo> looks better
19:33:15  <planetmaker> then I'll push the tag
19:33:24  <planetmaker> the rest can be fine-tuned later
19:33:25  <Yexo> wait a moment please
19:33:30  <planetmaker> ok
19:33:50  <andythenorth> one day we'll standardise a commit hook for missing files :P
19:34:05  <Yexo> build-on-push
19:34:23  <planetmaker> ?
19:34:35  <Yexo> for missing files, so you get noticed as soon as you push
19:34:39  <planetmaker> bb in a bit
19:35:06  <Yexo> planetmaker: is there no snowy version of the modern depot?
19:37:03  <Ammler> Yexo: "the name policy should be filtered (rpmlint)"
19:40:06  <Yexo> planetmaker: apart from the above all looks fine :)
19:40:18  <Yexo> and that can be fixed later if we don't have graphics now
20:12:25  <planetmaker> hm, you're right. Forgot about that. And yes, there's no snowy version of the (slightly) modified depot OpenGFX uses now.
20:12:45  <planetmaker> Its size is adjusted wrt the original one found in airports.png - which is what zero.eight made snowy
20:13:10  <Yexo> ok
20:13:27  <Yexo> so perhaps we should upload the new png to the forum so someone can make it snowy?
20:13:32  <Yexo> unless you want to do that yourself
20:13:57  <planetmaker> was not that quick as I hoped for. Postponed and forgot ;-)
20:14:04  <planetmaker> I'll amend my release posting
20:17:49  <Yexo> planetmaker: another request from the topic: make the grass under the antenna apron in the international (and perhaps also intercontinental) airport
20:17:50  <Yexo> see
20:18:22  <planetmaker> apron means concrete, right?
20:18:28  <Yexo> yes, sorry
20:18:41  <planetmaker> (I always wondered, assumed, but never looked up) - thanks
20:18:50  <Yexo> especially now the fences are gone (international) it would look better
20:19:02  <Yexo> *intercontinental
20:19:12  <planetmaker> yeah, also for 0.2.2 or 0.3.0... whatever :-)
20:19:42  <Yexo> sure, should I make a ticket?
20:20:45  <planetmaker> would be good, I think :-)
20:22:08  <Brot6> OpenGFX+ Airports - Bug #2758 (New): ground under antenna on intercontinental should be apron ins... (yexo) @
20:22:09  <Brot6> OpenGFX+ Airports - Bug #2759 (New): add snowy version of 'modern' depot (yexo) @
20:22:09  <Brot6> OpenGFX+ Airports - Bug #2752 (Rejected): DevZone compile failed (yexo) @
20:22:35  <planetmaker> @base 01 16 22518
20:22:42  <planetmaker> @base 10 16 22518
20:22:42  <Webster> planetmaker: 57F6
20:27:29  <planetmaker> @base 16 10 120057f6
20:27:29  <Webster> planetmaker: 302012406
20:30:07  <Brot6> OpenGFX+ Airports - Revision 104:ebb2cfbf4760: Added tag 0.2.1 for changeset e5b56d4ef503 (planetmaker) @
20:31:04  <Brot6> airportsplus: update from 0.2.0 to 0.2.1 done -
20:47:35  <Brot6> FIRS Industry Replacement Set - Revision 2076:2960e5eedeb2: Fix: conversion of basecosts was done... (yexo) @
21:42:24  <Ammler> hmm, nmlc isn't part of the source bundle anymore
21:51:21  *** andythenorth has quit IRC
21:57:31  *** JVassie has quit IRC
21:58:50  <Brot6> FIRS Industry Replacement Set - Revision 2077:f229ae5defaa: Fix: callbacks 0x38 and 0x3A always r... (yexo) @
22:09:04  <Brot6> FIRS Industry Replacement Set - Revision 2078:c6244bcc8cd0: Fix: switch-blocks with nvar=0 (retur... (yexo) @
22:23:36  *** ODM has quit IRC
22:25:43  <Brot6> NewGRF Meta Language - Revision 1419:b98645b020e4: Doc: (yexo) @
22:25:43  <Brot6> NewGRF Meta Language - Revision 1420:bf586d7db8c6: Doc: some basic functions in (yexo) @
22:25:43  <Brot6> NewGRF Meta Language - Feature #2544: rpm (Ammler) @
22:32:33  *** frosch123 has quit IRC
22:37:02  <Brot6> NewGRF Meta Language - Feature #2544: rpm (yexo) @
22:42:15  <Ammler> Yexo: I guess, simple workaround is to add the nmlc manually
22:42:28  <Ammler> as it is needed for test only
22:43:30  <Ammler> I guess, I have a fix :-)
22:43:43  <Ammler> cp /usr/src/packages/BUILDROOT/nml-0.1.1-0.x86_64/usr/bin/nmlc .
22:44:22  <Yexo> Ammler: in that case rather run it like: make -C regression/ NMLC="nmlc"
22:45:42  <Ammler> there is no nmlc installed
22:46:21  <Ammler> the build script has no root access
22:47:29  <Ammler> I could make an additional "testsuite rpm"
22:47:58  <Yexo> please don't and wait a bit until I can find a clean way to solve this
22:48:08  <Yexo> all that is just suggesting workarounds
22:48:28  <Ammler> well, the testsuite solution is the proper distro way :-)
22:48:30  <Yexo> all of which don't fix the fundamental problem that the source bundles are unusable
22:48:58  <Ammler> the source bundles aren't unusable
22:49:06  <Yexo> if they miss nmlc they are
22:49:07  <Ammler> just the regression test doesn't work
22:49:15  <Ammler> nmlc will be installed
22:50:42  <Ammler> I assume, if you use install without --root and as user root and then run regresion as you said, it works
22:51:02  <Ammler> the buildroot is just a special rpm way
22:53:22  <Ammler> on our cf, it works, because I use the whole hg repo
22:53:26  <Ammler> instead source bundle
22:54:42  <Yexo> source bundle indeed needs installation before you can run it
22:54:58  <Ammler> yep
22:56:32  <Ammler> which is (imo) fine
22:57:15  <Ammler> the question is can also build just a nmlc?
22:57:17  <Yexo> that means that the fix should indeed be somewhere in the spec file
22:57:58  <Ammler> I mean, can you you build nml to "run from source"
22:59:25  <Ammler> oh, did you change something so that nmlc doesn't need setuptools anymore for running?
23:00:14  <Ammler> ah no, still needed
23:00:20  <Yexo> Ammler: nmlc needs setuptools only if you install it via setuptools
23:00:30  <Yexo> if you just download the repo and run from there you don't need setuptools
23:00:56  <Ammler> yep, but it needs it also for runtime, that is very strange
23:01:17  <Ammler> uncommon, no other python tool I use had that requirement
23:01:37  <Yexo> again: only if you install it via setuptools
23:01:48  <Yexo> which is the same for all scripts build via setuptools
23:01:51  <Ammler> yep, which is the usual way for distro
23:02:30  <Ammler> hmm, doesn't matter
23:02:59  <Yexo> Ammler: I can work around that (=don't let setuptools generate nmlc but include the version from the repo), but then we're back at the same situation where we were with buildtools
23:03:01  <Ammler> just not logical
23:03:09  <Yexo> ie it doesn't work automatically for andy
23:03:43  <Ammler> yeah, that is preferable :-)
23:08:25  <Yexo> which of the two was preferable?
23:08:47  <Ammler> the andy way
23:09:17  <Ammler> the setuptools requirement is just something what makes me wondering, why
23:09:22  <Ammler> it does not really harm
23:47:32  <Brot6> NewGRF Meta Language - Revision 1421:fac192646f82: Fix r1416: switch-blocks with parent scope wer... (yexo) @
23:52:17  <Brot6> NewGRF Meta Language - Revision 1422:276b83ca3cb7: Fix: make sure the number of sets per action1 ... (yexo) @

Powered by YARRSTE version: svn-trunk