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

workflow: nextflow script #13

Open
joschrew opened this issue Aug 8, 2022 · 1 comment
Open

workflow: nextflow script #13

joschrew opened this issue Aug 8, 2022 · 1 comment

Comments

@joschrew
Copy link
Collaborator

joschrew commented Aug 8, 2022

maybe test param.venv in nextflow-scripts. venv should not be provided by the user it should be hidden to the user as the user doesn't necessarily have to deal with it

@MehmedGIT
Copy link
Collaborator

MehmedGIT commented Aug 9, 2022

Agree. The user should not know the param.venv. However, the venv requirement comes from the OCR-D software itself. When installing the software locally it is still required to activate the venv (reference). I am not sure if there is an easy way to go around that.

Edit: Btw, param.venv path can be left as an empty string in the Nextflow script. When executing a Nextflow instance, we can set the proper venv path as a parameter.

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

2 participants