Is set_script() fine for a lot of nodes?

WombatTurkey

Hey guys, For my custom chat room node I'm making. I am using

var new = RichTextLabel.new() new.set_script(load(&quot;res://Scripts/TooltipExtendRichTextLabel.gd&quot;))

Inside my tooltipExtendRichTextLabel.gd I have a basic _draw(): func there (to hide the scrollbar) Would this is be okay if I were to say, have 100 or so of these RichTextLabel nodes? Or would there be a better away about doing this? Reason I ask is because each time a player sends a message in chat, this is created. I will have a limit of around 100 or so messages max on screen... so that might help, but just not sure if set_script is the proper way to do this, thank you!

Delirus

For me this method has worked just fine.In my current project I have a lot of dynamically generated buttons, and the same script is applied to all of them upon creation.  I've gotten up to several dozen buttons without any issues.If there is an upper-limit to the number of nodes that a script can be applied to, I've not yet found it.  Should be simple enough to test.Certainly sounds like the best method of going about it to me, so long as you're simplifying your script as you go along.Another method might be to create and dynamically update a dictionary containing one message per entry (deleting the oldest entry whenever a new one is added), and use only a single RichTextLabel to display the contents of the dictionary.This way might conceivably allow some speedup, but could be more unwieldy to work with.

Akien

There's no fundamental issue with using [tt]set_script()[/tt] on a lot of nodes, but it's more the choice to instance a new node each time a player types a message that I found potentially problematic. That might have an unnecessary impact on the performance (not sure how much, RichTextLabel is still relatively simple) when you could probably use one single multiline RichTextLabel and append content to it.If you're keeping the current method, use [tt]preload[/tt] instead of [tt]load[/tt] to avoid reloading the same script resource over and over again; preload happens once and is cached. You could also create a scene with your RichTextLabel + its script, preload it and instance it directly.

WombatTurkey

Thank you Akien and Delirus,Good to know that the preload way is cached.  That eases my mind a bit! I really like Godot and still learning the ropes and gots lot to learn appreciate the info.