Skip to content

Vulkan coordinate System - #3592

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

Wunka wants to merge 20 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

Comment thread assets/cubyz/shaders/bloom/color_extractor_downsample.vert Outdated
#ifdef OPEN_GL
texCoords = vec2(inTexCoords.x, 1 - inTexCoords.y);
#else
texCoords = inTexCoords;

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why are the vulkan and OpenGL versions differently here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this may actully be a general thing... I have removed the if. We will see how the vulkan code behaves once we are there

Comment thread assets/cubyz/shaders/ui/button.frag Outdated

This branch has not been deployed

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

Labels

None yet

Projects

Status: In review

Development

Successfully merging this pull request may close these issues.

3 participants