-
Notifications
You must be signed in to change notification settings - Fork 16
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
Fix documentation around --enable-graphics flag #224
Fix documentation around --enable-graphics flag #224
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While reading this page, another question came to mind: with the recently added CLI flag --enable-streaming
to amc launch
comman, the webrtc
platform will be used in such cases. Since the webrtc
platform automatically detects the GPU and adjusts rendering or video encoding capabilities (software vs hardware) accordingly, should we mention this on this page as well?
I have added a note because it is worth informing the user that there is another flag ( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Overall reads much better but we should still get rid of the platform usage as IMHO it might add more confusion/involves another advanced topic the user might not care about to know at this point. Wdyt?
A: Yeah, sort of. However, if we want to remove the platform context here, we should avoid mentioning software and hardware rendering or video encoding (which are the focus of the chapter about using the GPU in instances). Instead, we should only refer to the visual output, which could be influenced by the |
We are in a page titled "GPUs and instances" so why shouldn't we talk about hardware rendering and video encoding? Both are essential functionalities we use from a GPU. We need to explain the --enable-graphics switch as even if you assign a GPU to the instance you won't get it being used by Anbox unless you have it set. |
* Change title to reflect current content * Focus on purpose of `--enable-graphics` flag (render visual output) * Fix redirects
@morphis @adglkh I have rearranged the entire topic to focus on graphical rendering based on the I intentionally removed the information about |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@adglkh do you want to take another look? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Fixes #216