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

Fix installation on Python 3.8 due to missing Tensorflow-io wheel #759

Merged
merged 1 commit into from Jan 4, 2024

Conversation

juhoinkinen
Copy link
Member

@juhoinkinen juhoinkinen commented Jan 4, 2024

TensorFlow IO v0.35.0 was released in December 19th. That release in PyPI does not provide many platform wheels that were provided by v0.34.0. See the tracking issue tensorflow/io#1906.

Now Annif installation with NN ensemble backend fails on Python 3.8 (there were some failed CI runs).

We may want to drop support for Python 3.8 for Annif v1.1, but for the meantime, this fixes the installation by making TensorFlow IO a direct dependency for Python 3.8 and pins its version to 0.34.*.

Copy link

sonarcloud bot commented Jan 4, 2024

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@juhoinkinen juhoinkinen changed the title Fix installation on Python 3.8 due to missing ensorflow-io wheel Fix installation on Python 3.8 due to missing Tensorflow-io wheel Jan 4, 2024
Copy link

codecov bot commented Jan 4, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (0348b28) 99.67% compared to head (84f46fb) 99.67%.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #759   +/-   ##
=======================================
  Coverage   99.67%   99.67%           
=======================================
  Files          89       89           
  Lines        6404     6404           
=======================================
  Hits         6383     6383           
  Misses         21       21           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@juhoinkinen juhoinkinen added bug dependencies Pull requests that update a dependency file labels Jan 4, 2024
@juhoinkinen juhoinkinen added this to the 1.1 milestone Jan 4, 2024
@juhoinkinen juhoinkinen marked this pull request as ready for review January 4, 2024 18:08
@juhoinkinen juhoinkinen merged commit 8bef843 into main Jan 4, 2024
17 checks passed
@juhoinkinen juhoinkinen deleted the pin-tf-io-version branch January 4, 2024 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant