frosch wrote:

Adding a flag for also drawing fences on snow is fine.
But the snowy fence should not use new offsets. The GRF can check itself for snow and various other things.

This comment was imported from FlySpray: https://bugs.openttd.org/task/6315#comment13956

George wrote:

Do you mean graphics? Snowed graphics may be, for example, 1 px higher.
Or what offsets did you mean?
https://dev.openttdcoop.org/projects/sprite-tests/repository/revisions/5962aed2dc5e

This comment was imported from FlySpray: https://bugs.openttd.org/task/6315#comment13957

George wrote:

How do you intend a GRF should provide different graphics it snow?

This comment was imported from FlySpray: https://bugs.openttd.org/task/6315#comment13958

pi1985 wrote:

You maybe mean that this sprites should be in different sprites group (RTSG_FENCES_FRONT, RTSG_FENCES_BACK for example) with same indexes?
And GRF should detect snowed tile and return different base image (if this is possible at all)?

This comment was imported from FlySpray: https://bugs.openttd.org/task/6315#comment13959

frosch wrote:

Railtype GRFs draw snowy tracks, underlays and depots using the "terrain_type" variable. For example swedish rails: https://dev.openttdcoop.org/projects/swedishrails/repository/entry/src/railtypes.pnml# L19

Drawing fences should be no different, so there is no need for RFO_XXX_SNOW_YYY. It only needs a flag for considering drawing them in the first place.

I notice, the patch also adds another thing, which you call XXX_TWO_YYY. This looks like you add fences for more orientations. Enabling this could also be done by extending GetCustomRailSprite to return the number of sprites in the set (group->GetNumResults). Compare it with vehicles, which can also provide either 4 or 8 sprites. It does not need a flag for enabling. (Though if it would need one, it should be separate from "draw on snow".)

This comment was imported from FlySpray: https://bugs.openttd.org/task/6315#comment13960

George wrote:

I do not see tile height variable here http://newgrf-specs.tt-wiki.net/wiki/NML:Railtypes# Railtype_variables, like nearby_tile_height for houses

This comment was imported from FlySpray: https://bugs.openttd.org/task/6315#comment13961

George wrote:

Also, I can't understand how can GRF recognize the railway side (XXX_TWO_YYY)
I suppose you could provide some explaining based on https://dev.openttdcoop.org/projects/sprite-tests/repository/revisions/b2a3cc8bfea7 for example
because I used townzone instead of terrain_type and I can't make it work

Attachments

• 2015-06-09 22-41-17 OpenTTD r27291.png (334.38 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/6315#comment13962

planetmaker wrote:

You don't need height information. terrain_type gives you the indication by means of TILETYPE_SNOW on when to draw snow and when not.

This comment was imported from FlySpray: https://bugs.openttd.org/task/6315#comment13964

George wrote:

I need it, because I want to provide several levels of snow - "a little snow", "some snow", "much snow" and "fully snowed"

This comment was imported from FlySpray: https://bugs.openttd.org/task/6315#comment13965

pi1985 wrote:

Patch update.

Attachments

• rail-fences-r27298.diff (11.53 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/6315#comment13966

frosch wrote:

The 16 sprite version has been implemented in r27343.

The fences-on-snow needs more work:
- Currently the code to decide which fences to draw is duplicated between DrawTrackDetails and TileLoop_Track in the patch.
- Fences are placed instantly above snow, but not below snow.
- Fences do not work on half-snow tiles.

Probably this needs reshuffling of the RailGroundType enum. There are 5 different ground types, with up to 4 fence combinations. However, not all combinations are possible, so 4 bits would still suffice:
0 barren, no fence
1 grass, no fence
2 grass, fence 1
3 grass, fence 2
4 grass, fence 1 and 2
5 snow/desert, no fence
6 snow/deset, fence 1
7 snow/deset, fence 2
8 snow/deset, fence 1 and2
9 half-snow, no fence
10 half-snow, fence
11 water, no fence
12 water, fence
13-15 unused

This comment was imported from FlySpray: https://bugs.openttd.org/task/6315#comment14019

pi1985 wrote:

Here is remake fences in snow patch.

Attachments

• rail-fences-in-snow-r27396.patch (20.28 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/6315#comment14068

pi1985 wrote:

Here is remake fences in snow patch.

Attachments

• rail-fences-in-snow-r27396.patch (20.28 KiB)

This comment was imported from FlySpray: https://bugs.openttd.org/task/6315#comment14070

George wrote:

Any progress on the adding the patch to the trunk?

This comment was imported from FlySpray: https://bugs.openttd.org/task/6315#comment14098

Thanks for this. There's been no activity on this for some time, and as it stands, it doesn't look likely that it will go any further. Since OpenTTD moved to GitHub, we use pull requests rather than patches, as they are a much more productive workflow.

I'm planning to close this soon (in 7 days), as we try to keep the issue count low for OpenTTD, it helps us focus on things that are important and fun.

If you would like to continue with this patch, the best way would be to move the patch to your own GitHub fork, update it for the current OpenTTD master, and then create a pull request. For more information, please see our CONTRIBUTING.md.

We are also happy to discuss directly on the issue, or in #openttd irc, including help to get this into a pull request. Thanks for your contribution!

No longer stale, won't close currently.

This issue has been automatically marked as stale because it has not had any activity in the last two months.
If you believe the issue is still relevant, please test on the latest nightly and report back.
It will be closed if no further activity occurs within 7 days.
Thank you for your contributions.