Do I have to close the pull request and do a new one with your comments and modifications @glemaitre ?

But in fact, it seems that @glemaitre commit his modification and that's enough no ?

I actually did only comment. I pushed the changes now and trigger the CI to be sure that we don't have issue when running the tutorial.

@adrinjalali The image used in the tutorial used coins. However, we don't install skimage on our CIs. What would be best here to solve the issue of dependencies?

I'm trying to figure out why it doesn't fail on plot_coin_ward_segmentation.py then. We already use skimage in two examples.

Kinda wondering why we're testing the docs without installing skimage on azure. We do install that on circleci which is where we test the docs. So basically, our doc build tests (at least on circleci) already has the scikit-image dependency. @thomasjpfan isn't testing the docs in azure redundant when we test them on circlci already?

I'm trying to figure out why it doesn't fail on plot_coin_ward_segmentation.py then. We already use skimage in two examples.

It is run in CircleCI and we install skimage there.

I am +0 on removing the doc test from azure. They are tested on two instances and takes 30 seconds.

As for this PR, we can install scikit-image in test_docs.sh to get this to pass doctest.

@thomasjpfan I genuinely don't know why we test the docs on Azure. Is it because we want to test them in many platforms?

Is it because we want to test them in many platforms?

This is the only reason I can think of as well. (The azure config is a copy of the original travisci config)

I am concerned with how fragile this is. I'll look into this.

Edit: Posted response in wrong issue.

@nicaogr We finally solved the issue with scikit-image #15065. Could you merge the master branch into your branch such that we run the CIs.

Note that we will need to skip the unsupervised_learning.rst file when scikit-image is not installed and that we test the documentation.

The rendered version looks a bit odd.

I will check and fix it

Indeed, this is already broken in master: https://scikit-learn.org/dev/tutorial/statistical_inference/unsupervised_learning.html

The display of the image is not proper

doc build fails

@adrinjalali it should be ok now. I center the images which take a bit more space than it was intended to but at least this is readable.

Thanks @nicaogr it took a bit of time due to CI but do not hesitate to contribute again.

Thanks a lot. I will contribute again as soon as possible.