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

adapter tutorials to new framework #357

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

lisa-sousa
Copy link
Collaborator

PR Checklist

  • Documentation in docs is updated

Description of changes

Changed and adapted the end to end selection tutorials to the oligo designer tool suite API. Updated dependency description and added examples for config files.

Technical details

New API is significantly faster and requires significantly less memory. The memory consumption is now limited by a maximum number of entries that is loaded into ram. The db_max_memory defines how many entries are loaded and has to be at least 2*n_job+1 to run without errors. db_max_memory is set internally in the pipeline to the value describes above. If the memory consumption is too high because too many entries are loaded into ram the performance of the program will be significantly decreased and hence its adviced to decrease the number of jobs n_jobs. For a laptop with 8cores and 16GB RAM and values of n_jobs=4 is a good value.

Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

Copy link

Hi @lisa-sousa,

It looks like this pull-request is has been made against the theislab/spapros master or main branch.
The master/main branch should always contain code from the latest release.
Because of this, PRs to master/main are only allowed if they come from any theislab/spapros release or patch branch.

You do not need to close this PR, you can change the target branch to development by clicking the "Edit" button at the top of this page.

Thanks again for your contribution!

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.

1 participant