search

presentday-presenttime / Deluge-Rainmeter-Skin

Skin that uses deluge-console to display data about current torrents
2 stars 0 forks source link

Change CheckDelugeRate based on torrent activity #9

Open maxdd opened 7 years ago

maxdd commented 7 years ago

Rough idea: Torrent Active = 0 --> CheckDelugeRate = 10 Torrent Active != 0 --> CheckDelugeRate = 5

I feel like it it is way to slow when updating stuff

presentday-presenttime commented 7 years ago

What do you mean by 'Torrent Active'?

10 seconds was the fastest I could consistently run the skin at without causing memory leaks

If it updates too quickly, it will create a new instance of deluge-console before closing the previous.

!properly limit update so deluge isn't tied to the rest of the skin updating, and deluge won't update if the previous one is still running.

Is referencing this issue.

maxdd commented 7 years ago

If there are Downloading or Uploading torrents it would be better to have more real time values. Anyway it is strange that it requires so much, did you try to measure where it takes so much? Can't you just run a single console (let it standby forever) with cyclic commands every 5 secs?

presentday-presenttime commented 7 years ago

I hadn't considered having the console run indefinitely. That is probably a much better way to handle things.

presentday-presenttime commented 7 years ago

10

console running indefinitely will not work.

I think it would still be possible to adjust the delay based on number of active torrents though.