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 singularity definitions #59

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

Conversation

dadrasarmin
Copy link

  1. Added a Singularity definition file for Mikado. I could not one on this repo.
  2. Changed the Singularity definition files for Minos.tools and Busco. With these changes the singularity can be build via remote builder or on a VM via a mac machine.
  3. Changed the Augustus path in minos_config.yaml, due to the changes in paths of Augustus installation in Singularity.busco.def.

I could not build an executive file with the previous definition file on remote builder or mac os. I guess there are some dependencies on a specific linux distribution. Be careful, this definition will install the latest version from bioconda. Please change the augustus_config_data in minos_config.yaml to /opt/conda/config
Change the augustus configuration directory to work well with changes in singularity definitions.
A singularity definition to build mikado and its dependencies.
I could not build a singularity container based on the previous definition via a mac or remote builder. I ran a successful test with the container build by this new definition.
@gemygk
Copy link
Collaborator

gemygk commented Jul 22, 2022

Thanks @dadrasarmin for PR. Much appreciated!

From a quick look, I can see that you have changed the BUSCO version and we have not tested it yet as we have seen issues where the BUSCO database format and directory structures are different between releases. I will have a detailed look later (mostly next week)

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.

None yet

2 participants