Skip to content

Vulkan coordinate System - #3592

Open
Wunka wants to merge 11 commits into
PixelGuys:masterfrom
Wunka:vulkanCoor
Open

Vulkan coordinate System#3592
Wunka wants to merge 11 commits into
PixelGuys:masterfrom
Wunka:vulkanCoor

Conversation

@Wunka

@Wunka Wunka commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

This is not ready... first every thing has to be tested to still work. The nice thing is, for this can even a normal window be used, without vulkanTestingMode, as this changes also in general how opengl things need to be renderd.

  • projection matrix
  • change shaders where an obvious switch can be made
  • fix clipping ( c.glClipControl(c.GL_UPPER_LEFT, c.GL_ZERO_TO_ONE) does not apply this setting to everything. clipping for example still works on the old coordinate system.)
  • fix wierd image moving behavior (only in vulkan window)
  • fix item drops vanishing (the problem is item_drop.frag, the FragDepth calculation seems to be doing something not correct anymore) (done, by not doing the depth conversion)
  • problems with light sources, can be good seen on torches.
  • fix problems with fog (can be good seen in the rainbow_forest) (done, by not doing the depth conversion)
  • fix reflections
  • fix stars
  • find more bugs and fix them (hopefully there are no more...)
image

@Wunka
Wunka marked this pull request as draft September 9, 2026 18:05
@Crepestrom

Copy link
Copy Markdown
Contributor

good job with all this work

@Wunka
Wunka marked this pull request as ready for review September 10, 2026 15:42
@Wunka Wunka moved this to High Priority in PRs to review Sep 10, 2026
@Wunka

Wunka commented Sep 13, 2026

Copy link
Copy Markdown
Contributor Author

As talked with quantum, while this should be looked at in general to find bugs, it's probably better to only merge this after 0.4.0 release to not get any bugs in the release

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

Labels

None yet

Projects

Status: High Priority

Development

Successfully merging this pull request may close these issues.

2 participants