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

DietPi-Software | rTorrent: Remove deprecated commands/variables #3273

Closed
bbsixzz opened this issue Dec 20, 2019 · 7 comments
Closed

DietPi-Software | rTorrent: Remove deprecated commands/variables #3273

bbsixzz opened this issue Dec 20, 2019 · 7 comments
Labels
Bug 🐞 Solution available 🥂 Definite solution has been done
Milestone

Comments

@bbsixzz
Copy link

bbsixzz commented Dec 20, 2019

Creating a bug report/issue

Required Information

DietPi version |
G_DIETPI_VERSION_CORE=6
G_DIETPI_VERSION_SUB=26
G_DIETPI_VERSION_RC=3
G_GITBRANCH='master'
G_GITOWNER='MichaIng'

Distro version | buster 10.2

Kernel version |
Linux DietPi 4.19.75-v7l+ #1270 SMP Tue Sep 24 18:51:41 BST 2019 armv7l GNU/Linux

SBC device | RPi 4 Model B (armv7l)

Power supply used | 3A Official

SDcard used | SanDisk ultra

Extra details

There are lots of errors in rTorrent console as per this post

I edited the file manually and fixed them.

[EDIT]

This one must be a pretty big deal too:

@MichaIng MichaIng added the External bug 🐞 For bugs which are not caused by DietPi. label Dec 20, 2019
@MichaIng MichaIng added this to the v6.27 milestone Dec 20, 2019
@MichaIng MichaIng added Bug 🐞 and removed External bug 🐞 For bugs which are not caused by DietPi. labels Dec 20, 2019
MichaIng added a commit that referenced this issue Dec 20, 2019
+ DietPi-Software | rTorrent: Update deprecated commands and variables in our default config: #3273
@MichaIng
Copy link
Owner

MichaIng commented Dec 20, 2019

@bbsixzz
Many thanks for your report. If I see right, those commands/config entries are deprecated already for a long time and have now finally been removed? I remember I updated the config once already, but the session_save schedule entry was missed obviously.

... okay there were lots of commands/variables which were not updated to post 0.9.

I hope I got all now: 73b07d1
Might you compare with your config?

Great find and report btw 👍, quite important one!
Changelog: 6f2e51d

@bbsixzz
Copy link
Author

bbsixzz commented Dec 21, 2019

I'm testing your file as it was more complete, everything seems in order.

@MichaIng
Copy link
Owner

MichaIng commented Dec 21, 2019

@bbsixzz
Perfect, many thanks for testing. Since it was not created by me and also quite differs from the official template/example AND I do not 100% understand everything (e.g. how setting a variable save.session can trigger an event), I am open for suggestions about changes. Probably we should even switch to the official template and adjust the base/download/session/log/config paths only?

@bbsixzz
Copy link
Author

bbsixzz commented Dec 21, 2019

@MichaIng
I too had a look at that template and mainly noticed settings like these:

Original config:

throttle.max_uploads.set = 4
throttle.max_downloads.set = 30

New config:

throttle.max_uploads.set = 100
throttle.max_downloads.set (missing, load many simultaneous torrents = crash?)

The new template is modern, but I have no idea what settings would be good for SBC's.

It would be great if someone with rTorrent experience could advise.

@MichaIng
Copy link
Owner

@bbsixzz
Limiting downloads and seeds by default sounds reasonable to me, as long as clients/web UI show reasonable errors when the limit has been reached, so users can adjust, when required.

However yeah long term someone with more experience and regular rTorrent use would be great.

@bbsixzz
Copy link
Author

bbsixzz commented Dec 21, 2019

On a tangent, I noticed torrents appear as Completed after restarting rTorrent.

Out of 43 torrents (all from 3 private trackers) only 3 start seeding automatically!

Two of the Seeding torrents are from the same tracker; doesn't seem to makes any difference.

It's always the same 3 torrents that show Seeding automatically.

If I Start any Completed torrent in ruTorrent it will change to Paused.

If I Stop the same torrent it will show Completed and ALL other torrents will show Seeding.

Finally, if I start the remaining Completed torrent it will show Seeding as well.

If I start multiple Completed torrents they will all show Paused until one of them is stopped.

All OTHER torrents will then show Seeding and I will have to manually start each Paused torrent.

I have no idea what's going on, but I know rTorrent should automatically start seeding.

I commented the following lines in .rtorrent.rc but it didn't change anything:

# Enable the default ratio group.
# ratio.enable=yes
# Change the limits, the defaults should be sufficient.
# Upload to a minimum ratio of 1.01
# ratio.min.set=101
# Upload to a maximum ratio of 1.25
# ratio.max.set=125
# Upload a minimum of x MB
# ratio.upload.set=1M

# When seeding ratio is reached close the torrent
# method.set = group.seeding.ratio.command,d.close=

Bonus: all of those commands are deprecated except maybe group.seeding.ratio.command

Is this making any sense?

It's like rTorrent doesn't know where the files are until a torrent is first started and then stopped!

[EDIT]

If I stop one of the Seeding torrents (after restart) all other torrents change to Seeding immediately.

The stopped torrent becomes Completed and if started again will change to Seeding.

Similar behavior to starting/stopping a Completed torrent, but without the Paused state.

[EDIT]

I removed the 3 torrents that were always seeding only to find 3 other random seeding after restart.

Looking through .rtorrent.rc I noticed the magic number 3 here:

scheduler.max_active.set = 3

I couldn't find any info in the official command list so I just commented the line.

Lo and behold everything works now after restart!

I really think we need to rework .rtorrent.rc in the future, it's a mess.

@MichaIng
Copy link
Owner

MichaIng commented Dec 22, 2019

@bbsixzz
Interesting, this years old issue here states the same: rakshasa/rtorrent#29
This setting seems to never have worked as expected, lol. It should limit the number of seeding torrents, although documentation is missing and name as well not self-explaining.

There is another one: throttle.max_uploads.set
Does this limit in your case the max number of seeding torrents or the max number of uploading connections, e.g. when having multiple peers per seeing torrent.


Ref: https://github.com/rakshasa/rtorrent/wiki/Performance-Tuning#peers-and-slots

  • throttle.max_uploads.global: Max global upload slots
  • throttle.max_uploads: Max upload slots per torrent
  • throttle.max_peers.seed: Max peers per seeding torrent, which should be higher than the number of upload slots, since not every peer is currently transferring data. Lol their template does not match their own recommendations in this point: https://github.com/rakshasa/rtorrent/wiki/CONFIG-Template

Hmm couldn't find a value to limit currently seeding torrents, probably throttle.up.max, e.g. throttle.up.max.set = 3. However I'll remove the above broken one from our config.

Done: e162351

MichaIng added a commit that referenced this issue Dec 22, 2019
+ DietPi-Software | rTorrent: Remove broken setting "scheduler.max_active.set" from our default config: #3273 (comment)
@MichaIng MichaIng changed the title .rtorrent.rc using deprecated commands DietPi-Software | rTorrent: Remove deprecated commands/variables Dec 22, 2019
@MichaIng MichaIng closed this as completed Jan 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug 🐞 Solution available 🥂 Definite solution has been done
Projects
None yet
Development

No branches or pull requests

2 participants