Skip to content

Conversation

@ananasmoe
Copy link
Member

@ananasmoe ananasmoe commented Jan 2, 2026

image

thought it would be good that every cloudy-org app showed what resources it uses.

ui.seperator() is a bit ass to use in a grid but idk what else i should use

@ananasmoe ananasmoe requested a review from THEGOLDENPRO January 2, 2026 09:05
@THEGOLDENPRO
Copy link
Member

THEGOLDENPRO commented Jan 2, 2026

I did want to add something like this then I found out egui already has it all, so the plan changed to just implement that. Check out the egui demo website and click on the backend button.

So to be completely honest this seems useless compared to what egui already provides built in, with the bonus of not introducing new dependencies and all the other wonder features for debugging it comes with.

https://www.egui.rs/#demo

https://docs.rs/egui/latest/egui/struct.Context.html#method.inspection_ui

Also for future reference do ask or create an issue before going and writing lots of code. 😭

@ananasmoe
Copy link
Member Author

ananasmoe commented Jan 3, 2026

we could add a CollapsingHeader for inspection ui and put inside the ResourceManager widget.

@THEGOLDENPRO
Copy link
Member

I don't think system info like cpu name and usage is relevant to us here. The mean cpu usage egui has in their demo makes more sense. So I don't think we should have the system info section at all unless it's displaying thing like the render (e.g: wgpu, etc) that is being used and GPU name, both of which are already displayed in logs so personally I can't see sufficient demand to introduce a whole dependency just to display GPU name in the gui.

Screenshot_20260103-142850.png

All the memory usage info with a quick look at memory_ui and texture_ui is already displayed nicely. So I insist we just include those UIs and the others too:

  • settings_ui
  • inspection_ui
  • texture_ui
  • memory_ui

We should also include the cpu mean usage from the demo. Renderer and backend could be added if we don't need to introduce an additional dependency, it may be possible to just query egui for this. Will have to look at the demo source code.

@THEGOLDENPRO
Copy link
Member

Also I'm not too sure on having it be in a collapsing header. Windows would be nice but if it looks okay in a collapsing header i don't mind that.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants