[bugs] Underground slopes

[gerw]

The new underground slopes are a little bit buggy.

http://www-user.tu-chemnitz.de/~gerw/patches/pics/uslope1.png
You can build a flat route over a slope. Maybe it's due calc_straigt_route also tries the route backwards.

http://www-user.tu-chemnitz.de/~gerw/patches/pics/uslope2.png
You can build a way on top of an existing slope. (Last tile not checked?)

http://www-user.tu-chemnitz.de/~gerw/patches/pics/uslope3.png
Directly on top of a slope can be a flat tile.

http://www-user.tu-chemnitz.de/~gerw/patches/pics/uslope4.png
By combining the above bugs, a train can drive trough the floor...

In my opinion, building the slopes with the standard slope tools isn't very intuitive.

[Dwachs]

should be fixed in 2546.

[prissi]

Suddenly I cannot click on tiles below sea level. This needs to be changed. Thus I cannot built deeper than the grundwasser. Thus tunnel below the sea are not possible any more.

[gerw]

Directly on top of a slope can be a flat tile.

Isn't fixed. You can lower the tile after you built the slope.

Edit: Another (new) bug:
You can connect through a hang:
http://www-user.tu-chemnitz.de/~gerw/patches/pics/uslope5.png

Edit: Another (new) bug:
You can't built trams in street tunnels.

[Dwachs]

Edit: Another (new) bug:
You can't built trams in street tunnels.

do you have an idea, which change introduces this?

[gerw]

After a short look, this should be due to "bd_von->get_weg_nr(0)->get_waytype() == besch->get_wtyp()".

Another bug: You can't build tunnels on top of other, flat tunnels. It's line 1336: It should also be possible if bd_von->get_weg_hang() == flach. (and also the same for the last tile).

Imho, the concept of the tunnel_ceilings would avoid many checks at different places...

[Dwachs]

Edit: Another (new) bug:
You can connect through a hang:
http://www-user.tu-chemnitz.de/~gerw/patches/pics/uslope5.png

should be fixed in 2551

[Dwachs]

Can you check if these tram tunnels work correct directly after introducing sliced underground view?

Since the tram-builder does not call intern_calc_straight_route, which is only called for tunnels and if one uses ctrl+click for ways, I suspect this bug is around some more revisions.

[gerw]

Should one be able to build tram on road tunnels with the normal way tool or with the tunnel tool?

Edit:
Back routes should be disabled for tunnels. Otherwise you will get strange results (or check, if the starting tile mathes):
http://www-user.tu-chemnitz.de/~gerw/patches/pics/uslope6.png

[Dwachs]

rev. number?

tunnels on flat tunnels should be fixed, all-down-slopes on tunnel slopes tool in 2552.

However, terraforming with raise and lower (keys u and d) is not fixed.

[gerw]

rev. number?

This bug (uslope6) is also present in newest rev.

tunnels on flat tunnels should be fixed, all-down-slopes on tunnel slopes tool in 2552.

However, terraforming with raise and lower (keys u and d) is not fixed.

How about the tunnel_ceiling? With it, you will fix all those bugs with one 'klappe'.

Edit: Another bug, which would be easily fixed with the tunnel_ceiling (without any further code): 3d switches

http://www-user.tu-chemnitz.de/~gerw/patches/pics/uslope7.png

[Dwachs]

Rev 2553 is my last commit on this subject.

And yes I feel an additional tunnel-ceiling ground object would have saved me some headaches.

[knightly]

In my opinion, building the slopes with the standard slope tools isn't very intuitive.

I strongly agree with Gerw on this. It would be clearer to players to have separate underground slope tools, than to add such functionality to the standard slope tools.

[gerw]

Another bug: You can lower a way to the see.


And in the last rev (r2562) it isn't possible to build a way over an up slope.

[prissi]

Code Select Expand

And in the last rev (r2562) it isn't possible to build a way over an up slope.

Sorry, do you mean underground or overground?

[gerw]

Sorry, do you mean underground or overground?

I mean underground. If you build an up slope, you can't extend the way on the upper level by just building over the up slope. It works fine with down slopes.

[prissi]

This is intended, as you could but a three way switch. Os am I misunderstanding you?

EDIT: Ok, understood and fixed this.

[gerw]

I don't know what you mean, but I think, you misunderstood me, too
The upper image shows what I'm doing and the lower one, what I expect, but what doesn't happen. More precise: nothing happens at all, if I click there.


Another bug: If I lower a canal, it doesn't get connected to the sea, which is the standard behaviour of canals.

[z9999]

Why lowered should be connected automatically ?
Lowered is lowered, you need to connect manually, don't you ?

He may want to make a brigde.

[z9999]

Problem in r2564.

[gerw]

Why lowered should be connected automatically ?
Lowered is lowered, you need to connect manually, don't you ?

He may want to make a brigde.

That's true, but this wasn't possible with the old code. If you build a canal onto a down slope, it's get connected. So this two behaviours should be unified.

Edit:
Another bug: you can connect to other players rails in underground. This is really a 'hidden' feature

@prissi: What do you think of the tunnel_ceiling thing? In my opinion, this could save many checks - and frustrated programmers

[prissi]

What of these stuff could be solved by tunnel ceiling? You could built ramps through bridges also with the old code overground, thus those checks for ground above/below are needed. The same stuff like you shown with the curve would be also not solved by a ceiling.

[gerw]

What of these stuff could be solved by tunnel ceiling?

Imho, the main point is lowering land above a tunnel slope. It also prevents that a way can be build on top of a slope. Of course, this can be done also by just checking, if there is a ground one height deeper. Do it as you want.

You could built ramps through bridges also with the old code overground

But only, if you build a bridge under an over. Imho, this should be forbidden anyway.

[Dwachs]

which of these bugs are still in present in the most recent revision?

[gerw]

which of these bugs are still in present in the most recent revision?

The 'Lower track to sea', the one with the canal, no trams in street tunnels, lower land above underground slope.

Maybe I forget one, I don't have a overview anymore.

[Dwachs]

The 'Lower track to sea',  lower land above underground slope.

should be fixed in 2575

the one with the canal

is intended bugehavior ?

[Dwachs]

Here is a patch [incorporated in 2601] that allows
-- building tram tracks in road tunnels with the normal (non-tunnel) tram track tool
-- deleting of tram tracks on road tunnels, which is also broken. But I am not sure whether this change in grund.cc does not have unwanted consequences.

Moreover, the tram-track wayremover does not work. Would it be possible to internally treat the tram-track-remover as track remover but change only the tooltip?

[ASV62]

I found the texture under sea level missing though trains can go through the sea

●the entrances are at ground level.

●it went down 2 levels to the level under sea surface.
I was using r2576 + pak128 1.4.5

[Dwachs]

this is known. You can however use the sliced underground view, then you can see your undersea tunnels, if you change the underground level to undersea levels.