Skip to content

Conversation

@sunyi000
Copy link
Contributor

adding cellpose-sam, v4.0.7 container, with gui and gpu supports. Please review. Thank you

also ping @rmassei

Checklist

  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

@biocontainers-bot
Copy link
Collaborator

No test-cmds.txt (test file) present, skipping tests

@mboudet
Copy link
Contributor

mboudet commented Oct 23, 2025

There is the same issue here that you had in #598 :

  • The nvidia image usage force the user to accept the end user agreement. I don't think biocontainers can accept the license for the end user, and I'm not sure how to forward it to the user itself.
  • You are using a specific (and old) cuda version, which might not work depending on the hardware it's going to run on

I'm not really fond of distributing gpu-support version of software with biocontainers

@biocontainers-bot
Copy link
Collaborator

No test-cmds.txt (test file) present, skipping tests

@sunyi000
Copy link
Contributor Author

thanks @mboudet I removed nvidia cuda related..
it's cpu only now.

@mboudet
Copy link
Contributor

mboudet commented Oct 23, 2025

Alright, should be fine then

@mboudet mboudet merged commit c7ce77e into BioContainers:master Oct 23, 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.

3 participants