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

docs: add guidance for setting GPU for encoding in ALVR #2499

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dvtfl
Copy link

@dvtfl dvtfl commented Nov 9, 2024

Relevant issue: #2325
Relevant Discord thread

@The-personified-devil
Copy link
Collaborator

There's multiple other options that should work, and this one combines quite poorly with them, so I'd only like to add this if the other fixes don't work because otherwise I think it'd be a bit confusing having multiple solutions to the same problem. Even if so it needs to be reworded to show that both are potential solutions to the same problem and should both be tried. Plus a very simple guide on how to find the right file for your distro might be nice

@32bitx64bit
Copy link

Seems more confusing than helpful in my eyes.

@Meister1593
Copy link
Collaborator

I think this should be closed if no changes are being done over this and other ppl disagreeing too. What do you think @The-personified-devil ?

@dvtfl
Copy link
Author

dvtfl commented Nov 30, 2024

Apologies for the late response, i was being ill.

There's multiple other options that should work

Could you please name them?

@The-personified-devil
Copy link
Collaborator

Could you please name them?

They're listed on the wiki page already. You could perhaps add this as a sort of addendum along the lines of "if that doesn't work also add this option". But relying on it alone isn't a good idea and the other options should already work if nvidia didn't break/change anything.

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

Successfully merging this pull request may close these issues.

4 participants