Skip to content

Allow prefix in config file #13

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

endrebak
Copy link
Contributor

@endrebak endrebak commented Dec 5, 2019

This allows users to run multiple analyses from the folder in which the Snakefile is located and the results from each will be put in different folder. They can also use the prefix to use another disk for storing the results with prefix='/mnt/different_folder'.

This allows users to run different analyses from the folder
in which the Snakefile is located.
@wdecoster
Copy link
Owner

Good idea, but I wouldn't create a new index for each {prefix}... because that's a slow step which also takes quite some disk space.
Can we keep those without {prefix}?

@endrebak
Copy link
Contributor Author

endrebak commented Dec 5, 2019

If the files are different, you need to make different indexes, no? But I see your point.

If you want to avoid recomputation based on the names (by seeing that {sample}.index already exists) problems happen if people use the same names between analyses.

This is completely up to you :) I do not care either way. I can scrap the PR.

@endrebak
Copy link
Contributor Author

endrebak commented Dec 5, 2019

We could think about this some more. No need to take a stance now :)

@wdecoster
Copy link
Owner

Sure, no rush for this one! If users reuse the sample name for different samples the that's on them ;-)

The current implementation also allows to analyze multiple regions by listing multiple regions in the config.yaml:
image

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.

2 participants