Opened 13 years ago

Closed 13 years ago

#1506 closed patch (Fixed)

scheduler plugin does not restore speed on yellow->green transition

Reported by: wyrm Owned by: Cas
Priority: minor Milestone: 1.3.2
Component: Plugin Version: 1.3.1
Keywords: scheduler Cc:

Description

Everything works as expected on a green->yellow transition, but max speed is NOT restored on a yellow->green transition.

Change History (5)

comment:1 Changed 13 years ago by wyrm

Im running deluge-daemon on my htpc server, and deluge-gtk on my laptop, the speed reported on the deluge-gtk status line is always the max_(download|upload)_speed from core.conf, the GUI doesn't change the status line on green->yellow transition, but it DOES limit the speed as expected.

I have the following plugins enabled: Scheduler, Stats and WebUi?.

Some relevant stuff from my configuration:

wyrm@htpc:~/.config/deluge$ cat core.conf | grep -E "\"max_(upload|download)_speed\""
  "max_upload_speed": 20.0, 
  "max_download_speed": 80.0, 

wyrm@htpc:~/.config/deluge$ cat scheduler.conf | grep -E "\"low_(up|down)\""
  "low_up": 5.0, 
  "low_down": 20.0, 

All the other config values, such as "max_active_limit/low_active", "max_active_downloading/low_active_down" and "max_active_uploading/low_active_up" does work as expected, it changes .torrent state to "queued" on green->yellow transition and resumes the torrent properly on yellow->green transition. The issue only happens with the "speed" values.

comment:2 Changed 13 years ago by wyrm

Turns out that only max_upload_speed wasn't being correctly restored, seems to be just a copynpaste issue. patch attached.

diff -bruN scheduler.orig/core.py scheduler/core.py
--- scheduler.orig/core.py	2011-02-01 12:13:58.000000000 -0200
+++ scheduler/core.py	2011-02-01 12:14:24.000000000 -0200
@@ -64,7 +64,7 @@
 
 CONTROLLED_SETTINGS = [
     "max_download_speed",
-    "max_download_speed",
+    "max_upload_speed",
     "max_active_limit",
     "max_active_downloading",
     "max_active_seeding"

comment:3 Changed 13 years ago by Cas

  • Milestone changed from Future to 1.3.2
  • Priority changed from major to minor
  • Type changed from defect to patch

comment:4 Changed 13 years ago by Cas

  • Owner set to Cas
  • Status changed from new to accepted

comment:5 Changed 13 years ago by Cas

  • Resolution set to fixed
  • Status changed from accepted to closed

1.3-stable: 1731fd641b

Note: See TracTickets for help on using tickets.