[bug]Abnormal CPU load

Question

[bug]Abnormal CPU load

Closed this issue 5 months ago · 10 comments

What happend?

When I was using the latest version of giu, I found a CPU exception. You can see the difference in CPU load by using programs packaged in examples/widgets and programs compiled directly to run

Code example

main.go

examples/widgets

To Reproduce

Run my demo
will see the crash...

Version

master

OS

windows10

Answer 1 · 2024-07-23T08:43:12.000Z

@lzytaro I just did a quick test on macOS, the CPU occupation is 0% when the gui is idle.

Answer 2 · 2024-07-23T08:56:35.000Z

@lzytaro I just did a quick test on macOS, the CPU occupation is 0% when the gui is idle.

Sorry, my business pc cannot upload the screenshot. Maybe its only happend on a Windows 10 PC？
go version go1.21.12 windows/amd64

Answer 3 · 2024-07-23T10:32:36.000Z

@lzytaro may i ask you to write issue title and description in English pls Thank you!

Also, could you check if example from cimgui-go had the same problem?

Answer 4 · 2024-07-24T07:36:56.000Z

Maybe there is any problem with my computer. I tried the same case today, but it works normally. I'll close this issue, sorry to bother you.

Answer 5 · 2024-08-06T07:49:41.000Z

I used the fps example in the example directory and used the two versions 0.7.0 and 0.8.1 for a long time. The CPU usage of 0.8.1 is significantly higher than that of 0.7.0. Due to the many changes in the two versions, no specific problem points have been identified for the time being.
The CPU usage of 0.7.0 is about 0%, and that of 0.8.1 is 2-5%. It may not be obvious in high-performance CPU scenarios, but it will be obvious when running on low-performance devices.

Answer 6 · 2024-08-06T09:35:33.000Z

The problem here is that the program will fall into a short endless loop while, causing a high CPU load. 0.7.0 used time.Ticker to process the timer in go, so this problem did not occur.

code file:
https://github.com/AllenDang/cimgui-go/blob/main/glfw_backend.cpp

@AllenDang @gucio321

Answer 7 · 2024-08-06T11:18:09.000Z

I created a pr to solve this problem AllenDang/cimgui-go#294

Answer 8 · 2024-08-06T14:18:03.000Z

Ok! well, Generally I'm in the middle of the forest right now but will try to review asap

Answer 9 · 2024-08-06T14:21:54.000Z

Oh i see it's already merged. Could you please create a pr here to upgrade cimgui-go version?

Answer 10 · 2024-08-06T15:03:11.000Z

Oh i see it's already merged. Could you please create a pr here to upgrade cimgui-go version?

OK sure, here is it https://github.com/AllenDang/giu/pull/833