-
Notifications
You must be signed in to change notification settings - Fork 164
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
Remedy for faulty monitor which fails to respond its capabilities #270
Comments
operation.log: |
Thanks for the explanation. Generally I do not prefer to add modifications which result in extra function calls even in the case of decent monitors. It is undesirable to make the systems which have only decent monitors to use extra resources for the sake of faulty monitors. Thus, I will not merge your PR #269. In addition, if you wish check the functionality of a monitor without the information on its capabilities, you have to test not only getting brightness but also setting brightness both in high and low levels. It will be cumbersome. Instead, I think if a user is confident that a monitor will work anyway, it will be relatively better to skip testing process alltogether using a white list of such monitors provided by, perhaps, command-line arguments. I am still wondering but this is my initial thoughts. P.S. To obtain meaningul information from operation.log, you need to use this app for a while. |
Is there any command-line can let the software skip the check of the VCP return codes? |
|
The new operation.log is here
|
No, not yet. |
I think it is possible to add an option or command to the hidden menu. If the user thinks his monitor is okay, he can try it. There may be more Monitors with such problems, and the returned VCP data may change frequently, so it may take a lot of effort to design a whitelist. |
In fact, a whitelist by command-line option is the least costly approach. Added preclear option by 6e5d4a7 and 1f0d50a
In case you have multiple monitors, simply add thier Device Instance IDs.
|
Thank you very much! I downloaded the latest code and test the "preclear" commandline. It now works well! |
Thanks for the confirmation. |
Hi there. I've got the same issue like here.
And I did the same action as here
And I got this new log:
And I got nothing in my Monitorian software: @emoacht @CC0126 Do you know why this happens? Thanks a lot. |
@HaiyiMei Could you show probe.log? |
|
@HaiyiMei Thanks for the log.
This option is not for such case. |
Thanks for the reply. |
In case it is any useful information; I have a Samsung T24D390EW connected via HDMI. |
@jacob-pro Thanks for the information. |
@emoacht indeed:
|
As I wrote in another issue, preclude option was added by c9ed609 This option is to intentionally exclude a specfic monitor from those to be handled by this app. The usage is the same as that of preclear option while the effects are opposite. #270 (comment) Use this option as shown below.
|
At first I only used it to control my AOC Q2790PQ, it had no problems, but after I replaced a monitor AOC Q2790PC, the software did not work properly.
After many searches, I found the open source address of your software, downloaded its source code, and studied it.
After I tested DDC-Control, AOC iMenu, FreeMonitorManager, I found the problem.
This new monitor has multiple interfaces (such as HDMI, DP, USB-C), the VCP value of the USB-C interface returns to normal, but HDMI returns garbled characters, which makes the software unable to obtain whether the display actually supports DDC /CI command, which hinders the work of the software.
I can get the actual brightness by judging that the VCP has a response but cannot be resolved. To determine whether the monitor supports DDC/CI non-standard, just like FreeMonitorManager.
Now he works fine on my monitor.
probe.log is here:
The text was updated successfully, but these errors were encountered: