Log for #openttdcoop.devzone on 30th December 2011:
Times are UTC Toggle Colours
00:00:42  <planetmaker> just running bootstrap fails?
00:00:57  <planetmaker> hm... it was - iirc - what created the egg files... but memory is fuzzy
00:01:29  <planetmaker> ah... buildout?
00:01:32  <Yexo> yes
00:01:45  <Yexo> (=setuptools) creates egg files
00:02:51  <planetmaker> python does *something*
00:03:33  <planetmaker> but I don't know what :-)
00:04:30  <planetmaker> must have refreshed develop-eggs and friends
00:05:39  *** KenjiE20 has quit IRC
00:55:08  *** JVassie_ has quit IRC
00:55:09  *** JVassie_ has joined #openttdcoop.devzone
01:21:09  *** Zuu has quit IRC
02:09:36  *** JVassie_ has quit IRC
08:44:18  *** ODM has joined #openttdcoop.devzone
09:24:47  <Brot6> OpenGFX - Bug #3429 (Confirmed): Semaphore signals (planetmaker) @
09:57:14  *** andythenorth has joined #openttdcoop.devzone
10:02:27  <andythenorth> Yexo: so...any ideas on FIRS bug #3428? :)
10:02:28  <Brot6> andythenorth: Yexo: #3428 is "FIRS Industry Replacement Set - Bug #3428: Metal Fab. Plan and Metal Workshop incorrectly using same tile - #openttdcoop Development Zone"
10:06:04  <andythenorth> hmm
10:06:11  * andythenorth has suspicion
10:08:20  <andythenorth> item(FEAT_INDUSTRYTILES, THIS_ID(tile), 188)
10:08:31  <andythenorth> what is the 188 doing? ^
10:09:10  <planetmaker> andythenorth: ^^ it's the tileID
10:09:19  <andythenorth> fuck :P
10:09:22  <planetmaker> thus if you use 188 in both cases, you force it :-)
10:09:26  <andythenorth> indeed
10:09:51  <andythenorth> oh :(
10:10:03  <planetmaker> we need an ID list of tiles?
10:10:03  <andythenorth> hmm
10:10:07  <andythenorth> yup
10:10:10  <planetmaker> like for industry IDs
10:10:12  * andythenorth wonders
10:10:36  <andythenorth> the nfo->nml conversion probably preserved the IDs
10:10:44  <Terkhen> it did
10:10:45  <planetmaker> possibly
10:11:00  <andythenorth> so the nfo defines probably are still valid
10:11:05  <Terkhen> we already discussed what was that number and the necessity of a DI list :P
10:11:09  <Terkhen> ID*
10:11:15  <planetmaker> :-)
10:11:53  <andythenorth> so we can probably use the nfo
10:12:03  <planetmaker> hu?
10:12:12  <andythenorth> it's just defines ;)
10:12:33  <planetmaker> well. It'll be better to write anew. We merged tiles and stuff
10:12:37  <andythenorth> oh
10:12:45  <andythenorth> that's more dull :P
10:12:52  <planetmaker> 20 minutes work or so
10:13:16  <andythenorth> ok
10:13:27  <planetmaker> but just using the older list might introduce bad bugs [TM] due to inconsistencies
10:13:37  <andythenorth> agreed
10:15:01  <planetmaker> <-- there
10:18:54  <planetmaker> andythenorth: grep -Ri "blahblah" sprites/nml/* can be very helpful.
10:19:44  <planetmaker> as can sed -i.bak "s/item_FEAT_INDUSTRYTILES_225, 225/item_FEAT_INDUSTRYTILES_225, ID_TILE_225/g"
10:19:53  <planetmaker> (adopted for each tileID
10:19:55  <planetmaker> )
10:34:06  <andythenorth> :)
10:35:39  <Brot6> FIRS Industry Replacement Set - Bug #3428: Metal Fab. Plan and Metal Workshop incorrectly using s... (andythenorth) @
10:36:45  <andythenorth> o/t - how does nml ensure consistency of vehicle IDs across different revisions, compile environments etc?
10:41:11  <planetmaker> you either declare them explicitly - like the tileIDs here. Or you ensure that you don't change the order of vehicles
10:41:43  <planetmaker> i.e. only add new vehicles at the end of the list. That might be much easier. Oh, and of course, don't delete vehicles
10:41:54  <planetmaker> Just make them unavailable ;-)
10:42:26  <planetmaker> it sequentially numbers all features, if no explicit IDs are given. For industries, for tiles, for houses, for vehicles
10:42:33  <planetmaker> and whatever has IDs
10:43:04  <planetmaker> that's actually the much easier approach
10:43:26  <planetmaker> no worry about IDs. Just use the names.
10:44:13  <andythenorth> ok, so I have to be sure not to change order
10:44:15  <andythenorth> hmm
10:44:23  <andythenorth> maybe I'll use IDs :)
10:44:32  <planetmaker> yes. But one could introduce probably also a define-id-file like what we have now instead of the defines...
10:44:52  <planetmaker> item (FEAT_INDUSTRYTILE, claypit_1, 188) { }
10:44:54  <andythenorth> for BANDIT, I have to add trailers at the end of the list, which means new trucks have to be inserted before the trailers....
10:44:59  <planetmaker> not sure whether such empty definition works
10:45:23  <planetmaker> why does that need to be sorted like that, andythenorth?
10:45:55  <planetmaker> why do you need to have first trucks, then trailers?
10:46:10  <planetmaker> why not truck and trailer. truck2 and trailer 3 and 4, ...
10:46:15  <planetmaker> do you care?
10:46:17  <planetmaker> why?
10:47:30  <andythenorth> trailers don't seem to be available unless included after trucks
10:47:34  <andythenorth> although that might be bad science
10:47:45  <andythenorth> (trailers are shared between multiple trucks)
10:47:54  <andythenorth> I could test some more...
10:48:21  <planetmaker> I don't quite understand
10:48:33  <planetmaker> bandit, right?
10:48:48  <andythenorth> yup
10:48:58  <andythenorth> let me verify it's a real case....
10:52:01  <andythenorth> it's a bogus case
10:54:37  <andythenorth> hmm
10:54:44  <andythenorth> also, I should set grf v8 for BANDIT?
10:55:24  <Brot6> BANDIT - Revision 48:695af4ba1a9a: Codechange: remove an incorrect comment about order of includes (andythenorth) @
11:08:45  <planetmaker> andythenorth: if you use NML, you've no choice to select a specific newgrf version
11:09:01  <andythenorth> k
11:09:12  <planetmaker> you'll get grf v8 automatically. Unless you force NML 0.2.x. Which won't give you the improved articulated vehicle CBs and stuff
11:15:16  <Rubidium> bah bah bah... I'm getting "spam" from devzone@
11:15:49  <Rubidium> or rather, someone seems to have spammed the (I guess ALL) news messages
11:17:52  <andythenorth> planetmaker: planning to do the FIRS ID defines?  Or I can start now
11:18:19  <planetmaker> uh, Rubidium?
11:18:40  <planetmaker> oh :-(
11:20:06  <andythenorth> hmm
11:20:06  <andythenorth> pdq2s-macbook-3:firs_build andy$ sed -i.bak "s/item_FEAT_INDUSTRYTILES_225, 225/item_FEAT_INDUSTRYTILES_225, ID_TILE_225/g"
11:20:06  <andythenorth> sed: -i may not be used with stdin
11:21:14  <planetmaker> yes, I forgot that you must specify the files to work on
11:21:28  <planetmaker> like add sprites/nml/*.pnml sprites/nml/industries/*.pnml
11:21:44  * andythenorth wonders if this can be scripted further...
11:21:45  <andythenorth> meh
11:21:52  <andythenorth> I'll just get on with the sed
11:27:11  *** JVassie_ has joined #openttdcoop.devzone
11:27:20  *** KenjiE20 has joined #openttdcoop.devzone
11:28:36  <planetmaker> it can be scripted further. But is probably not worth the effort
11:28:48  <planetmaker> you can supply a file with the replacement rules via -f filename
11:33:32  * andythenorth must first learn about sed ;)
11:34:36  <andythenorth> hmm
11:34:45  <andythenorth> so that's a multi-file find and replace
11:40:58  <planetmaker> yup
11:41:31  <Rubidium> OpenGFX 0.4.0 is NOT pre 1.2.0 compatible
11:42:03  <planetmaker> uhm... that's a mistake then. How does it show?
11:42:04  <Rubidium> is loads monospace sprites for older versions, which will crash because of that
11:43:19  <Rubidium> possibly around nfo line 801
11:45:31  <planetmaker> so... no monospace font glyph must be defined in pre 1.2?
11:45:41  <Rubidium> exactly
11:46:13  <Rubidium> it does a nice write beyond buffer
11:46:22  <planetmaker> meh
11:49:02  <andythenorth> hmm
11:50:46  <andythenorth> planetmaker: will you be annoyed if I just use my text editor to do the replace?  Somehow I always get stuck with sed :P
11:51:40  <planetmaker> why would I mind? Results are what count, not the method employed
11:51:53  <Rubidium> just quickly write something in C ;)
11:52:27  <Rubidium> planetmaker: how long do you think is needed to fix opengfx (and release 0.4.1)? Or should we, for now, change the min version?
11:52:58  <planetmaker> I'm right on it now
11:53:11  <planetmaker> thus an hour or so
11:59:47  <planetmaker> I really wonder though why that didn't happen during my testing; I'm pretty sure I tested 1.1.4, too
12:04:32  <Rubidium> there's a quite reasonable explanation for that
12:04:48  <Rubidium> - Fix: [NewGRF] Prevent against writing data for unknown fonts (r23283)
12:05:11  <planetmaker> :-)
12:05:29  <planetmaker> ok, that explains it. I didn't test 0.7. As I probably should have done
12:17:41  <planetmaker> <-- Rubidium, can you confirm that that version is safe?
12:21:25  <Rubidium> seems to not crash 1.1.3
12:21:58  <planetmaker> good, works for 1.1.0 for me, too. Then I'll ship it as 0.4.1
12:22:58  <Brot6> FIRS Industry Replacement Set - Code Review #3430 (New): 'legacy' tiles in basetiles.pnml (andythenorth) @
12:23:03  <Rubidium> I'm trying 0.7.0 but ...
12:23:06  <Rubidium> no compily
12:24:46  <Rubidium> damn... 0.7.0 has many warnings
12:26:30  <Rubidium> but it seems to work there as well
12:38:08  <Brot6> OpenGFX - Revision 895:a963149f8f89: Fix: Monospace fonts must not be made available in OpenTTD p... (planetmaker) @
12:38:08  <Brot6> OpenGFX - Revision 896:c3712b941001: Backport r892, r894 (issue #3423): (planetmaker) @
12:38:08  <Brot6> OpenGFX - Revision 897:2a97002408ca: Doc: Update changelog (planetmaker) @
12:42:39  <andythenorth> meh
12:43:31  <andythenorth> I somehow broke the dairy farm
12:55:31  <andythenorth> or maybe it's been broken for a while
12:57:18  <Brot6> FIRS Industry Replacement Set - Bug #3428 (Closed): Metal Fab. Plan and Metal Workshop incorrectl... (andythenorth) @
12:57:18  <Brot6> FIRS Industry Replacement Set - Revision 2670:960891129acf: Codechange: make ID string for Metal ... (andythenorth) @
12:57:18  <Brot6> FIRS Industry Replacement Set - Revision 2671:d40e5dfd5399: Codechange: use defines for tile IDs;... (andythenorth) @
12:57:19  <Brot6> FIRS Industry Replacement Set - Bug #3428 (Closed): Metal Fab. Plan and Metal Workshop incorrectl... (andythenorth) @
12:59:39  <Brot6> FIRS Industry Replacement Set - Revision 2672:fa9732df0583: Fix: Metal Workshop didn't have Chemi... (andythenorth) @
13:00:24  <Brot6> FIRS Industry Replacement Set - Bug #3431 (New): Dairy Farm showing ground sprites only (andythenorth) @
13:02:28  <Brot6> NewGRF Meta Language - Revision 1784:5e28eb09e258: Fix: each action4 is limited to 255 strings. W... (yexo) @
13:02:49  <Yexo> andythenorth: has the firs problem with IDs been solved or do you still need my help?
13:03:14  <planetmaker> iirc it's solved. It was explicit tileIDs given
13:07:03  *** andythenorth has quit IRC
13:12:13  *** andythenorth has joined #openttdcoop.devzone
13:12:27  <Brot6> OpenGFX - Revision 898:79426c38bb17: Added tag 0.4.1 for changeset 8a4374637229 (planetmaker) @
13:14:31  <andythenorth> meh.  late breaking bugs :/
13:14:37  <andythenorth> #3431
13:14:38  <Brot6> andythenorth: #3431 is "FIRS Industry Replacement Set - Bug #3431: Dairy Farm showing ground sprites only - #openttdcoop Development Zone"
13:15:06  <Brot6> FIRS Industry Replacement Set - Bug #3431 (New): Dairy Farm showing ground sprites only (andythenorth) @
13:16:53  <Brot6> OpenGFX - Revision 899:4bc37de07e5e: Merge: Changes to changelog from 0.4.1 (planetmaker) @
13:21:59  <Brot6> GRFCodec - Revision 851:63dfdd7158e6: -Fix: GRF container documentation was wrong for special spr... (michi_cc) @
13:21:59  <Brot6> GRFCodec - Revision 852:dcd44557bcdf: -Add: Support for aircraft property 1F (michi_cc) @
13:23:59  <Rubidium> bad indentation there :(
13:32:57  <Brot6> GRFCodec - Revision 853:f4d5463cfe6a: -Fix: Bad indentation (michi_cc) @
13:34:03  <michi_cc> Yexo: Are you sure the comments about the new refit properties (rev 845) in nforenum's data tables are right? The comment about SUBDATA takes about a LIFO order and not about order as defined.
13:35:53  <Yexo> if it's wrong the code for stations (feature 04) is also wrong
13:36:25  <michi_cc> It is entirely possible the comment at the #define SUBDATA is wrong instead :)
13:43:29  <Yexo> I can't find anything in act0.cpp that points to LIFO, it seems FIFO
13:51:17  <Brot6> opengfx: update from 0.3.7 to 0.4.1 done -
13:54:04  <planetmaker> Rubidium: ^ uploading now
13:56:15  <andythenorth> can anyone replicate #3431 ?
13:56:15  <Brot6> andythenorth: #3431 is "FIRS Industry Replacement Set - Bug #3431: Dairy Farm showing ground sprites only - #openttdcoop Development Zone"
13:56:27  * andythenorth wonders if it's due to a change in NML
14:09:08  <Yexo> I'll test it now
14:11:08  <andythenorth> I've tried bisecting to see if it's an ottd or FIRS problem
14:11:23  <andythenorth> didn't get any useful evidence :P
14:11:36  <Yexo> I don't remember recent changes to industry code in openttd
14:11:40  <Yexo> so I suspect FIRS
14:15:25  <Brot6> Dutch Road Furniture - Revision 78:2d93561389d6: Feature: guardrails for straight medians (issue ... (foobar) @
14:17:54  <Yexo> might be due to a change in nml
14:18:14  <andythenorth> I wondered if it was snowline or such
14:18:21  <Yexo> no, it's not
14:19:05  <Yexo> it's a bug in firs
14:19:10  <andythenorth> \o/
14:19:16  <Yexo> CONDITIONAL_BUILDING_SPRITE has comment: * condition            The building will only be drawn if this condition is true.
14:19:29  <Yexo> CONDITIONAL_BUILDING_SPRITE(THIS_ID(spriteset_house), 24, 0) <- from dairy farm, last parameter is "condition"
14:19:38  <Yexo> 0 = false, so the house is not drawn
14:19:53  <andythenorth> I wouldn't have found that :P
14:21:06  <Yexo> I'll test and push a fix it that was the problem
14:21:35  <andythenorth> k
14:21:36  <andythenorth> thanks
14:24:10  <Brot6> FIRS Industry Replacement Set - Revision 2673:fee22ad464dc: Fix: all dairy farm buildings were un... (yexo) @
14:26:19  <andythenorth> Yexo: fixes the issue
14:26:53  <andythenorth> :)
14:26:56  <Brot6> FIRS Industry Replacement Set - Bug #3431 (Closed): Dairy Farm showing ground sprites only (andythenorth) @
14:26:56  <Brot6> FIRS Industry Replacement Set - Bug #3431 (Closed): Dairy Farm showing ground sprites only (andythenorth) @
14:27:37  <andythenorth> planetmaker: only cargo class issues remaining for 0.7.0 :)
14:27:37  <andythenorth>
14:27:52  * andythenorth is going to toysrus now ;)
14:28:01  <andythenorth> bbl
14:28:19  *** andythenorth has quit IRC
14:33:48  <Brot6> Dutch Road Furniture - Revision 79:9b0d02b1acac: Change: denote authors in graphics files (foobar) @
15:36:34  <Brot6> Dutch Road Furniture - Revision 80:3e45451a92da: Feature: guardrails for median of matrix signs (... (foobar) @
15:36:34  <Brot6> Dutch Road Furniture - Revision 81:ecce383bd5f4: Feature: guardrails for median of gantry signs (... (foobar) @
17:15:42  <Brot6> grfcodec: update from r850 to r853 done -
17:23:31  *** andythenorth has joined #openttdcoop.devzone
17:25:52  <Brot6> nml: update from r1772 to r1784 done -
18:05:48  *** Lakie has joined #openttdcoop.devzone
18:07:46  <Rubidium> aloha Lakie
18:08:36  <Lakie> Hello Rubidium
18:09:34  <Rubidium> are two of the #2790 patches really enough to get RPN working?
18:09:34  <Brot6> Rubidium: #2790 is "GRFCodec - Feature Request #2790: Renum: Escape RPN expressions. - #openttdcoop Development Zone"
18:13:28  <Lakie> Um, not sure, been a long time since I looked at them
18:14:31  <Lakie> I believe so
18:14:56  <Lakie> The first two should make renum rewrite anythings it finds like \b(1 2 +) to \b3.
18:15:32  <Lakie> Value is returned after rewriting the expression which might not be desirable
18:15:47  <Lakie> (Since I believe it's result is then value checked)
18:16:21  <Lakie> the last merely adds support for writing \wx(15) into \wxF
18:22:41  <Lakie> The only issue I see is the value being tested after it rewriting the expression
18:23:13  <Lakie> \b(255 1 +) would result in the output having \b256 and an error about the value.
18:23:51  <Lakie> Not an issue for simple expressions or the makefile system, but might be annoying for manually run checks and complex RPN expressions
18:27:43  <Lakie> I used to use an object nfo as a test case.
18:28:15  <Rubidium> but it should fail/warn when the result of the RPN is more than 255 in that case, right?
18:28:35  <Lakie> It'll fail but the outputted nfo will still be rewriten to the result
18:29:14  <Lakie> The check is in the function calling this function.
18:29:16  <Rubidium> what *might* be wanted is a command line parameter to instruct nforenum to replace the RPN with a value
18:29:27  <Rubidium> so normally it just checks
18:29:34  <Lakie> Yeah,
18:29:52  <Rubidium> but with that extra parameter it will output the evaluated value which would be needed for grfcodec ofcourse
18:30:26  <Lakie> Ok, I can have a look at doing that after some tea.
18:30:31  <Lakie> bbiab, sorry.
18:30:42  <Rubidium> on the other hand... 95% of the NFO/GRF development I'm seeing are constructing the .nfo already from multiple files... so there you'd always just replace the stuff
18:36:55  *** andythenorth is now known as Guest22209
18:36:55  *** andythenorth has joined #openttdcoop.devzone
19:58:33  *** andythenorth is now known as Guest22225
19:58:34  *** andythenorth has joined #openttdcoop.devzone
19:58:48  *** Guest22225 has quit IRC
20:09:22  <Lakie> In which case, maybe a paramter to disable it rewriting would be better, Rubidium?
20:14:11  *** andythenorth is now known as Guest22227
20:14:11  *** andythenorth has joined #openttdcoop.devzone
20:28:43  *** andythenorth is now known as Guest22228
20:28:43  *** andythenorth has joined #openttdcoop.devzone
20:32:08  *** Guest22228 has quit IRC
20:57:29  <andythenorth> planetmaker: hi hi - care to discuss FIRS a little? :)
21:35:29  <Rubidium> Lakie: fine by me as well; I'm actually not sure how many people will/are going to use it anyway
21:35:48  <Lakie> Probably very few now NML is quite mature
21:36:08  <Rubidium> given more and more seem to migrate to NML, although I'm not certain whether openttd's base GRF will ever be made by NML
21:36:21  <Rubidium> on the other hand... the base GRF probably doesn't need RPN
21:36:33  <andythenorth> Lakie: I won't migrate HEQS to nml
21:36:35  <andythenorth> nor FISH probably
21:36:48  <andythenorth> HEQS might be able to use expressions in templates - maybe
21:37:05  <Lakie> Possibly.
21:37:28  <Lakie> There main use it taking variables defined in a header, doing calculations and returning a value
21:37:36  <Lakie> Their*
21:42:40  <Lakie> Also, why won't you port them, andythenorth?
21:42:48  <andythenorth> I like nfo
21:42:55  <andythenorth> it exercises my brain
21:42:56  <Lakie> I thought that was becoming an almost automated process?
21:42:57  <andythenorth> and patience
21:43:07  <andythenorth> simplicity isn't everything :)
21:43:13  <andythenorth> I (mostly) enjoy coding nfo
21:43:16  <Lakie> Hehhe
21:43:23  <Lakie> Save when there are hard to find bugs?
21:43:27  <andythenorth> yup
21:43:38  <andythenorth> or when I have to figure out masks
21:44:05  <andythenorth> otherwise it's fun
21:44:29  <Lakie> Heh, bit maskes can be interesting.
22:10:47  <Brot6> BANDIT - Feature #3399 (Closed): Consider concatenating identifier for vehicle name string with CPP (andythenorth) @
22:15:21  <Brot6> BANDIT - Feature #3432 (New): Template auto-refit (andythenorth) @
22:37:28  *** andythenorth is now known as Guest22245
22:37:29  *** andythenorth has joined #openttdcoop.devzone
22:37:54  *** Guest22245 has quit IRC
23:01:07  *** andythenorth has quit IRC
23:17:26  *** ODM has quit IRC
23:24:09  *** leroot has joined #openttdcoop.devzone

Powered by YARRSTE version: svn-trunk