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

Minor Vulkan & DX12 stuff + graphics class and function #23

Merged
merged 7 commits into from
Oct 5, 2023

Conversation

MarkEcza
Copy link
Contributor

Fixes GPU selection when 2 or more GPU's are present and also other minor validation checks.

(PS: If anyone is using an AMD card let me know! Having issue's myself with RX VEGA64)

@github-actions
Copy link

github-actions bot commented Sep 16, 2023

Download the artifacts for this pull request:

@MarkEcza MarkEcza marked this pull request as draft September 20, 2023 12:41
@MarkEcza MarkEcza changed the title Minor Vulkan FIX Vulkan GPU Selection FIX Oct 1, 2023
@MarkEcza
Copy link
Contributor Author

MarkEcza commented Oct 1, 2023

I'm going to split it into 2 pr's since the other one is going to take some time(fixing for AMD) but the current issue is fixed.

@MarkEcza MarkEcza marked this pull request as ready for review October 1, 2023 10:15
@MarkEcza MarkEcza changed the title Vulkan GPU Selection FIX Vulkan & DX12 fix + graphics class and function Oct 1, 2023
@MarkEcza
Copy link
Contributor Author

MarkEcza commented Oct 1, 2023

Changed pr concept a bit also fixed some dx12 stuff

@MarkEcza MarkEcza changed the title Vulkan & DX12 fix + graphics class and function Minor Vulkan & DX12 stuff + graphics class and function Oct 1, 2023
@MarkEcza MarkEcza added the enhancement An enhancement to an already existing feature in HorseMenu label Oct 1, 2023
@DayibBaba DayibBaba merged commit 1b942a0 into YimMenu:master Oct 5, 2023
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement An enhancement to an already existing feature in HorseMenu
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants