DescentDevelopers / Descent3

Descent 3 by Outrage Entertainment
GNU General Public License v3.0
2.73k stars 231 forks source link

Merge piccu renderer #473

Open tophyr opened 6 days ago

tophyr commented 6 days ago

Pull Request Type

Description

Replace renderer/ with the Piccu engine.

Related Issues

Likely massively conflicts with @Lgt2x's #457

Checklist

Lgt2x commented 6 days ago

No way we integrate that in one go. Need to go step by step, understanding changes and their implication. Piccu and this project have massively different goals and standards, if we want to merge we need to go slowly and wisely

tophyr commented 6 days ago

Yeah, I agree. This is mostly a POC. The overall goal, for this project, I think should just be "modernize the renderer" so that it uses GL3+, VBOs, and shaders. Piccu is one way to accomplish that, but certainly not the only one. This commit stack so far isn't even tested at runtime, I just literally got it to build on my macbook and put it up. Digging in to grok the old renderer, piccu renderer, and the differences, will take a chonkin bit more time.

tophyr commented 6 days ago

weird that none of these build failures occur locally for me. hm