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

Add option to compress messages #274

Closed
maniyar1 opened this issue May 27, 2020 · 2 comments
Closed

Add option to compress messages #274

maniyar1 opened this issue May 27, 2020 · 2 comments

Comments

@maniyar1
Copy link

I'm still new to the codebase, so let me know if I misrepresent something...

I was curious whether an option to compress packets would be useful? Looking at #225, I was wondering if compressing could help avoid fragmentation where it would otherwise be needed.
As far as algorithms I don't know what would be best; quake 3 used per-packet Huffman compression. We could also use gzip I guess, or implement multiple algorithms and let the user choose. I'm not really that informed about compression algorithms.
Let me know if this is something that would be useful and I can try my hand at implementing it.

@brandonlamb
Copy link

Hello!

My understanding is that when it comes to building multiplayer netcode, you really want to hand pack your packets for optimal size, or use something like google protocol buffers.

Is the context for something like Huffman coding for string values?

@maniyar1
Copy link
Author

maniyar1 commented Aug 2, 2020

That could be one application...now that I've played with everything a bit more I imagine this isn't really something that needs to be in the library, it can be something the user does themselves. The quake 3 example is...valid but quake 3 has a very unique network setup. I think I'll end up closing the issue soon

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

2 participants