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

DRACOLoader: Update Draco decoders to v1.5.6 #25475

Merged
merged 1 commit into from
Feb 10, 2023

Conversation

donmccurdy
Copy link
Collaborator

Note that DRACOLoader now uses Draco APIs that weren't available in earlier versions of the Draco library. Users upgrading to the latest THREE.DRACOLoader will also need to update the decoders.

@epreston
Copy link
Contributor

epreston commented Feb 10, 2023

Quick question on what is public API; are the following public ?

decodeDracoFile( buffer, callback, attributeIDs, attributeTypes, vertexColorSpace = LinearSRGBColorSpace )

decodeGeometry( buffer, taskConfig )

debug()

Looks like no public API changes but, checking the doc all the same, and these popped out as possible additions.

@mrdoob mrdoob merged commit a79f078 into mrdoob:dev Feb 10, 2023
@donmccurdy donmccurdy deleted the chore/draco-decoders-v1.5.6 branch February 10, 2023 14:04
@donmccurdy
Copy link
Collaborator Author

I'd prefer to consider those internal APIs, no need to document them for now!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Dependencies Pull requests that update a dependency file Loaders
Projects
None yet
Development

Successfully merging this pull request may close these issues.

DRACOLoader: Draco library version and usage is outdated
3 participants