-
Notifications
You must be signed in to change notification settings - Fork 56
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ImGui integration is not working as specified in documentation #57
Comments
You need to:
|
Step by step ouhhhhh babbbbyyyy gonna get to you gurrrllllll https://www.youtube.com/watch?v=ay6GjmiJTPM:
|
Hmm. Weird, shouldn't happen. Did you reconfigure/regenerate CMake before hitting rebuild? |
Visual Studio popped a dialog which asked me to reload all. I've checked the additional dependencies:
|
Yeah well that's wrong. You need to hit Configure/Generate from the CMake GUI. Please post here what the log says. |
🤔 right, yes that explains the linker error. Since this didn't happen for anybody before I optimistically assume this is just some internal CMake state getting confused (yes, this happens sometimes, CMake is a complex beast). Can you delete the build directory generated by CMake and do the CMake configuration again from scratch? |
|
Looks like you forgot to point |
I added the entry in the UI and still error: I tried to launch CMake from the command line instead:
Damn when I saved the files from Github it saved them to FindImGui.cmake.txt and FindMagnumIntegration.cmake.txt. Let me rename them! |
Finally :
|
Alright I tried to use the example code from the ImGuiIntegration tutorial at : https://doc.magnum.graphics/magnum/examples-imgui.html But I can't compile because of some RGB value in the code. |
This literal is defined in the Math::Literals namespace, so you need to using namespace Math::Literals; to make use of it. The example code has this line, right? |
Yeah you are right it is inside of the code but I missed it somehow because my code is a bit different from the example. It seems that using namespace can be used everywhere but I find it better to put all in the beginning. |
Here I have another problem which is when I run in Debug and Release mode the app is working well. When I switch to MinSizef I get this error: The translation in English is : The execution of the code can not continue because imgui.dll was not found. Weird the file is there. The CMakeList.txt from the example contains:
Mine are two files: Root
Multiverses App
|
It's complaining about It's also possible that Vcpkg (or the imgui Vcpkg package) works well only with Debug or Release and not MinSizeRel. Not many people use it, so maybe you hit a corner case that's not handled properly in the vcpkg package recipe for imgui. |
I think I found what is the problem, it is in the Solution Configuration :
The last line: and it works. So it seems that at the generation stage CMake is not able to make the difference between Debug and MinSizef and Release |
Does the file at To me this sounds like some strange packaging error (vcpkg to blame, not cmake), but looking at the vcpkg package for imgui, I wasn't able to find anything suspicious. Good to know you found a workaround. |
Anything else to be done here? Otherwise I'll suggest closing this issue. The above is most probably a packaging error in the ImGui vcpkg package, I can't really do anything about that. |
Hello again dear Magnumers,
I tried to use the ImGui implementation of Magnum and Visual Studio exit the compilation with the following output:
My CMakeList.txt file I've modified like in the documentation at : https://doc.magnum.graphics/magnum/namespaceMagnum_1_1ImGuiIntegration.html
Am I missing something?
The text was updated successfully, but these errors were encountered: