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

roamguide: add package #185

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

sargon
Copy link
Contributor

@sargon sargon commented May 12, 2018

Latest state of roamguide. A tool to solve the situation where the client thinks he "sees" the AP nicely, but the AP have problems listing to the client. In the End it increases the user experience.

We have tested this in several high density settings, and the user experience increased dramatically.

@@ -0,0 +1,11 @@
# repeat the following for every wireless AP device you like to guide.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i would prefer a solution where this package detects itself if there is a single-band or a dual-band device, which in the latter case would need two sections for client0 and client1

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I opened an issue here: sargon/gluon-sargon#14

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This issue is said to be fixed in sargon/gluon-sargon@c9539a0 (though it just creates both and fails silently on single band devices).

@sargon
Copy link
Contributor Author

sargon commented May 30, 2018

@rotanid That a good thing, maybe someone can implement that, after the PR is through.

@rotanid
Copy link
Member

rotanid commented Jun 3, 2018

@sargon usually we ask for improvements before the PR is merged, but maybe you can put my mind at rest by answering my doubts:
how is this intended to be deployed? if i understand it correctly, it would only do its work on 5GHz on most dual band devices by default, as there's only a default config for client0, not for client1. this means, it can't fully work in an RF environment with mixed single-band/dual-band devices as on the dual-band devices it won't touch the clients connected to the 2,4GHz wifi VIF

@@ -0,0 +1 @@
* * * * * /usr/bin/roamguide
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

New line is missing

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -0,0 +1,11 @@
# repeat the following for every wireless AP device you like to guide.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I opened an issue here: sargon/gluon-sargon#14

@rubo77
Copy link
Contributor

rubo77 commented Oct 9, 2018

TODO:

  • Lua statt bash
  • Unterstützung für 5GHz

@maurerle
Copy link
Member

As this package is unlikely to be merged in the current state, but seemst to actually be used by at least ffki, you might like to add this to the community-packages repository instead if this is still needed and used.

That way gives a better overview of packages used in communities, while having a lower barrier set @sargon.

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

Successfully merging this pull request may close these issues.

4 participants