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

conda config #12

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

conda config #12

wants to merge 2 commits into from

Conversation

wadeschulz
Copy link

Added miniconda instructions (for licensing reasons), updated docs, added data/ and results/ to gitignore

@@ -21,16 +21,29 @@ up an issue here on Github or by contacting @cwinsnes.

== Installation and Setup
NOTE: It is recommended to run the module in a separate virtual environment
such as https://www.anaconda.com/[Anaconda] or
such as https://conda.io/projects/conda/en/latest/user-guide/install/linux.html[miniconda] or
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I believe this should link to the Miniconda page rather than the linux install.

https://docs.anaconda.com/free/miniconda/

@@ -0,0 +1,11 @@
name: cm4ai-hpa
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Not entirely sure on this default name. What does cm4ai mean?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

CM4AI is Cell Maps for AI - the name of our DGP within Bridge2AI

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@cwinsnes Sorry that we forgot to inform you! The CM4AI project is a new large collaborative project that the lundberg lab runs with several other collaborators across the U.S. (e.g., also the Ideker lab). The purpose is to build more and better MuSIC maps. We used the HPA densenet in a hackathon that was part of the CM4AI project and @wadeschulz has been incorporating the densenet into an Azure platform for the hackathon. To do that he had to do a few changes and then I encouraged Wade to send these also as a pull request since I thought these could be helpful for other users in the future. Thank you that you replied @cwinsnes and discuss it!

@cwinsnes
Copy link
Member

cwinsnes commented Mar 14, 2024

Thank you for contributing with an easier way of setting up the environment!

Could you expand on the licensing reasons for choosing Miniconda over Anaconda? As far as I can tell, both Anaconda and Miniconda require a commercial license for organizations larger than 200 people, while Conda itself is entirely FOSS.

Specifically, their terms of use cover usage of the "Free tier" (which includes Miniconda, as per the website) software with

You may not use Free Offerings for commercial purposes, including but not limited to external business use, use in organizations over two hundred (200) employees (unless such use is for an Educational Purpose), third-party access to the Cloud Offerings, or Content redistribution or mirroring (each, a “Commercial Purpose”). Using the Free Offerings for a Commercial Purpose requires a Paid Plan with Anaconda.

@wadeschulz
Copy link
Author

My understanding is that it's fine to use Miniconda (a free offering) for internal business use, but have been aware of organizations receiving licensing requests for Anaconda even for internal business use (here is some additional detail, although not clear where the line would be drawn: https://community.anaconda.cloud/t/is-conda-cli-free-for-use/14303). Would also be fine with switching to conda specifically, but I believe a miniconda install with conda cli falls under the BSD license.

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