Don McCurdy

Results 268 issues of Don McCurdy

Now that version 2.x supports HTTP requests, it'd be helpful to add a few "quality of life" things for working with remote files. A couple I've noticed: - Logger could...

feature
good first issue
package:cli
package:core

**Describe the bug** After resampling, the animation sequence in this example is broken. **To Reproduce** [Archive.zip](https://github.com/donmccurdy/glTF-Transform/files/8081046/Archive.zip) Steps to reproduce the behavior: 1. `gltf-transform resample Arm.glb Arm-resampled.glb` Decreasing tolerance will fix...

bug
package:functions

Currently each transform in the `@gltf-transform/functions` package includes a manual `cmd: Complete` print statement at its end. It's easy to forget those when adding new functions and they're not particularly...

code health
package:functions

Planned for a TBD future version, most likely 3.0. Moving forward, this will be an [ESM package](https://gist.github.com/sindresorhus/a39789f98801d908bbc7ff3ecc99d99c).

feature
code health
package:extensions
package:core
package:functions

Options provided for the squoosh encoders are intentionally limited - i don't understand them all, and many are not documented by squoosh itself yet.: https://github.com/donmccurdy/glTF-Transform/blob/6671e0c480ee990bfa119ea946e328bbe91f4233/packages/functions/src/squoosh.ts#L20-L24 But there are a few...

feature
package:cli

For example, accept a folder path as a global option, or use `~/.gltf-transform`. Ideally it load scripts in that folder and let them add new commands to the CLI by...

feature
help wanted
package:cli

The [Writer](https://github.com/donmccurdy/glTF-Transform/blob/main/packages/core/src/io/writer.ts) and [Reader](https://github.com/donmccurdy/glTF-Transform/blob/main/packages/core/src/io/reader.ts) classes are static classes with a single static method each, `write()` and `read()`. Both would benefit from refactoring into individual `writeJSONDocument()` and `readJSONDocument()` functions, rather than...

good first issue
code health
package:core

**Describe the bug** API documentation for `@gltf-transform/extensions` is missing source links to implementations coming from the `@gltf-transform/core` package. **To Reproduce** 1. Visit https://gltf-transform.donmccurdy.com/classes/extensions.materialsvariants.html#isRequired 2. Scroll to `dispose` or `isRequired` definitions...

bug
documentation

I'd like to bring unit test coverage back above 90% again. Codecov reports the files with low coverage — click the graph to open it in a new tab, then...

good first issue
code health

See discussion in https://github.com/donmccurdy/glTF-Transform/issues/543 — documentation for writing custom implementations of extensions, without or without property references, would be a very useful addition to the library. Particularly, how to extend...

documentation