Skip to content

Conversation

sunyi000
Copy link
Contributor

@sunyi000 sunyi000 commented May 8, 2025

  1. Misc
  • My tool doesn't exist in BioConda
  • The image can be built
  1. Metadata
  • LABEL base_image
  • LABEL version
  • LABEL software.version
  • LABEL about.summary
  • LABEL about.home
  • LABEL about.license
  • MAINTAINER
  1. Extra (optionals)
  • I have written tests in test-cmds.txt
  • LABEL extra.identifier
  • LABEL about.documentation
  • LABEL about.license_file
  • LABEL about.tags

ping @k-dominik

@mboudet
Copy link
Contributor

mboudet commented May 12, 2025

Seems like you are using the gpu version (for the archive), and not the default one (this means that the path in the tests does not work).

I'm not sure about distributing the gpu version. in addition to the issue with distributing a specific version of the cuda toolkit, the ilastik website specifies By downloading and using these builds, you accept the terms and conditions of the [CUDA End User License Agreement (EULA)](https://docs.nvidia.com/cuda/eula/index.html). . I'm not sure that 'biocontainers' can accept the terms for the users.

@sunyi000
Copy link
Contributor Author

Seems like you are using the gpu version (for the archive), and not the default one (this means that the path in the tests does not work).

I'm not sure about distributing the gpu version. in addition to the issue with distributing a specific version of the cuda toolkit, the ilastik website specifies By downloading and using these builds, you accept the terms and conditions of the [CUDA End User License Agreement (EULA)](https://docs.nvidia.com/cuda/eula/index.html). . I'm not sure that 'biocontainers' can accept the terms for the users.

Thanks, I changed it to the CPU version and now it passed all the ci tests.

@mboudet mboudet merged commit eb5588f into BioContainers:master May 13, 2025
1 check passed
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