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

Merge to Warp to compare features / overhead #9

Closed
JackNewman12 opened this issue Jan 11, 2021 · 2 comments
Closed

Merge to Warp to compare features / overhead #9

JackNewman12 opened this issue Jan 11, 2021 · 2 comments

Comments

@JackNewman12
Copy link
Owner

I think warp might be better than rocket for such a small project.

TODO: Merge over to warp and see if it reduces the memory footprint (its only 2mb atm).
Also just curious how that library compares in features.

@JackNewman12
Copy link
Owner Author

JackNewman12 commented Jan 13, 2021

There is now a warpmerge branch where I have done this work.
e7ba968
f814c02

Binary size is about the same. (windows jumped up by 2.5MB=>3.5MB)
Speed is the same. 1ms for everything (altho enabling compressed for javascript makes it take 9ms, but 250KB=>70KB, worth it on slow network)
memory usage 3MB => 1.1MB (windows usage increased 2MB => 3MB)

Overall I think its an improvement in every factor. (altho rust-analyser does not seem to play as nicely)

@JackNewman12
Copy link
Owner Author

Finally decided to merge it over to main in #27
Also gives us websockets so maybe we can update the web interface

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

1 participant