Tilemap collisions not showing with 'visible collision shapes' in 3.3

[chucklepie]

Godot version:
3.3 stable

OS/device including version:
Linux mint mate 19 64 bit

Issue description:
This is Godot 3.2.3 with debug/visible collisions showing all the tiles that have collision shapes, plus the player kinematic body spaceship hiding at the bottom right

This is 3.3. It has forgotten to include tilemap collision shapes. The only collision shape you see is the kinematic body.

It is the same project.

Steps to reproduce:

1. create a tilemap
2. add a tileset with collisions
3. enable visible collisions
4. profit

Minimal reproduction project:
See my comment below with code.

[jusw85]

I cannot reproduce.

3.3, Arch Linux

The tile comes from a tileset with a single tile with collision

[chucklepie]

I'll add as new comment. I didn't before in case it is a known fault.

[SirLich]

I am not experienced this bug, on 3.3. I am on windows.

You could create a demo project for the bug. I think it would be quite fast to setup and would allow others to try and repro.

[chucklepie]

I am not experienced this bug, on 3.3. I am on windows.

You could create a demo project for the bug. I think it would be quite fast to setup and would allow others to try and repro.

I'm doing it now. Be there in a few minutes. I was waiting to know if it was a valid and unknown fault first.

[chucklepie]

Code is attached or clone:
no_shapes.zip

git@gitlab.com:chucklepie-productions/raised_bugs/no_collision_visible_33.git
https://gitlab.com/chucklepie-productions/raised_bugs/no_collision_visible_33.git

For me on 3.3 stable linux mint this is the screen:

On 3.2.3 stable, this is the screen

[Calinou]

You need to enable the property in TileMap to display collision shapes. It's disabled by default because of its performance impact with large TileMaps.

[chucklepie]

What property? And why is it working on 323 and not 33? Is this a change in the changelog wr never noticed? And why are the others posting saying its working for them in 33?

…

On Tue, 11 May 2021, 15:46 Hugo Locurcio, ***@***.***> wrote: You need to enable the property in TileMap to display collision shapes. It's disabled by default because of its performance impact with large TileMaps. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#48631 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADCERRKXM56LEMEYGIO4NGTTNE7M3ANCNFSM44UWWRWQ> .

[Calinou]

And why is it working on 323 and not 33?

The behavior has changed; tilemaps no longer display collision shapes by default to improve performance. This was initially prompted by #34923 because drawing outlines has a performance cost (but it does improve the debug experience a fair bit).

Is this a change in the changelog wr never noticed?

It's at the top of the Added > Core section: https://github.com/godotengine/godot/blob/3.x/CHANGELOG.md#core

Remember to read the changelog on the 3.x branch instead of master, as master does not cover minor releases anymore.

[chucklepie]

Thanks, good to know. So how did it work for the others commenting here if its turned off by default? I'm no designer, but maybe even if others didn't know either, a better way would be to incorporate it somehow into the debug menu? Otherwise it'll have to be done on a per Tilemap basis, especially if there's no default project setting to do this either and should debug flags be part of a node?

…

On Tue, 11 May 2021, 15:53 Hugo Locurcio, ***@***.***> wrote: And why is it working on 323 and not 33? The behavior has changed; tilemaps no longer display collision shapes by default to improve performance. This was initially prompted by #34923 <#34923> because drawing outlines has a performance cost (but it does improve the debug experience a fair bit). Is this a change in the changelog wr never noticed? It's at the top of the *Added > Core* section: https://github.com/godotengine/godot/blob/3.x/CHANGELOG.md#core Remember to read the changelog on the 3.x branch instead of master, as master does not cover minor releases anymore. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#48631 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ADCERRLRX7AJTOW6AFQS4E3TNFAGZANCNFSM44UWWRWQ> .

[SirLich]

Thanks, good to know. So how did it work for the others commenting here if its turned off by default? I'm no designer, but maybe even if others didn't know either, a better way would be to incorporate it somehow into the debug menu? Otherwise it'll have to be done on a per Tilemap basis, especially if there's no default project setting to do this either and should debug flags be part of a node?
…
On Tue, 11 May 2021, 15:53 Hugo Locurcio, @.***> wrote: And why is it working on 323 and not 33? The behavior has changed; tilemaps no longer display collision shapes by default to improve performance. This was initially prompted by #34923 <#34923> because drawing outlines has a performance cost (but it does improve the debug experience a fair bit). Is this a change in the changelog wr never noticed? It's at the top of the Added > Core section: https://github.com/godotengine/godot/blob/3.x/CHANGELOG.md#core Remember to read the changelog on the 3.x branch instead of master, as master does not cover minor releases anymore. — You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub <#48631 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADCERRLRX7AJTOW6AFQS4E3TNFAGZANCNFSM44UWWRWQ .

I am new to Godot. I simply turned on the option in the Tilemap.

I did not realize this was something "special", so I ended up accidentally doing it correctly.

If anything, that's evidence that its not confusing for beginners, at least :)

[chucklepie]

There you go then, us old people don't notice these minor changes ;)

But jokes aside, my point really is it doesn't quite make sense that a debug specific functionality should be part of the node and given this is debug, the 'performance' hit it gives should be understood and default to on not off especially given it has to be done for each tilemap (I have over 20 in my game spread across various levels). However, given tilemaps are being completely redone these flags (things like 'compatibility mode' and this) may just be temporary until 4.

[chucklepie]

@Calinou

Hello, I've just realised why this collision functionality was changed and I'm not sure it's a good thing :(

It now makes collision shapes on tilemaps consistent with all other nodes by making collision shapes visible, which I get. But I think tilemaps are a bit of an exception. You do not want to see collision shapes normally in a map as it makes editing a map impossible.

I say this because the way it's done you have to turn on and off every single tilemap in your game, which there may be dozens and spread across various scene instances, and it goes against how the debug/view collision objects menu work because unless you have it set for tilemaps in the editor you do not see collision boxes at runtime, which is a bit mad and opposite to how every other node's collision visibility works! i.e. you still cannot see tilemap collision shapes unless you have them visible in the editor.

Is this just a temporary thing and won't be around in 4?

Surely a better way is to make this visibility just an editor thing, i.e. it does not affect debug mode....