Docker image "macosm1-p2p-image" throwing 'sklearn' error about stratospherelinuxips

[eldraco]

It is valid. Just check that maybe the p2p docker was not updated. check the requirements of the latest m1 without p2p.

[roshnaeem]

@eldraco, i am replacing sklearn with scikit-learn on this line. I tried removing the line too. I tried removing sklearn in requirements.txt as well.
However, it keeps giving me the same error, do i need to update something else too or change it somewhere else too? It is not somehow reading my changes in the file and still trying to install sklearn.

[eldraco]

So do a PR to develop with the change, even if your build fails ok?

[eldraco]

Ah yes, this is because in our deployment strategy, the Docker image is always built from master branch, and therefore if you only change the develop branch, it will still pull from master, and your changes are not there yet.
So to do this:
I need to update the master branch first
then I can pull and build the docker

So from your PR is good if you only modify the file as you did

[roshnaeem]

Done @eldraco :)))Please check

[AlyaGomaa]

fixed here #470