-
Notifications
You must be signed in to change notification settings - Fork 8
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
Params to change Memory and CPU #30
Comments
Ahh! Good question! I started fixing this but got busy with other work. The dev branch has lower requirements, and I have a PR to bump the version so I could make a PR to master. If you have 5 min, could you review #27. Then I will make another PR to master which will fix the problem. |
Chris, i created a commit task on #27 , i don't know how to do it in another
way, sorry!
…On Tue, Mar 11, 2025 at 6:44 PM Chris Hakkaart ***@***.***> wrote:
Ahh! Good question! I started fixing this but got busy with other work.
The dev branch has lower requirements, and I have a PR to bump the version
so I could make a PR to master.
If you have 5 min, could you review #27
<#27>. Then I will make another PR to
master which will fix the problem.
—
Reply to this email directly, view it on GitHub
<#30 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/BGUNSFJMXGWN35GOA4MIGS32T5KKHAVCNFSM6AAAAABY2AB3BKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOMJVG43TENRSG4>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
[image: christopher-hakkaart]*christopher-hakkaart* left a comment
(nf-core/demo#30)
<#30 (comment)>
Ahh! Good question! I started fixing this but got busy with other work.
The dev branch has lower requirements, and I have a PR to bump the version
so I could make a PR to master.
If you have 5 min, could you review #27
<#27>. Then I will make another PR to
master which will fix the problem.
—
Reply to this email directly, view it on GitHub
<#30 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/BGUNSFJMXGWN35GOA4MIGS32T5KKHAVCNFSM6AAAAABY2AB3BKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDOMJVG43TENRSG4>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
No worries. The improvement has already been merged into the I need to merge the version bump into |
Great news! Waiting the new version 🤝 |
Description of feature
Hi Chris,
I am using demo pipeline to introduce students to Nextflow. So, i run one test using Github Codespace (from Nextflow Training) and that was fine to run -profile test. But, when i try to run the example from pipeline Demo site, there is a error both from memory and cpus limitations.
I think that if we put some params to limitate the amount this will run fine.
What you think about that?
The text was updated successfully, but these errors were encountered: