-
-
Notifications
You must be signed in to change notification settings - Fork 187
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
Loss of image quality on v2.7.0 vs 2.6.0 #240
Comments
@master255 There's no change between 2.7.0 and 2.7.2 affecting image quality or the black bar issue you're seeing (which is in fact #145, so let's track there :-) ). |
@bk138 You closed the bug too quickly. |
@bk138 Re-open the bug |
@master255 here are the changes between 2.6.0 and 2.7.2: v2.6.0...v2.7.2 Are you able to build from source and do a bisect maybe? If not, having a look at each change would help. The only thing I can imagine causing image quality loss is 3da1dda but that was already on 2.4.0. |
@bk138 Too much unfamiliar code. I can test apk files if you provide them and collect logcat logs. I've downloaded the project now and I'm looking at the code. But the project does not compile because of errors in the code. Upd: I was able to compile and run the application. But noVnc doesn't work. |
@bk138 How can this be fixed? |
@master255 can you compile a list of steps for me to reproduce? With versions used, settings used, outcome, as specific as possible? I can then have a look myself. |
@bk138 Just all defaults and image quality to max. |
You mean server side scaling? Which client was used? Which server version worked, which did not? |
@bk138 1. Yes. |
@master255 I now tried
I get the same image quality with both, a black bar on the right is present in both. In short, can't reproduce the issue :-/ You might want to check your browser's local storage for noVNC quality settings (if none there, it's using defaults), or try with another browser. Another thing that comes to mind is trying on a different network. |
@bk138 Android 8.0? What's the device? |
Android 12, some Samsung. |
@bk138 So I'm testing on Android 8.0. You should test on that version too. Or your app is only for Android 12+? |
@bk138 On my Galaxy S22 Ultra Android 12 on version 2.7.2, I can't see the side line. But the image quality is also very poor, even at the highest setting. |
@bk138 And I'm sure on version 2.6.0 the picture quality will be fine. |
@master255 can you please test with an Android 8 emulator image and post the setup and results here? |
@bk138 No. I can't test with the emulator because it doesn't have network access |
--> https://developer.android.com/studio/run/emulator-networking should help |
@bk138 I added port redirection: Through the emulator console (telnet) Maybe I need to do something else? |
I just tested with
Then http://127.0.0.1:5800/vnc.html?autoconnect=true&host=127.0.0.1&port=5900 works. Can't reproduce the image quality issue either. |
@bk138 Yes. I was able to forward the ports too. |
Well if we can find a fix I'm happy to add it. I'm not saying you don't see an issue. I am just not able to reproduce it nor can I see a recent code change that might trigger it. To get this clear: I hear you, but what can be a proper fix? |
@bk138 It feels like libvncserver has changed. I think the problem is in it. |
@bk138 On the latest version 2.7.2 from the Play market, I got a black bar on the right side of the screen and the image quality deteriorated, even on the maximum image quality setting.
Connection via novnc
This did not happen on version 2.6.0.
The screenshot shows the maximum image quality, but you can clearly see that there are artifacts:
The text was updated successfully, but these errors were encountered: