search

MichaelWallace30 / CommProtocol

SImple generic Protocol for communication
2 stars 8 forks source link

Complete pinging system merge with Master #36

Closed Jester565 closed 7 years ago

Jester565 commented 7 years ago

https://docs.google.com/presentation/d/1TLZO3QDt7jV3geJX2tr0D69aT1TVEd-wQujBbS3sxno/edit?usp=sharing

There's a presentation on how the pinging system works. Or at least most of it.

The main concerns with the Pinging system are

  1. There is no option not to have a PingManager.
  2. Needs testing on multiple computers. After about 8 clients all on one computer, a lot of them will be set to inactive but this seems to be due to high packet loss.
MichaelWallace30 commented 7 years ago

auto_queue.h has library shared_mutex which is c++17 you can't use c++17. http://en.cppreference.com/w/cpp/thread/shared_mutex This needs to be altered

MichaelWallace30 commented 7 years ago

I think a vs update will fix it. I will add the notes into the guide if this is the issue and will just leave it.