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

github: add --gpu-debug to issue templates #11515

Closed
wants to merge 1 commit into from

Conversation

haasn
Copy link
Member

@haasn haasn commented Mar 28, 2023

This is never harmful and always helpful in debugging GPU-related issues.

Read this before you submit this pull request:
https://github.com/mpv-player/mpv/blob/master/DOCS/contribute.md

Reading this link and following the rules will get your pull request reviewed
and merged faster. Nobody wants lazy pull requests.

This is never harmful and always helpful in debugging GPU-related issues.
@kasper93
Copy link
Contributor

kasper93 commented Mar 28, 2023

This is never harmful

Well, it fails to create the d3d11 context when debug layers are not installed. As demonstrated by #11512

We probably should make DXGI_ERROR_SDK_COMPONENT_MISSING non critical error.

Make a log file made with -v -v or --log-file=output.txt, paste it to
https://0x0.st/ or attach it to the github issue, and replace this text with a
link to it.
Make a log file made with --gpu-debug and -v -v or --log-file=output.txt,
Copy link
Member

Choose a reason for hiding this comment

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

So you mean --gpu-debug and (-v -v or --log-file=output.txt) but it doesn't read like if you don't know it already.

How about:

Make a log file with --gpu-debug, and either -v -v or --log-file=output.txt

@sfan5
Copy link
Member

sfan5 commented Aug 8, 2023

ping

@kasper93
Copy link
Contributor

kasper93 commented Sep 1, 2023

It is the same with d3d11 gpu-debug, it also need SDK to be installed.

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

Successfully merging this pull request may close these issues.

None yet

4 participants