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

GLTF Texture Filter Importing #54044

Closed

Conversation

TheTophatDemon
Copy link
Contributor

The GLTF/GLB importer does not seem to be taking into account filter, wrapping, and mipmap settings built into the file.
This feature implements a fix on the 3.x branch by creating and parsing the texture sampler objects from the GLTF file and applying its properties as flags to the created textures.

@TheTophatDemon
Copy link
Contributor Author

This pull request is based on the issue: #45206

@TheTophatDemon
Copy link
Contributor Author

Currently, these three cube models are not importing correctly with the current code. Investigation is ongoing.
gltf_textured_cubes.zip

@fire
Copy link
Member

fire commented Oct 22, 2021

Policy is that we backport from master to 3.x. So I'll see if I can help you.

@Sirus97
Copy link

Sirus97 commented Sep 22, 2022

@TheTophatDemon What's the status of this PR?

@TheTophatDemon
Copy link
Contributor Author

@TheTophatDemon What's the status of this PR?

I believe @fire made a separate pull request to continue working on this, since I ran out of time in my schedule.

@Calinou
Copy link
Member

Calinou commented Nov 16, 2022

I believe @fire made a separate pull request to continue working on this, since I ran out of time in my schedule.

I don't see any PR made by @fire related to glTF filter in 3.x.

Nonetheless, I've labeled this as salvageable in case another contributor wishes to continue this work.

@Sirus97
Copy link

Sirus97 commented Nov 16, 2022

@Calinou
This was actually implemented awhile ago here:
#66856

@Calinou
Copy link
Member

Calinou commented Nov 16, 2022

Superseded by #66856. Thanks for the contribution nonetheless!

@Calinou Calinou closed this Nov 16, 2022
@YuriSizov YuriSizov removed this from the 3.x milestone Dec 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants