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

[mitigated] HDR Screen Dim #88491

Closed
AzureWolf opened this issue Jan 11, 2020 · 37 comments
Closed

[mitigated] HDR Screen Dim #88491

AzureWolf opened this issue Jan 11, 2020 · 37 comments
Assignees
Labels
chromium Issues and items related to Chromium insiders-released Patch has been released in VS Code Insiders upstream-issue-linked This is an upstream issue that has been reported upstream windows VS Code on Windows issues
Milestone

Comments

@AzureWolf
Copy link

  • VSCode Version: 1.42.0-insider (user setup)
  • OS Version: Windows_NT x64 10.0.19041

Steps to Reproduce:

  1. Open VSCode on Win 10 with an HDR screen

Does this issue occur when all extensions are disabled?: Yes

I know this issue was resolved previously (#68069 and #72373). The --force srgb trick still works, but for the past few weeks it was not necessary.

@connor4312
Copy link
Member

@deepak1556 looks like this was fixed originally by an electron update, something regress here?

@AzureWolf
Copy link
Author

AzureWolf commented Jan 17, 2020

Just wanted to update that the latest update still has this issue. --force srgb still works but had to reimplement it.

@deepak1556
Copy link
Contributor

Seems like a regression from upstream chromium, will have to wait for a fix from them. https://bugs.chromium.org/p/chromium/issues/detail?id=1038770

@deepak1556 deepak1556 added chromium Issues and items related to Chromium electron-7-update upstream-issue-linked This is an upstream issue that has been reported upstream windows VS Code on Windows issues confirmed Issue has been confirmed by VS Code Team member labels Jan 17, 2020
@bpasero bpasero added this to the January 2020 milestone Jan 21, 2020
@bpasero
Copy link
Member

bpasero commented Jan 21, 2020

If we cannot get a fix, we should make sure force-color-profile is a supported option for argv.json (https://github.com/microsoft/vscode/blob/3aa5383/src/main.js#L128)

@AzureWolf
Copy link
Author

If we cannot get a fix, we should make sure force-color-profile is a supported option for argv.json (https://github.com/microsoft/vscode/blob/3aa5383/src/main.js#L128)

I second this. I love that you guys are updating and working on this amazing free software, but it's admittedly annoying and inconvenient to have to keep putting this in. At the very least, stop re-creating the shortcut after updates?

@bpasero bpasero self-assigned this Jan 24, 2020
@bpasero bpasero modified the milestones: January 2020, February 2020 Jan 24, 2020
@bpasero
Copy link
Member

bpasero commented Jan 25, 2020

@AzureWolf can we try out something:

I am trying to figure out if the --force-color-profile srgb works when passed in programmatically so that you do not have to specify it each time from the command line.

[1]

{
	"disable-color-correct-rendering": true,
	"force-color-profile": "srgb"
}

@AzureWolf
Copy link
Author

Hey sorry I just saw this. The issue seems to have been resolved. I also tested what you asked @bpasero and it doesn't appear to work (tested by following your instructions, that version is still dark, and running and re-running does not resolve it).

So it seems like it's resolved and hopefully won't be an issue? I don't know what changed.

@deepak1556
Copy link
Contributor

deepak1556 commented Jan 27, 2020

@AzureWolf we reverted to electron 6 in master, so its temporarily fixed in the insiders, but based on the original issue it should reappear once we get electron 7 update again. The chromium issue is still open but I will backport a potential fix the chromium authors mentioned. I can get a custom build some time today or tomorrow, would be great if you can test that.

@AzureWolf
Copy link
Author

Sure happy to help!

@bpasero bpasero removed their assignment Jan 28, 2020
@bpasero
Copy link
Member

bpasero commented Feb 12, 2020

@AzureWolf ok let's try that experiment again if you can. There is a newer build available [1]

  • close any running instance of VSCode
  • download [1]
  • quick open command palette and pick "Configure Runtime Arguments"
  • replace the file with the contents below [2]
  • close
  • start same version again

=> are the colors better?

[1] https://az764295.vo.msecnd.net/insider/a1f569c66f67287c525ee50b5f46ced8a50b4551/VSCode-darwin-insider.zip

[2]

{
	"disable-color-correct-rendering": true,
	"force-color-profile": "srgb"
}

@bpasero
Copy link
Member

bpasero commented Feb 17, 2020

@AzureWolf ping

@bpasero bpasero mentioned this issue Feb 17, 2020
@bpasero bpasero changed the title HDR Screen Dim [mitigated] HDR Screen Dim Feb 21, 2020
@bpasero
Copy link
Member

bpasero commented Feb 21, 2020

I pushed support for configuring the color profile via our runtime arguments file. Steps to enable this:

  • use command palette to find runtime arguments command [1]
  • add "force-color-profile": "srgb" to the end of the file
  • restart

[1]
image

@AzureWolf
Copy link
Author

Hey super sorry. I'll check it out when I get a chance. For now I just removed HDR from my monitor settings so I have to set that up again when I'm not busy. It was a real pain to see a daily loss of brightness at the start of the day and it's been busy. I'll let you know.

Very sorry about that

@AzureWolf
Copy link
Author

AzureWolf commented Feb 23, 2020

Hey just tested this out and it does and doesn't work. So putting "disable-color-correct-rendering" to false fixes the issue. Setting it to true does not resolve the issue. Setting it to true and adding the color profile (as the screenshot you see here) does not fix the issue.

Annotation 2020-02-23 135848

Thank you so much for this! I can finally go back to using HDR on my monitor! I don't know what it is, but it doesn't really make a big difference but all the colors are "softer" or less intense except where appropriate. It's hard to describe, but it makes it easier to work longer, so I'm super grateful. I'll be sure to thank you guys wherever possible. You guys are an amazing team making an exceptional and free product.

EDIT: may be a stupid question, but the other application I'm experiencing this issue on was mpv (the video player). Any chance you know what would be the equivalent setting there?

@bpasero
Copy link
Member

bpasero commented Feb 25, 2020

@AzureWolf thanks for giving this a try. Since you see yellow squigglies in the editor I wonder if you maybe tried with VSCode stable and not insiders? Can you recheck this with our insiders build?

You can give our preview releases a try from: https://code.visualstudio.com/insiders/

You should not see a yellow warning like that which indicates that this property is not being applied.

@AzureWolf
Copy link
Author

I'm using Insiders:
Version: 1.43.0-insider (user setup)
Commit: 07c3397
Date: 2020-02-24T05:33:18.353Z
Electron: 7.1.11
Chrome: 78.0.3904.130
Node.js: 12.8.1
V8: 7.8.279.23-electron.0
OS: Windows_NT x64 10.0.19041

@fidonet
Copy link

fidonet commented Jun 23, 2020

Just to note that this is still an issue in 1.46.1
image

You can see the contrast difference (and the bizarre random white box) between the VS Code window and a normal Windows UI window it's sitting on top of ..

Untitled-4

@deepak1556
Copy link
Contributor

The issue should be fixed in latest insiders https://code.visualstudio.com/insiders/, please verify by removing any workarounds added via argv.json

@deepak1556 deepak1556 added fixed-in-electron-9 and removed confirmed Issue has been confirmed by VS Code Team member labels Jul 28, 2020
@seanthegeek
Copy link

@deepak1556 Verified fixed in Insiders. Thanks!

@phillipo6
Copy link

Still exists for me with the insiders version, as long as i have vs code windowed it works, but as soon as i go to fullscreen the screen dims.

@deepak1556
Copy link
Contributor

deepak1556 commented Aug 4, 2020

Sorry about that, we have reverted the runtime which contains the fix for this iteration. The fix will be available again from next week once we update to Electron 9 in master.

@NikilRagav
Copy link

Hi this is still broken for me
image
Not sure if it can be seen but most of the screen is dim except for a couple of blocks where there is no text

For what it's worth, other electron apps that I use don't have these problems

@NikilRagav
Copy link

I edited my runtime args disable-color-correct-rendering to false and now things work. Granted, I have no idea if it is "correct".

Both of my screens (internal laptop 4k OLED w/ HDR and external 4k LCD no HDR) support DCI P3. Would like to make sure my colors are correct.

@deepak1556
Copy link
Contributor

The upstream fix will be available with upcoming stable release and is already available with current insiders, please check. Thanks!

@UltimaHoarder
Copy link

UltimaHoarder commented Sep 10, 2020

Putting "force-color-profile": "srgb" in argv.json fixed it for me

(Samsung G9 HDR full brightness)

@rtheil
Copy link

rtheil commented Sep 21, 2020

This has finally been fixed in 1.49.1 with Electron 9.2.1

@github-actions github-actions bot locked and limited conversation to collaborators Oct 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
chromium Issues and items related to Chromium insiders-released Patch has been released in VS Code Insiders upstream-issue-linked This is an upstream issue that has been reported upstream windows VS Code on Windows issues
Projects
None yet
Development

No branches or pull requests