Skip to content
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

New presenter + renderer backend #1492

Merged
merged 133 commits into from
Feb 8, 2023
Merged

Conversation

heinezen
Copy link
Member

@heinezen heinezen commented Jan 11, 2023

Rewrite of most of the level 2 rendering backend.

  • Use presenter as main interface for graphical display
  • New GUI code using Qt6 and new renderer
  • Remove dependencies to SDL2 in renderer
  • Game entity rendering
  • Terrain rendering (in 3D!)
  • Camera
  • Simulation clock

@heinezen heinezen added area: renderer Concerns our graphics renderer lang: c++ Done in C++ code big stuff High-impact changes, mainly foundation work labels Jan 11, 2023
@heinezen heinezen force-pushed the new-presenter-real branch 2 times, most recently from 7c32b36 to 1cc49b7 Compare January 11, 2023 18:53
@heinezen heinezen marked this pull request as draft January 12, 2023 01:10
@heinezen heinezen force-pushed the new-presenter-real branch 3 times, most recently from 6b6e5e2 to 84cb113 Compare January 27, 2023 21:30
@heinezen heinezen marked this pull request as ready for review January 27, 2023 21:30
Copy link
Member

@TheJJ TheJJ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

awesome! so many new cool things!

assets/qml/main.qml Outdated Show resolved Hide resolved
doc/code/renderer/README.md Outdated Show resolved Hide resolved
libopenage/console/console.cpp Show resolved Hide resolved
libopenage/console/console.cpp Show resolved Hide resolved
libopenage/assets/assetmanager.cpp Show resolved Hide resolved
libopenage/presenter/presenter.cpp Outdated Show resolved Hide resolved
libopenage/presenter/presenter.cpp Outdated Show resolved Hide resolved
libopenage/presenter/presenter.cpp Show resolved Hide resolved
libopenage/renderer/entity_factory.h Outdated Show resolved Hide resolved
#include "renderer/opengl/geometry.h"
#include "renderer/opengl/lookup.h"
#include "renderer/opengl/shader.h"
#include "renderer/opengl/texture.h"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

isn't that just "context.h" etc, no need for include search path injection.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

indeed, but here it's right in the same folder anyway

@heinezen heinezen force-pushed the new-presenter-real branch 15 times, most recently from e56446e to 7dca92d Compare February 3, 2023 14:05
@heinezen heinezen added the kevin-rebuild-pl0x instruct kevin to rebuild this pull request label Feb 7, 2023
@SFTbot SFTbot removed the kevin-rebuild-pl0x instruct kevin to rebuild this pull request label Feb 7, 2023
@TheJJ TheJJ merged commit b124236 into SFTtech:master Feb 8, 2023
@heinezen heinezen mentioned this pull request Jun 24, 2023
13 tasks
@heinezen heinezen deleted the new-presenter-real branch October 15, 2023 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: renderer Concerns our graphics renderer big stuff High-impact changes, mainly foundation work lang: c++ Done in C++ code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants