-
Notifications
You must be signed in to change notification settings - Fork 3
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
The texture of the Simple Texture model of glTF 2.0 is upside down #2
Comments
Drawing the connection between javagl/JglTF#10 (comment) and more importantly CesiumGS/cesium#4808 (comment) - and as you can see in these links: It's difficult. The texture coordinates have been flipped for the 2.0 version of the models, as per request in the second link. Ironically, this was supposed to avoid confusion ;-) However, now I have also updated the texture image (the PNG file), and added a note in the README at https://github.com/javagl/gltfTutorialModels/blob/master/SimpleTexture/README.md , emphasizing that the texture image now actually shows the real texture coordinates. Also, another disclaimer: Much of the discussion around 2.0 is not finalized. The models here are highly preliminary. |
Sorry, this....
... was misleading: The 2.0 branch is not yet completely updated! Particularly, the "SimpleTexture" model in the 2.0 branch is older than the "SimpleTexture" model in the 1.1 branch. In the 1.1 branch, the texture is flipped and the texture coordinates are updated. In the 2.0 branch, this is not done yet. The following models have been updated to 2.0, as far as this is possible:
But these models are still in an old 1.1-ish state:
Sorry, I know that this is a mess. But the update to 2.0 came like a bolt from the blue. The changes between 1.0 and 2.0 mean that much of the existing code basically has to be rewritten or at least heavily refactored. Beyond that, it's still not clear whether the tutorial should cover PBR (which is still far from being finalized), or whether the tutorial models should be based on PBR or on The best thing would probably be to not rely on this |
I understood. I will try again when the model is updated. |
The SimpleTexture model is not displayed in the latest version.
It seems that unitSquare0.bin does not exist.
I think so, too. I would like to update the sample again when the specifications have made progress. |
Right now, I cannot even test the models myself - I have not yet set up the infrastructure to load 2.0. There also is no validator for 2.0, so using these models probably does not make sense. I'll notify you when they are merged into |
The |
I think that it is probably a mistake, but it seems that the orientation of the texture is reversed in glTF 1.1 and 2.0 models.
https://github.com/javagl/gltfTutorialModels/blob/master/SimpleTexture/glTF/square.bin
![image](https://cloud.githubusercontent.com/assets/1833979/22627737/4694d0e2-ec0b-11e6-9adc-4e8557e3cb3f.png)
https://github.com/javagl/gltfTutorialModels/blob/2.0/SimpleTexture/glTF/square.bin
![image](https://cloud.githubusercontent.com/assets/1833979/22627741/5347d726-ec0b-11e6-83be-ed3f1ab28de0.png)
The text was updated successfully, but these errors were encountered: