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

Loss of image quality on v2.7.0 vs 2.6.0 #240

Open
master255 opened this issue Sep 10, 2024 · 24 comments
Open

Loss of image quality on v2.7.0 vs 2.6.0 #240

master255 opened this issue Sep 10, 2024 · 24 comments
Labels
answer-needed Waiting for response from OP bug Something isn't working

Comments

@master255
Copy link

master255 commented Sep 10, 2024

@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:
image

@master255 master255 added the bug Something isn't working label Sep 10, 2024
@bk138
Copy link
Owner

bk138 commented Sep 10, 2024

@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 bk138 closed this as not planned Won't fix, can't repro, duplicate, stale Sep 10, 2024
@bk138 bk138 added the duplicate This issue or pull request already exists label Sep 10, 2024
@master255
Copy link
Author

@bk138 You closed the bug too quickly.
My mistake. There were no problems on version 2.6.0. Here's a screenshot. I just installed it. Probably the degradation happened between versions 2.6.0 and 2.7.0
image

@master255
Copy link
Author

@bk138 Re-open the bug

@bk138 bk138 reopened this Sep 10, 2024
@bk138
Copy link
Owner

bk138 commented Sep 10, 2024

@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 bk138 changed the title Black line on the right Loss of image quality on v2.7.2 Sep 10, 2024
@bk138 bk138 added answer-needed Waiting for response from OP and removed duplicate This issue or pull request already exists labels Sep 10, 2024
@master255
Copy link
Author

master255 commented Sep 10, 2024

@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.

@master255
Copy link
Author

@bk138 How can this be fixed?
image

@bk138
Copy link
Owner

bk138 commented Sep 11, 2024

@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.

@master255
Copy link
Author

@bk138 Just all defaults and image quality to max.
Android 8.0.
Galaxy S8+.

@bk138
Copy link
Owner

bk138 commented Sep 11, 2024

@bk138 Just all defaults and image quality to max. Android 8.0. Galaxy S8+.

You mean server side scaling? Which client was used? Which server version worked, which did not?

@master255
Copy link
Author

@bk138 1. Yes.
2. noVNC buildin
3. 2.6.0 good, 2.7.0+ bad

@bk138
Copy link
Owner

bk138 commented Sep 11, 2024

@master255 I now tried

  • v2.6.0 and v2.7.0
  • on the same device,
  • built from source,
  • with default settings (esp. server-side scaling)
  • and using "fast" screen sharing (i.e. not the fallback mode),
  • connecting via the builtin noVNC:

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.

@master255
Copy link
Author

@bk138 Android 8.0? What's the device?

@bk138
Copy link
Owner

bk138 commented Sep 11, 2024

Android 12, some Samsung.

@master255
Copy link
Author

@bk138 So I'm testing on Android 8.0. You should test on that version too. Or your app is only for Android 12+?

@master255
Copy link
Author

@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.
image

@master255
Copy link
Author

@bk138 And I'm sure on version 2.6.0 the picture quality will be fine.

@bk138
Copy link
Owner

bk138 commented Sep 12, 2024

@master255 can you please test with an Android 8 emulator image and post the setup and results here?

@bk138 bk138 changed the title Loss of image quality on v2.7.2 Loss of image quality on v2.7.0 vs 2.6.0 Sep 12, 2024
@master255
Copy link
Author

@bk138 No. I can't test with the emulator because it doesn't have network access

@bk138
Copy link
Owner

bk138 commented Sep 12, 2024

@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

@master255
Copy link
Author

master255 commented Sep 14, 2024

@bk138 I added port redirection:
redir add tcp:5800:5800
redir add tcp:5900:5900

Through the emulator console (telnet)
But still can't open the link http://127.0.0.1:5800/vnc.html?autoconnect=true&host=127.0.0.1&port=5900

Maybe I need to do something else?

@bk138
Copy link
Owner

bk138 commented Sep 14, 2024

@bk138 I added port redirection: redir add tcp:5800:5800 redir add tcp:5900:5900

Through the emulator console (telnet) But still can't open the link http://127.0.0.1:5800/vnc.html?autoconnect=true&host=127.0.0.1&port=5900

Maybe I need to do something else?

I just tested with

adb forward tcp:5800 tcp:5800
adb forward tcp:5900 tcp:5900

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.

@master255
Copy link
Author

master255 commented Sep 17, 2024

@bk138 Yes. I was able to forward the ports too.
I think your problem is that you are overprotecting your program.
Here, I found an obvious problems with the image quality.
image

@bk138
Copy link
Owner

bk138 commented Sep 17, 2024

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?

@master255
Copy link
Author

@bk138 It feels like libvncserver has changed. I think the problem is in it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
answer-needed Waiting for response from OP bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants