Skip to content

Dependencies Update #436

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

Open
2 of 5 tasks
AMRD1234 opened this issue Apr 3, 2025 · 3 comments
Open
2 of 5 tasks

Dependencies Update #436

AMRD1234 opened this issue Apr 3, 2025 · 3 comments
Labels
Feature New feature to the engine

Comments

@AMRD1234
Copy link

AMRD1234 commented Apr 3, 2025

Hi, don't be tired, this game engine is really cool! I looked at the Dependencies and noticed that their versions are very old! Especially Imgui!
It doesn't seem that difficult to update the Dependencies.
And many of the Dependencies bugs have been fixed.

  • Imgui Docknig v1.77 ------ Update ------ > v1.92.0
  • GLEW ------ Update ------ > v2.2.0
  • GLFW ------ Update ------ > 3.4
  • Lua ------ Update ------ > 5.4.7
  • Bullet3 ------ Update ------ > 3.2.5

I'm really sorry because I don't know how to work with Git and I can't make changes to the project!
I hope you always shine.

@AMRD1234 AMRD1234 added the Feature New feature to the engine label Apr 3, 2025
@adriengivry
Copy link
Member

Will look into it! I know @litelawliet is our dependency expert 😜

@litelawliet
Copy link
Contributor

litelawliet commented Apr 3, 2025

Indeed most of them shouldn't bring breaking changes, GLEW, GLFW, Lua and Sol. As for Bullet3, I haven't look into it yet.

Now for ImGui, that's where things gets tricky. Besides the new process of handling inputs in ImGui wich will need a pass if we bump (not that hard), Overload has made internal changes to imgui. Sadly even if we redo the changes that were made after a bump, since ImGui has reworked a few pieces of the codebase we won't have the same behavior out of the box inside Overload.
The real work regarding bumping ImGui starts with a refactoring of the input context and awareness of OvPanels. The way we handled mouse inputs was to just get it working at the time and some logic from one panel is somewhat "overlapping" over the logic of another.
So first and foremost, a good refactoring of inputs in Overload is due prior/alongside bumping ImGui. But that's indeed something that will have to be addressed at some point regarding bumping said dependency.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature New feature to the engine
Development

No branches or pull requests

4 participants
@litelawliet @adriengivry @AMRD1234 and others