ldtk icon indicating copy to clipboard operation
ldtk copied to clipboard

very slow with 380 to 390 rules

Open morganholly opened this issue 3 years ago • 10 comments

if my counting is correct, 351 enabled and 9 disabled on one intgrid layer, and 25 on another intgrid layer. i have a large level for testing all the rules, i had made a small level for seeing it on a realistic map, and i created a large one that i was going to use for procedurally pasting smaller rooms within in godot. over time it has slowed down noticeably and at this point is basically unusable, even with the autotile rendering off. i'm ok with it taking time to preview with the autotiling, but having a way to draw in maps with that turned off that was more performant would be much appreciated. additionally it would have a significant bit of lag when reordering groups.

morganholly avatar Mar 21 '22 21:03 morganholly

i am on 0.9.3 on macos catalina 10.15.7 (19H1715). i do not think it is a hardware issue, this is a 2019 high end macbook pro

morganholly avatar Mar 21 '22 21:03 morganholly

basically i'd like a mode where i draw in the intgrid values and then manually have it autotile it

morganholly avatar Mar 21 '22 21:03 morganholly

Thanks for reporting the problem! I have some reworks to be done on then "rendering" side of the app, specifically with AutoLayer rules, which tend to be super slow when you have many ones.

In 1.0 beta 7, the undo/redo was rewritten completely and is about 50 times faster for complex maps. I've also optimized lots of things in 1.0, so this will probably help. Unfortunately, there's no macOS beta for now, only a Win build.

So it would help me a lot if you could send me a copy of your current LDtk project, if you're ok with that. I won't redistribute anything, it's only to have a good example of the "target".

You can email me: ldtk at deepnight dot net

deepnight avatar Mar 25 '22 13:03 deepnight

ok i sent an email, hopefully that helps you with optimization

morganholly avatar Mar 25 '22 20:03 morganholly

I found some culprit for the rules performance issues. Here is a before/after optimization gif: compare

You should have much better perfs in next update. I will upload a new beta soon for you to test that.

deepnight avatar Mar 28 '22 13:03 deepnight

You can check Beta 8 here: https://ldtk.io/files/beta

Please tell me if things are better for your project.

deepnight avatar Mar 28 '22 14:03 deepnight

Cool I will try this out soon

Thanks, Morgan On Mar 28, 2022, 10:21 AM -0400, Sébastien Bénard @.***>, wrote:

You can check Beta 8 here: https://ldtk.io/files/beta Please tell me if things are better for your project. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.Message ID: @.***>

morganholly avatar Mar 28 '22 17:03 morganholly

@morganholly Is this issue fixed for you?

deepnight avatar Apr 01 '22 10:04 deepnight

So it looks like the beta zip only contained a windows installer. I’m on mac so I unfortunately can’t try out the beta with the current download link

Thanks, Morgan On Apr 3, 2022, 2:28 PM -0400, Morgan Holly @.***>, wrote:

oops i forgot to download and try the beta, i will do that in a moment

Thanks, Morgan On Apr 1, 2022, 6:03 AM -0400, Sébastien Bénard @.***>, wrote:

@morganholly Is this issue fixed for you? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

morganholly avatar Apr 03 '22 18:04 morganholly

So it looks like the beta zip only contained a windows installer. I’m on mac so I unfortunately can’t try out the beta with the current download link Thanks, Morgan

The 1.1 update is out, with macOS support :) You can download it here: https://ldtk.io

deepnight avatar Apr 03 '22 19:04 deepnight

oops i forgot to download and try the beta, i will do that in a moment

Thanks, Morgan On Apr 1, 2022, 6:03 AM -0400, Sébastien Bénard @.***>, wrote:

@morganholly Is this issue fixed for you? — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

morganholly avatar Oct 11 '22 07:10 morganholly

Yes, this is much much better, thank you!

Thanks, Morgan On Apr 3, 2022, 3:37 PM -0400, Sébastien Bénard @.***>, wrote:

So it looks like the beta zip only contained a windows installer. I’m on mac so I unfortunately can’t try out the beta with the current download link Thanks, Morgan The 1.1 update is out, with macOS support :) You can download it here: https://ldtk.io — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you were mentioned.Message ID: @.***>

morganholly avatar Oct 11 '22 08:10 morganholly

Related #8

deepnight avatar Dec 22 '22 15:12 deepnight

Related #984

deepnight avatar Dec 29 '23 17:12 deepnight

Things should now run smoother, closing this for now

deepnight avatar Dec 29 '23 17:12 deepnight