UI does not display MAX_RATE variable properly

Bug #216576 reported by chapterthree
2
Affects Status Importance Assigned to Milestone
LottaNZB
Fix Released
High
Severin H

Bug Description

~/.hellanzb/hellanzb.conf
# Uncomment this line to limit all server connections to the specified KB/s
# bandwidth
Hellanzb.MAX_RATE = 150 # limit to 150kB/s

~/.lottanzb/hellanzb.conf
Hellanzb.MAX_RATE = 150

UI preferences window:
(Checked) Limit download speed: 1000 KBit/s

How is it getting 1000 KBit/s?
1000 KBit/s / 8 = 125kB/s
150kB/s * 8 = 1200 KBit/s

(Although the conversions should be consistent, I'll mention I use http://www.easycalculation.com/bandwidth-calculator.php to double check my conversions)

Revision history for this message
chapterthree (chapterthree) wrote :

Ah, I see now, it's kilobytes, not kilobits. The text on the UI needs to be corrected to read KB/s.

So with that clarification out of the way, now the question is why does LottaNZB not honor the MAX_RATE specified in the config file?

Revision history for this message
Severin H (severinh) wrote :

Thanks for reporting this bug! We apparently need to do more testing before releasing new versions of LottaNZB.

The UI label should indeed read KB/s and when we turned the input control into a spin button, we forgot to accordingly change the method that fills in the MAX_RATE value specified in the HellaNZB config file (and the one that saves the value).

I'm commiting a fix right now.

Changed in lottanzb:
assignee: nobody → lantash
importance: Undecided → High
milestone: none → 0.2.1
status: New → In Progress
Severin H (severinh)
Changed in lottanzb:
status: In Progress → Fix Committed
Revision history for this message
Severin H (severinh) wrote :

The bug should be fixed now but I'm going to do some testing later just to make sure.

Just to clarify things: 1000 was the default value to be chosen when the user acivates bandwidth throttling. It's now set to 150 KB/s, the value that can be found in the default HellaNZB config file.

Severin H (severinh)
Changed in lottanzb:
milestone: 0.2.1 → 0.3
Severin H (severinh)
Changed in lottanzb:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.