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

Adds notice that --distributed is only available in beta. Fixes typo. #200

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Tomaz-Vieira
Copy link
Contributor

A user was confused that ilastik would not recognize the --distributed flag, which is still only available on the beta version. This patch adds documentation to reflect that.

Also fixes a typo in the heading.

@k-dominik
Copy link
Contributor

k-dominik commented Apr 19, 2022

the feature is not available in any binary, currently - do you think it makes sense, given that we even in the docs recommend to install mpi4py via pip to make sure it's build correctly for the respective hpc cluster?

This would imply that we provide information on how to create such an installation with mamba/conda...

@Tomaz-Vieira
Copy link
Contributor Author

Hmmm I'm not sure if I follow. Are you suggesting that we shouldn't advertise the feature since we don't provide an installation package for it?

@k-dominik
Copy link
Contributor

Hmmm I'm not sure if I follow. Are you suggesting that we shouldn't advertise the feature since we don't provide an installation package for it?

I suggest we have instructions on there, how to create the environment with mamba/conda + pip, and be clear that it is in fact not enough to use some ilastik binary, no matter what version.

@k-dominik
Copy link
Contributor

Hey @Tomaz-Vieira, this probably fell under the radar, but I think it would be very useful to document, how a user might set up an environment with mpi for their own cluster.

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