-
-
Notifications
You must be signed in to change notification settings - Fork 45
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
Support for other units than mbits #73
Comments
Part of the issue is that we need to really validate the input and convert to a common measure for the internal calculations (htb rates, ceil, etc). It would take a bit of work. But it can be done. i may not be able to do it right away but it's a good consideration for long term. |
It's recommended by tc to set rates at multiplication of 8 only in using kbits.
There are also some edge cases that requires queues at 256K or less to queue. I would not remove that feature. Maybe there is another way to archieve that? |
This makes sense. I just assumed ISPs would no longer offer service below 3Mbps in either direction so that it wouldn't matter if we had a 3Mbps min - but perhaps there are situations there worth considering. |
I analised deeply the situation we got also some clients with a wireless service 10/2 and they don't want to change their agreement maybe from 5-7years ago. I know that we could bump up the speed but then the client will have less motivation to change tariff. |
Ok in that case let's definitely see about at least lowering the threshold to 2Mbps min. That may hold us over until a bigger rework can be done. |
So now:
|
I need to also change something in LibreQoS.py to allow 2Mbps or more, one sec |
Ok now your 2Mbps use case should work cffbfa8 |
:) |
I like the ability to be clear about mbits in decimal or binary. |
Currently in ShapedDevices.csv it's Mbit in decimal form only. Would it be helpful to some operators to be able to provide it in binary? |
See! I think about it in binary, and so do many devices. I imagine operators think of it in decimal? It would explain a lot. It's not as bad a problem as newtons vs other measures but... binary.... |
Seriously. Straightening out decimal vs binary would be useful. |
I just try to integrate with our CRM where is the delaration of kbits instead of mbits. I am thinking what to do now.
I did a little research how others are generating queue configs.
I think a good approach for that is to allow: K, M, G into Shaper rates.
tc supports:
We should passthrouh the unit from ShapedDevices.csv to tc rules in linux-tc.txt also.
Currently it stores them as integers, so currently fractions would not be possible. Do you require that for your service plans? If so we can change to decimal if it's really needed.
Some few old customers just got some values - but this is not the point, because I could upper thirs limit in db one time.
I create ShapedDevices.csv using mbit or kbit unit declaration. With mbit there is a problem that You need to round kbit value and it gets You a decimal or 0 ;-)
The point is its better to have more clear configs to debug and definition in
linux-tc.txt
.The text was updated successfully, but these errors were encountered: