transformers.js icon indicating copy to clipboard operation
transformers.js copied to clipboard

Library no longer maintained?

Open pax-k opened this issue 1 year ago • 3 comments

Question

1 year has passed since this PR is ready for merge: Support React Native #118

Should we do our own fork of xenova/transformers.js ?

pax-k avatar Mar 14 '24 10:03 pax-k

The time and effort probably flows into v3 with WebGPU support - implying that this library is no longer maintained because the PR isn't merged is a bit of a stretch.

Did you try a quick test merge with v3? Supporting many plattforms is demanding a lot of work and testing.

Personally I would also try to keep the API unified - statements like import wav from 'node-wav'; are somewhat a breaking change, since you need to update import maps - I would probably aim for a simple AudioContext polyfill.

The polyfill route keeps the PR's small and reducing merge conflicts and you can even reuse unit tests e.g.

kungfooman avatar Mar 14 '24 11:03 kungfooman

Given last release was last week, I think it's still actively maintained. Take a look at https://github.com/xenova/transformers.js/releases

osanseviero avatar Mar 14 '24 18:03 osanseviero

@pax-k I encourage you to check out the v3 branch: https://github.com/xenova/transformers.js/pull/545. At the moment, we are prioritizing WebGPU support over React Native support, but I can assure you that the latter will come, eventually.

xenova avatar Mar 14 '24 21:03 xenova

For now, I'm prereleased here I will try my best to keep up to date with the latest version.

P.S. Our company is heavily on React Native and has some web compatibility. This package is very good to open the gate to edge AI.

hans00 avatar Mar 18 '24 09:03 hans00

Thank you for your kind words @hans00, and I appreciate your patience! We're also in contact with the onnxruntime-react-native team and we'll be prioritizing React Native support once WebGPU support has been completed.

Closing this issue, since the library is most certainly still being "maintained" 😉

xenova avatar Apr 03 '24 23:04 xenova