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

improve performance #23

Open
paul-hanneforth opened this issue May 17, 2021 · 3 comments
Open

improve performance #23

paul-hanneforth opened this issue May 17, 2021 · 3 comments

Comments

@paul-hanneforth
Copy link

I think the current package became a little to slow. Especially since the caching was removed. Are there any suggestions on how to solve this problem? I was thinking maybe we could write a native c library and then import it to NodeJS if that's possible

@mde
Copy link
Owner

mde commented May 19, 2021

Native C would definitely be faster, except we know that in some cases, Chrome's function inlining results in execution times faster than C code. I'm not sure how big of a win this would be, but I'd be happy to take a look at a PR that implements this.

@Miigon
Copy link

Miigon commented May 19, 2021

Yeah, actually there is a cool new technology called WebAssembly now which isn't a thing yet at the release of this package.
Anyone considering the possibility of optimizing the package with assembly code on platforms that supports it? Some benchmarking should be done beforehand, however, to make sure that such an optimization wouldn't make the package too fast to hit the state speed limit law of CA.

@SpeedyCraftah
Copy link

Is this going to be considered? I am running into performance issues with this package on Ubuntu running node 16.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants