Bug 1836744

Summary:	Current version is not importable
Product:	[Fedora] Fedora	Reporter:	Elliott Sales de Andrade <quantum.analyst>
Component:	python-scikit-learn	Assignee:	Sergio Pascual <sergio.pasra>
Status:	CLOSED RAWHIDE	QA Contact:	Fedora Extras Quality Assurance <extras-qa>
Severity:	unspecified	Docs Contact:
Priority:	unspecified
Version:	rawhide	CC:	besser82, igor.raits, lupinix.fedora, sergio.pasra
Target Milestone:	---
Target Release:	---
Hardware:	Unspecified
OS:	Unspecified
Whiteboard:
Fixed In Version:		Doc Type:	If docs needed, set a value
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2020-06-29 11:03:53 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	---	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	1837757
Bug Blocks:	1837013, 1838447, 1841732

Description Elliott Sales de Andrade 2020-05-18 03:22:40 UTC

See this build https://koji.fedoraproject.org/koji/taskinfo?taskID=44638804 which fails to import sklearn:

tests/mixin_tests.py:1: in <module>
    from geoplot.geoplot import (Plot, HueMixin, ScaleMixin, ClipMixin, LegendMixin, webmap)
../../BUILDROOT/python-geoplot-0.4.1-1.fc33.noarch/usr/lib/python3.8/site-packages/geoplot/__init__.py:1: in <module>
    from .geoplot import (
../../BUILDROOT/python-geoplot-0.4.1-1.fc33.noarch/usr/lib/python3.8/site-packages/geoplot/geoplot.py:18: in <module>
    import mapclassify as mc
/usr/lib/python3.8/site-packages/mapclassify/__init__.py:4: in <module>
    from .classifiers import (
/usr/lib/python3.8/site-packages/mapclassify/classifiers.py:8: in <module>
    from sklearn.cluster import KMeans as KMEANS
/usr/lib/python3.8/site-packages/sklearn/cluster/__init__.py:6: in <module>
    from ._spectral import spectral_clustering, SpectralClustering
/usr/lib/python3.8/site-packages/sklearn/cluster/_spectral.py:18: in <module>
    from ._kmeans import k_means
/usr/lib/python3.8/site-packages/sklearn/cluster/_kmeans.py:34: in <module>
    from ._k_means_lloyd import lloyd_iter_chunked_dense
sklearn/cluster/_k_means_lloyd.pyx:11: in init sklearn.cluster._k_means_lloyd
    ???
E   ModuleNotFoundError: No module named 'threadpoolctl'


Version-Release number of selected component (if applicable):
python3-scikit-learn            armv7hl  0.23.0-1.fc33

Comment 1 Sergio Pascual 2020-05-19 22:16:33 UTC

Oh, great. I missed the dependency in this new package threadpoolctl, which is not in Fedora yet. I have created a review request for it

Comment 2 Sergio Pascual 2020-05-24 22:38:06 UTC

New package, please check


https://koji.fedoraproject.org/koji/buildinfo?buildID=1510955

Comment 3 Miro Hrončok 2020-05-25 13:02:11 UTC

This comment is mass posted to all bugs blocking the Python 3.9 tracker, sorry if it is not 100 % relevant. When in doubt, please ask.


The Python 3.9 rebuild is in progress in a Koji side tag.

If you fix this bug, please don't rebuild the package in regular rawhide, but do it in the side tag with:

    $ fedpkg build --target=f33-python

The rebuild is progressing slowly and it is possible this package won't have all the required build dependencies yet. If that's the case, please just leave the fix committed and pushed and we will eventually rebuild it for you.

You are not asked to go and try rebuild all the missing dependencies yourself. If you know there is a bootstrap loop in the dependencies, let me know and we can untangle it together.

If you want to test your fix or reproduce the failure, you can still use the Copr repo mentioned in the initial comment of this bug: https://copr.fedorainfracloud.org/coprs/g/python/python3.9/

Comment 4 Miro Hrončok 2020-05-29 07:12:00 UTC

Python 3.9 update: The f33-python side tag is currently being merged.

New builds in f33-python are no longer possible, but python3 is not yet updated to Python 3.9 in rawhide. You can check when Python is Python 3.9 with:

    $ koji wait-repo f33-build --build python3.9-3.9.0~b1-3.fc3

And build the packages normally after that.