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

Re-disable GPU, but only on windows #142

Merged
merged 6 commits into from
Feb 5, 2024
Merged

Re-disable GPU, but only on windows #142

merged 6 commits into from
Feb 5, 2024

Conversation

hadley
Copy link
Member

@hadley hadley commented Feb 2, 2024

No description provided.

@hadley hadley marked this pull request as ready for review February 2, 2024 19:25
#' * [`"--disable-dev-shm-usage"`](https://peter.sh/experiments/chromium-command-line-switches/#disable-dev-shm-usage)
#' * Only added when `CI` system environment variable is set or when inside a docker instance.
#' * \verb{The /dev/shm partition is too small in certain VM environments, causing Chrome to fail or crash}
#' * The `/dev/shm` partition is too small in certain VM environments, causing Chrome to fail or crash.
Copy link
Member Author

Choose a reason for hiding this comment

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

Not sure why these entire sentences were wrapped in \verb

@hadley hadley requested a review from wch February 2, 2024 19:26
@wch
Copy link
Collaborator

wch commented Feb 2, 2024

Why is this change needed?

@hadley
Copy link
Member Author

hadley commented Feb 2, 2024

So the CI checks don't hang forever

@hadley hadley merged commit 3d01fff into main Feb 5, 2024
12 checks passed
@hadley hadley deleted the disable-gpu branch February 5, 2024 18:03
@dmurdoch
Copy link

dmurdoch commented Feb 9, 2024

Is it possible for chromote users to control this choice? I've heard complaints about the speed of snapshots in rgl for a while now. It uses webshot2::webshot(), which uses chromote. There are so many layers that I haven't had a chance to try to figure out why it is so slow, but an emulated GPU could be an explanation.

@hadley
Copy link
Member Author

hadley commented Feb 9, 2024

Maaaaybe, but given that it causes it to hang on GHA, I don't have a lot of confidence that it will generally help.

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.

3 participants