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

Merge ref_vk into main. #1019

Closed
Rykita opened this issue Jun 2, 2023 · 3 comments
Closed

Merge ref_vk into main. #1019

Rykita opened this issue Jun 2, 2023 · 3 comments
Labels

Comments

@Rykita
Copy link
Contributor

Rykita commented Jun 2, 2023

As the title suggests, I've been left to wonder why is the Vulkan renderer a separate repo? It's an outlier in this regard as GL1.4, GLES, GL3.2, Software are all in this repo. Is it due to the experimental nature of the VK renderer?

@protocultor
Copy link
Contributor

Read the whole history of the VK renderer here:
#505 (comment)

@0lvin
Copy link
Contributor

0lvin commented Jun 2, 2023

We are always appreciate contributions to ref_vk and when code will have same level of code quality and features set as gl3, it could be merged back to main repository or remove experimental flag at least.

As example of feature set difference:

@Yamagi
Copy link
Member

Yamagi commented Jun 5, 2023

The biggest problem is that Vulkans requirements are incompatible with Quake IIs architecture. For example Vulkan can abort a frame after each and every API call, Quake II only has very limited handling for aborted frames. While Vulkan may work well enough for the average use case, there are a lot of corner cases caused by these architectural limitations we weren't able to solve with the time we could invest into it. Since GL3 is is arguably a better renderer feature wise we have given up on Vulkan and moved it into an external repository for user who can't run GL3 but Vulkan.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants