Resampling emits unnecessary warning message

[viridia]

Describe the bug

I am using both the resample and dedup plugins, but nevertheless resample still prints a warning advising me to use dedup:

resample: Resampling required copying accessors, some of which may be duplicates. Consider using "dedup" to consolidate any duplicates.

I realize this is a fairly trivial cosmetic bug, but it's causing a bunch of extra console spam in my build script output. I realize that I could shut off all warning messages, but that is not what I want either - it's just this one message that is causing an issue.

To Reproduce

  gltf.transform(resample(), dedup());

Expected behavior

Ideally, the resample plugin would know which other plugins are in the pipeline and suppress the message if dedup is also present.

Versions:

• Version: 0.12.14
• Environment: Node v15

[donmccurdy]

Yeah, I agree it'd be nice to silence this warning... Had been hoping there was a good way to track and/or avoid the duplicate accessors more closely in resample() but that doesn't appear to be easy. Letting the resample function know something about the other functions in the pipeline would be an option, but would need to think about how that would work e.g. with user-provided functions and possibly function names possibly obfuscated by the compiler.

[viridia]

Normally, I'd suggest some general purpose way to allow functions to "know" about each other, but it seems to me that might not be necessary, because the relationship between resample and dedup is already a special case. This is evident by the fact that resample explicitly names 'dedup' in the text of the message. I don't know if there are any other cases where a function needs to know about the existence of another function.

I haven't looked at the internals of gltf-transform to make any concrete suggestions. However, one random thought is selective filtering of the console output - simply look for that specific string and suppress it.