Skip to content

Fix automatic texture memory on macOS, Android, OpenGL ES, and Intel graphics. #1398

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

Merged
merged 1 commit into from
Oct 28, 2021

Conversation

JulianGro
Copy link
Contributor

@JulianGro JulianGro commented Oct 10, 2021

Fixes #1394
While this PR was made to fix automatic texture memory on macOS, it actually also fixes it on Android, OpenGL ES, and Intel graphics.

In practice this will no longer make macOS users see mega blurry textures by standard and fix the same issue on some other configurations.
The issue with automatic texture memory on Nvidia proprietary Windows is not touched by this.

I have only tested this on macOS Catalina with AMD graphics for now.

@JulianGro JulianGro added needs testing (QA) The PR is ready for testing bugfix needs CR (code review) macos This is related to OSX and Mac. labels Oct 10, 2021
@JulianGro JulianGro requested a review from daleglass October 10, 2021 13:08
@digisomni digisomni changed the title Fix automatic texture memory on macOS, Android, OpenGL ES, and Intel graphics Fix automatic texture memory on macOS, Android, OpenGL ES, and Intel graphics. Oct 14, 2021
@digisomni digisomni added this to the 2021.2.0 Selene Release milestone Oct 14, 2021
@digisomni digisomni added CR Approved At least one code reviewer has approved the PR. and removed needs CR (code review) labels Oct 16, 2021
@digisomni
Copy link
Member

This crashes on startup with my Mac Mini (Intel i7, iGPU). https://hastebin.com/iwaxadiwut.yaml

@JulianGro
Copy link
Contributor Author

JulianGro commented Oct 17, 2021

@digisomni can you confirm that PR builds from any other PR run?
This PR doesn't do anything that should be able to cause a crash. On Intel graphics the only change is a simple multiplication that converts from mebibytes to bytes.

@digisomni
Copy link
Member

PR #1411 ran fine for me and that's a newer PR than this one.

@JulianGro
Copy link
Contributor Author

PR #1411 ran fine for me and that's a newer PR than this one.

Can you try any other PR? That one likely fixes the issue.
Also do I understand it right that it crashes on startup? Not on window resize, etc.?

@digisomni
Copy link
Member

So, I tried a couple of PR builds again (older and newer) and then tried this one again and it worked, for some odd reason.

@ArcadeFever
Copy link
Contributor

ArcadeFever commented Oct 21, 2021

This PR build worked fine for me: Macbook Pro (16-inch 2019), Intel I9, AMD graphics, Big Sur.

@Steve-Pruneau
Copy link

I am unsure whether PR 1398 is working or not. There may have been an issue with cache on my machine. The place to test this issue is at NDN World, upper floor where art works and descriptions are displayed. I got blurry images and text. I tried clearing cache (Reload Content) and also changed texture memory from automatic to 1024 and then back again. Still blurry. I also quit the app and started again. Still blurry. Then I selected Network > Clear Disk Caches, quit the app and started up again. This produced sharp images and text. iMac 27 inch. Graphics: Radeon Pro 570 4 GB. macOS 11.6 Big Sur (latest version)
vircadia-snap-by-Steve-Pruneau-on-2021-10-21_15-40-07
vircadia-snap-by-Steve-Pruneau-on-2021-10-21_16-00-06

@Steve-Pruneau
Copy link

My guess - it doesn't seem to harm anything. And others believe the fix works. So I say, go ahead and put it into production.

@JulianGro
Copy link
Contributor Author

@Steve-Pruneau could you zip up your logs folder and post that here? https://docs.vircadia.com/faq.html#interface-log-files
That way we can make sure that it does actually work as intended.
The blurry textures don't necessarily mean that it didn't work as intended.
Maybe even set it to automatic again, restart Interface, and send us a log of that.

@Steve-Pruneau
Copy link

Steve-Pruneau commented Oct 22, 2021

@JulianGro Unfortunately, last night, I deleted folders and files related to testing PRs. So I re-ran the test this morning. Logs attached. I think PR 1398 is working on my iMac now. I think it is because I clicked "Reset all my settings". The Interface app PR 1398 was crashing this morning when I tried to launch. So I chose the option to launch with "Reset all my settings". Then it launched smoothly. Graphics settings default to "Medium". Medium and High do not look good on Mac. Something about rendering on Mac. (I think this may not be related to PR 1398 and texture memory. In future, I think we should set graphics default "Low"). I changed graphics settings to Low. Appearance of NDN World changed immediately. This is new behavior and VERY GOOD. In past, changing graphics settings would take some number of seconds, like 15 seconds, 30 seconds or something like that to change, like slowly loading content. Upon checking upper level art work at NDN World, images are sharp (after all content is finished loading). So, I was not able to reproduce the experience yesterday. I believe "Reset all my settings" may have contributed to PR 1398 working for me now.
vircadia-log_2021-10-22_08.33.21.txt
vircadia-log_2021-10-22_08.34.09.txt
vircadia-log.txt

@JulianGro
Copy link
Contributor Author

JulianGro commented Oct 22, 2021

The logs indicate it working correctly to me as well.

@digisomni digisomni added QA Approved The PR has been tested successfully. renderer Touches or changes rendering and removed needs testing (QA) The PR is ready for testing labels Oct 23, 2021
@daleglass daleglass merged commit 17b4bf1 into vircadia:master Oct 28, 2021
@JulianGro JulianGro deleted the macOS_fix_texture_memory branch October 29, 2021 18:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bugfix CR Approved At least one code reviewer has approved the PR. macos This is related to OSX and Mac. QA Approved The PR has been tested successfully. renderer Touches or changes rendering
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automatic Texture memory not working on macOS
6 participants