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

Configureable watchable 'list length' #52

Open
jmgevers99 opened this issue Nov 22, 2024 · 7 comments
Open

Configureable watchable 'list length' #52

jmgevers99 opened this issue Nov 22, 2024 · 7 comments

Comments

@jmgevers99
Copy link

Hi just wanted to show my appreciation for this project!
Its been working wonderfully for me :)

I just noticed the list of torrent this project tracks seems to be limited to 10 torrents, is this correct?
Would be wonderful if this was something we could configure with an environment variable.

@ThijmenGThN
Copy link
Owner

Hoi @jmgevers99 👋🏻

Thank you so much for your kind words—I really appreciate it!

Regarding your concern, I’m not aware of any limiter built into Swaparr. It should be capable of tracking an unlimited number of downloads. If the downloads are marked as "Queued" it usually indicates that your download client has queueing enabled.

Could you double-check this for me? I’d be happy to assist further, but it would be great if you could provide some screenshots to help clarify the situation.

Looking forward to your reply!

Groetjes,

^ Thijmen

@jmgevers99
Copy link
Author

Hey Thijmen,

My example as follows:
afbeelding

afbeelding

As you can see there are many torrents active, but the log only shows 10 torrent at once.
I've tested this with movies and tv-shows.

I hope this clears up your question.

@ThijmenGThN
Copy link
Owner

Oh, I see - well, that's odd.

I'll attempt to replicate this, will report back in a day or two.

Thank you for bringing this to light.

^ Thijmen

@ThijmenGThN
Copy link
Owner

The issue has been identified! It turns out the starrs are using pagination in the queue, which is set to 10 by default.

I’ll work on addressing this over the next few days — expect a release soon!

You’ll be notified here as soon as it’s ready.

^ Thijmen

@ThijmenGThN
Copy link
Owner

Hi @jmgevers99 👋

The default has been set to 256, which I believe should suffice for the current requirements.

If this isn't adequate and you'd prefer an environment variable over the current solution, please don't hesitate to let me know. I'm happy to make that adjustment if necessary.

This change is now included in the latest next release.

Looking forward to your feedback!

^ Thijmen

@jmgevers99
Copy link
Author

Ah wonderful!

I'll be sure to test this in the coming days.
I think 256 is more than adequate, I could think of 1 or 2 scenario's where it might be favourable to have this limit accessible as an environment variable.
For instance if you have a download speed limit setup in your download manager, the service might discard 'inactive' torrents even though they are being limited by the downloader.

If it isn't too much work I think it would add flexibility to have this as an environment variable (with the default set to 256 for instance) :)

@ThijmenGThN
Copy link
Owner

Hi @jmgevers99 👋

I agree, let's do it - I'll keep you posted.

^ Thijmen

@ThijmenGThN ThijmenGThN added this to the 0.10.0 milestone Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants