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

Update minimum RAM for Flye #55

Open
diegomics opened this issue Jan 2, 2024 · 2 comments
Open

Update minimum RAM for Flye #55

diegomics opened this issue Jan 2, 2024 · 2 comments

Comments

@diegomics
Copy link

Hi!
To run, Flye needs to first get all the reads in the memory, therefore, the minimum RAM requirement equals the (uncompressed) reads file size. Would it be possible to modify the RAM to something like 1.2 times the reads file size?
Now it is:

toolshed.g2.bx.psu.edu/repos/bgruening/flye/flye/.*:
  cores: 20
  mem: 60

:)

@bgruening
Copy link
Member

You can define rules, similar to https://github.com/galaxyproject/tpv-shared-database/blob/main/tools.yml#L204

Or this example here: https://github.com/usegalaxy-eu/infrastructure-playbook/blob/master/files/galaxy/tpv/tools.yml#L695

You can also write more complex rules as we have done for hifiasm: https://github.com/usegalaxy-eu/infrastructure-playbook/blob/master/files/galaxy/tpv/tools.yml

@nuwang
Copy link
Member

nuwang commented Feb 10, 2024

Thanks for pointing this out - it'll make a great example for the docs too. A spec like this should work:

toolshed.g2.bx.psu.edu/repos/bgruening/flye/flye/.*:
  cores: 20
  mem: min(input_size*1.2, 256)

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

No branches or pull requests

3 participants