-
Notifications
You must be signed in to change notification settings - Fork 22
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
AMPCOMBI/PASRSETABLE default resources very underpowered #451
Comments
U still want to bump that? As I said it depends on the dataset size. 40 is too much for normal datasets also. |
What's a "normal" average size then and what would be the dataset size recommended for 40 CPUs (do you mean the wastewater one that James is running?)? |
Yes we should probably bump it. 6 works for our tiny fake dataset, 40 needed for a huge thing, so somewhere in the middle. Maybe 16? |
Im testing it always with two of our samples using 12 and thats enough |
Enough in what way? |
Let's agree on this before 2.1 release? I don't have much experience with this module's requirements. I would rather choose a slightly higher value in favor of module not crashing. Let's go for 16? |
Solved by #454 |
Description of the bug
Currently set to 6 CPUs, whereas recommendation from @Darcy220606 is at least 40
Command used and terminal output
Relevant files
No response
System information
No response
The text was updated successfully, but these errors were encountered: