Slab icon indicating copy to clipboard operation
Slab copied to clipboard

Wrong "Channel Count" in DrawCommand stats

Open megagrump opened this issue 4 years ago • 5 comments

Looking at issue #112, the "Channel Count" statistics is not accurate. It shows a value of "0" for every layer even though there are obviously more than 0 channels in some layers (otherwise no UI would be visible on screen).

The GetLayerDebugInfo function in Internal/Core/DrawCommands.lua is faulty. It uses the length of the Layer table for statistics, which is always 0 - it's a hash table that has no length.

As I'm trying to optimize the DrawCommands module for performance, I wonder if Channel Count is an important metric at all? I don't even understand what "Channel" even means in this context.

megagrump avatar Dec 21 '21 23:12 megagrump

Hmm right 🤔 i dont even know what channel is for in this context. I think it's just for demoing tree nodes but then again if i remember correctly there are other demos for nested tree nodes.

I'll take a closer look later and see if i can figure out its purpose. If not found, I'll remove it.

flamendless avatar Dec 22 '21 03:12 flamendless

Please leave it alone for the time being, until I'm done working on the DrawCommands module. It may fall all into place if I understand it better, and it may just need a better name.

megagrump avatar Dec 22 '21 07:12 megagrump

@megagrump i got an email notif about the styling issue but i dont see it anywhere. Did you delete it?

flamendless avatar Dec 22 '21 11:12 flamendless

Still cant. I get "no commit found for the ref 111"

flamendless avatar Dec 22 '21 12:12 flamendless

@megagrump yeah i click it and it doesn't redirect anywhere

I also dont see it in your forked repo 🤔

flamendless avatar Dec 22 '21 12:12 flamendless