Skip to content

Conversation

@JamesMillsAIE
Copy link

Updates:

  1. Removed gl_core in favour of GLEW to access newest OpenGL 4.6 features and fix some potential crashes
  2. Updated to latest GLM
  3. Removed OBJ loader in favour of Assimp which is linked correctly to Bootstrap
  4. Bootstrap is now a DLL that automatically gets placed into the build directory when project runs
  5. Created updated project templates which use the new bootstrap system
  6. Updated ImGui to latest version in the docking branch. (see: https://github.com/ocornut/imgui/wiki/Docking) for how to use
  7. Moved header files of Bootstrap into dependencies folder
  8. Cleaned up dependencies folder to split libs and includes into separate folders
  9. Added graphics project (Teacher Version, will be removed before 2024) to demonstrate how to use the Assimp system

Updates:
- Using latest OpenGL through GLEW
- Using latest ImGui
- Using latest stb
- Using latest GLFW
- Using latest GLM
- Reconfigured Bootstrap to be DLL instead of static lib
- Added build events to 2D and 3D projects to copy required DLL's across
- Restructured dependencies folder to separate include and lib files

Removed:
- gl_core_4_4

Added:
- Implemented Assimp to handle model loading (see https://assimp-docs.readthedocs.io/en/latest/about/introduction.html for supported file types)
- Unified naming conventions to AIE standardisation
- Fixed indentation across Bootstrap
- Moved all bootstrap and imgui files into include
- Outputting bootstrap builds to lib folder
- Reading Bootstrap libs from dependencies folder
- Updated 2D and 3D project templates to reflect new project setup
Updates:
- Updated Project 2D and 3D versions to C++ 20
- Fixed possible Null Reference issue with loaded pixels in texture
- Updated templates to use new project configurations
Added:
- Teacher only version of graphics starter project demonstrating how to implement Assimp loading
@JamesMillsAIE
Copy link
Author

@terrehbyte @JJDonlevyEdu if you can review this before next year that would be great. I don't want to do the merge without a review from a few people first.

@terrehbyte
Copy link
Member

I could take a look. A few questions:

  • What's the scope of the review?
  • Do we need to consider how tutorials/exercises may be impacted?
  • Do we care that the teacher version will be retained in the commit history?

@JamesMillsAIE
Copy link
Author

  • Scope: Just to verify that there is nothing that will break content
  • You shouldn't need to change ANYTHING related to content. The only thing that has changed really is the inclusion of the AssImp basic material system
  • We should care because the students can see the repository afaik and the teacher version could essentially be submitted as is and meet the criteria (although we would know because we can see it)

@jaredmulconry
Copy link

For the sake of future maintenance of this project, @JamesMillsAIE, what version of AssImp have you integrated into the project?

@JamesMillsAIE
Copy link
Author

For the sake of future maintenance of this project, @JamesMillsAIE, what version of AssImp have you integrated into the project?

It was so long ago I created this change, that I cannot remember. It was the latest at the time of the update.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants