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
Allow enabling computed mipmaps in LargeTextureStore
#6048
Conversation
No testing whatsoever? |
Added 👌 |
[BackgroundDependencyLoader] | ||
private void load(IRenderer renderer, GameHost host, Game game) | ||
{ | ||
dependencies.CacheAs(new LargeTextureStore(renderer, host.CreateTextureLoaderStore(new NamespacedResourceStore<byte[]>(game.Resources, "Textures")), manualMipmaps: false)); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Setting manualMipamaps: true
doesn't change anything in the test.
I'd hope the test shows the difference between the two.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Setting manualMipamaps: true doesn't change anything in the test.
That's weird, for me the difference is very clear (I'm on D3D)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May have been hot reload failing. But I'd also hope the test can be run in both modes. I don't want to have to change code each time I want to check a test.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Will do. Was just trying to ensure it works for you and not a renderer issue or something.
As per discussion.