Calame321 / Godot_Auto_GridMap

Auto tiling for the Godot's GridMap Node
MIT License
53 stars 8 forks source link

Godot4 compatibility (patch) #5

Open aes opened 1 year ago

aes commented 1 year ago

Hi.

This doesn't (quite) play well with Godot 4, I managed to get it working (sort of):

Auto_GridMap.gd.txt

I should point out that this is kind of a dirty hack, updating the sub_gridmap in _process, but it works. Since the sub_gridmaps can be 'stolen' from their parent Auto_Gridmaps, I think this is a win, even if it's not that pretty. CPU usage has been quite manageable so far.

Lmk if you think it's worth working on it, to making a proper PR:s, etc.

pearcemichal commented 9 months ago

Hello! thank you for working on this. I was able to get it working by creating a new scene and auto gridmap setup.

That being said, it appears to not work in the test scene that comes with this project. it says that it's painting, but the grid doesn't update

It DOES work with multiple autogridmaps for the layers, but after adding a second autogridmap I can tell there is a definite performance hit.

If you are able, it would be great if you could get your updates integrated officially.

stesproject commented 5 months ago

This patch works and that's great, but the performance are very poor. Has someone found a better way to update cells? When we paint with the GridMap a log "GridMap Paint" gets printed in the console. If only there was a way to connect to that signal...

rakzin commented 1 month ago

godot crashes..