search

cimgui / cimguizmo

This is a c-api wrapper for ImGuizmo
MIT License
20 stars 15 forks source link

No content of GraphEditor.h and another headers #2

Open gucio321 opened 1 month ago

gucio321 commented 1 month ago

content from here is not generated: https://github.com/CedricGuillemet/ImGuizmo/blob/822be7b44c37dbe98d328739ebe0d5a1ea87ecfc/GraphEditor.h

sonoro1234 commented 1 month ago

I gave GraphEditor a try running https://github.com/CedricGuillemet/ImGuizmo/blob/822be7b44c37dbe98d328739ebe0d5a1ea87ecfc/bin/ImGuizmoSample.exe but didnt like it. Have you tried?

in cimgui there are cimnodes and cimnodes_r

You can see examples in https://github.com/ocornut/imgui/issues/306#issuecomment-720544252

gucio321 commented 1 month ago

Have you tried?

generally I tried to add it as a header in generator.lua but got issue about #incldue <C++Library>. Maybe generator should use g++ to analyze code? Also, ImGuizmo has more headers e.g. ImCurveEdit and ImGradient. would be nice to have these functions wrapped...

sonoro1234 commented 1 month ago

I mean trying https://github.com/CedricGuillemet/ImGuizmo/blob/822be7b44c37dbe98d328739ebe0d5a1ea87ecfc/bin/ImGuizmoSample.exe just to see how the widgets work so that you know which ones you like and which ones not.

After that it is also useful to read https://github.com/CedricGuillemet/ImGuizmo/blob/822be7b44c37dbe98d328739ebe0d5a1ea87ecfc/example/main.cpp to see if the api is good for your needs.

gucio321 commented 1 month ago

https://github.com/CedricGuillemet/ImGuizmo/blob/822be7b44c37dbe98d328739ebe0d5a1ea87ecfc/bin/ImGuizmoSample.exe just to see how the widgets work so that you know which ones you like and which ones not.

To be honest I didn't try that as I'm on linux and .exe doesn't sound like something I'd like to use :smile:

fortunately that worked with wine for me :smile:

After that it is also useful to read https://github.com/CedricGuillemet/ImGuizmo/blob/822be7b44c37dbe98d328739ebe0d5a1ea87ecfc/example/main.cpp to see if the api is good for your needs.

In perspective of cimgui-go I didn't consider what I would like to use but what users would ever want to use and in my view, the more functions is added, the wider use-cases may be satisfied.

sonoro1234 commented 1 month ago

in cimgui there are cimnodes and cimnodes_r

Did you compare?

sonoro1234 commented 1 month ago

Uncomenting lines 152 and 153 on generator.lua you will get GraphEditor.h parsed. It includes vector which is a C++ header and needs g++ instead of gcc.

Compared with cimnodes and cimnodes_r I dont see it as a convinient addition

Also, virtual members from Delegate are not appropiate for C binding, function callbacks would be fine instead.