Skip to content

conf/marjorie.config: add default Slurm queue#889

Merged
ashotmarg merged 1 commit intonf-core:masterfrom
cbrueffer:marjorie-update
Apr 15, 2025
Merged

conf/marjorie.config: add default Slurm queue#889
ashotmarg merged 1 commit intonf-core:masterfrom
cbrueffer:marjorie-update

Conversation

@cbrueffer
Copy link
Copy Markdown
Member

conf/marjorie.config: add explicit Slurm queue since "interactive" seems to be the default now

@ashotmarg

Copy link
Copy Markdown
Collaborator

@pontus pontus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok I guess assuming it makes sense to have the partition as a user selectable - normally it does not.

See also process.queue as not using clusterOptions might make tuning easier if a user needs to do something.

@cbrueffer
Copy link
Copy Markdown
Member Author

Thanks Pontus, that seems like the better option.

@ashotmarg
Copy link
Copy Markdown
Contributor

Thanks for changing this, I also saw this today and had to change my config after running things on Marjorie after the update.

@ashotmarg ashotmarg merged commit 8afcb06 into nf-core:master Apr 15, 2025
148 checks passed
@cbrueffer cbrueffer deleted the marjorie-update branch April 15, 2025 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants