-
Notifications
You must be signed in to change notification settings - Fork 49
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
Specify a limit for the amount of simultaneous threads #370
Comments
Temporary workaround: |
This issue is caused by too many requests being made simultaneously, and is sometimes a result of a flaky internet connection. Instead of adding a delay, it should also be possible to decrease the number of threads created (75 by default). Line 122 in 340470c
When downloading mods: ferium/src/subcommands/upgrade.rs Line 76 in 340470c
|
Can you add an option to change that hardcoded 75 value as a command line flag in the next version? |
Same issue here; it's made the whole program useless. How did you apply that temporary workaround? Did you recompile after editing the code? |
Nevermind, I figured it out! First time compiling something on rust :) |
Can we expect a patched release soon? I installed using Homebrew and integrated it into automation and therefore can only use a release available from Homebrew. |
I managed to build from source and limit the threads both in the source files and using the "-t" attribute, but am still getting the error ONLY when using "ferium upgrade." Exact command: Are there any solutions to this? It can't even download one mod at a time. Listing mods with verbose works, but upgrading does not. EDIT: I found that the current limit is 64 mods (ironic) from Modrinth. I'm not sure how to implement the cooldown, and limiting the threads does not work for me. I am on macOS (I know, calm down- it's what I have). |
are you sure you're using the compiled build? your comment only shows "ferium", without any path. Which means (not sure about mac, but on linux) it will just use the one from your system path, i.e. from your package manager. |
False. It only started happening for me on about 120 mods iirc. |
For me, it was 64 mods, tested on two computers, both running macOS Ventura, one using the latest released version of ferium, and the other using a current compiled version. |
I compiled it using Homebrew. I verified that it was using the latest version and modified thread limit I put in place of 25 directly in the code. |
Acquire a semaphore permit while executing each download task, instead of while _creating_ each download task. - Fixes gorilla-devs#370. - Also prevents a deadlock when downloading more than 75 mods.
Description
When upgrading a ton of modrinth mods, this error occurs. This only started to occur today, it was working fine yesterday.
Steps to reproduce
Operating System
Linux
Are you using the latest version of ferium?
Additional Information
Command output:
The text was updated successfully, but these errors were encountered: