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
New Extract Interface #1173
base: main
Are you sure you want to change the base?
New Extract Interface #1173
Conversation
Here is a demo of the new interface. It seems to be working(!) |
Squash this commit, it's only a feat so it gets released on an alpha channel.
Here is a codesandbox demo of the API. |
remaining:
`curryMeyda` issueRight now {zcr: number} | {zcr: number}[] rather than returning a type with the correct number of results as per the tuple length of the signal argument. It's obviously because that's how I've written the code. But I've got to fix it. in other PRs
Spectral Flux is broken in v5, and the fix is a breaking change so it might be ok to not support spectral flux if we release this on v5, which would mean we could delay handling internal storage of history until v6. To be clear, I haven't decided whether we should commit to this. The current (prior to this) interface has some major footguns for the user as discussed in #257. While this API solves them, the lengths it goes to to provide nice intellisense and types have created some major complexities in the type layer for maintainers. There are some real batshit types in here tbh. See I'm going to also implement a class based interface. I'm not a fan, really, of those, but it would be good to see a class based implementation and think deeply about the tradeoffs in usability and maintainability. Either way, I learned a lot. And ENORMOUS thanks to @yasakbulut who paired with me this weekend to solve some of the absolutely gnarly typescript problems that we had and get this over the finish line after years. He will be in |
I'm at it again, and this time I've given up on currying