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

WEB3D_quantized_attributes link is broken #1282

Closed
eodabash opened this issue Mar 13, 2018 · 4 comments
Closed

WEB3D_quantized_attributes link is broken #1282

eodabash opened this issue Mar 13, 2018 · 4 comments

Comments

@eodabash
Copy link

eodabash commented Mar 13, 2018

This link has been broken since last week:

https://github.com/KhronosGroup/glTF/blob/master/extensions/Vendor/WEB3D_quantized_attributes/README.md

Is this a known issue, or was it intentionally removed?

@javagl
Copy link
Contributor

javagl commented Mar 13, 2018

The extensions have been moved into version-specific directories - here, the 1.0 part was added, so the new link is

https://github.com/KhronosGroup/glTF/blob/master/extensions/1.0/Vendor/WEB3D_quantized_attributes/README.md

(If the old link still appears somewhere (maybe in another README), just drop a note here, then it will be fixed quickly)

@eodabash
Copy link
Author

Ah I see, thanks. I was just trying to read the spec for the extension and every link I followed from Google was taking me there.

A few quick examples:
#588
https://cesium.com/blog/2016/08/08/cesium-web3d-quantized-attributes/
https://www.khronos.org/assets/uploads/developers/library/overview/gltf-overview.pdf

@javagl
Copy link
Contributor

javagl commented Mar 14, 2018

Thanks for the hints.

The link in the issue has been updated.
The link in the blog entry could probably only be updated by the Cesium team (@lasalvavida ? )
The task to update the links in the overview is already scheduled as of javagl/gltfOverview#3

If there are no objections, I'd close this issue later today.

(If there are further outdated links, they could still be mentioned here, and if necessary, the issue can be reopened)

@pjcozzi
Copy link
Member

pjcozzi commented Mar 14, 2018

Thanks @eodabash!

We'll fix the Cesium blog post link.

@pjcozzi pjcozzi closed this as completed Mar 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants