search

Jellevanderwerff / thebeat

thebeat: Rhythms in Python for science
https://thebeat.readthedocs.io
GNU General Public License v3.0
19 stars 1 forks source link

CI fails on Mac OS, lilypond error #82

Open Jellevanderwerff opened 2 months ago

Jellevanderwerff commented 2 months ago 
WARNING: thebeat 0.1.dev1+gf63e405 does not provide the extra 'music-notation'
INFO: pip is looking at multiple versions of thebeat[music-notation] to determine which version is compatible with other requirements. This could take a while.
ERROR: Ignored the following versions that require a different python version: 1.11.0 Requires-Python <3.13,>=3.9; 1.11.0rc1 Requires-Python <3.13,>=3.9; 1.11.0rc2 Requires-Python <3.13,>=3.9; 1.11.1 Requires-Python <3.13,>=3.9; 1.11.2 Requires-Python <3.13,>=3.9; 1.11.3 Requires-Python <3.13,>=3.9; 1.11.4 Requires-Python >=3.9; 1.12.0 Requires-Python >=3.9; 1.12.0rc1 Requires-Python >=3.9; 1.12.0rc2 Requires-Python >=3.9; 1.13.0 Requires-Python >=3.9; 1.13.0rc1 Requires-Python >=3.9; 1.25.0 Requires-Python >=3.9; 1.25.0rc1 Requires-Python >=3.9; 1.25.1 Requires-Python >=3.9; 1.25.2 Requires-Python >=3.9; 1.26.0 Requires-Python <3.13,>=3.9; 1.26.0b1 Requires-Python <3.13,>=3.9; 1.26.0rc1 Requires-Python <3.13,>=3.9; 1.26.1 Requires-Python <3.13,>=3.9; 1.26.2 Requires-Python >=3.9; 1.26.3 Requires-Python >=3.9; 1.26.4 Requires-Python >=3.9; 2.0.0b1 Requires-Python >=3.9; 2.0.0rc1 Requires-Python >=3.9; 2.1.0 Requires-Python >=3.9; 2.1.0rc0 Requires-Python >=3.9; 2.1.1 Requires-Python >=3.9; 2.1.2 Requires-Python >=3.9; 2.1.3 Requires-Python >=3.9; 2.1.4 Requires-Python >=3.9; 2.2.0 Requires-Python >=3.9; 2.2.0rc0 Requires-Python >=3.9; 2.2.1 Requires-Python >=3.9; 2.2.2 Requires-Python >=3.9; 3.8.0 Requires-Python >=3.9; 3.8.0rc1 Requires-Python >=3.9; 3.8.1 Requires-Python >=3.9; 3.8.2 Requires-Python >=3.9; 3.8.3 Requires-Python >=3.9; 3.8.4 Requires-Python >=3.9; 3.9.0rc2 Requires-Python >=3.9
ERROR: Could not find a version that satisfies the requirement lilypond; extra == "music_notation" (from thebeat[music-notation]) (from versions: none)
ERROR: No matching distribution found for lilypond; extra == "music_notation"

Also check out the warning, is that a typo of "music-notation"?

YannickJadoul commented 2 months ago

macOS tests seem to be running on Apple silicon (ARM64), while Lilypond only provides Intel 64-bit wheels. Seems Lilypond only provides x86_64 binaries, indeed, but I guess they should run?

I'll check out further, and maybe ping the lilypond wheels repo on releasing "universal" wheels for macOS...

YannickJadoul commented 2 months ago

Fixed the tests in cefa9d9f386602db198b7a8a1183e3fc3cf1443a

And I opened https://gitlab.com/jeanas/lilypond-wheels/-/issues/1 to see if we can fix the upstream wheel names! Let's wait a couple more days, to see if that issue moves. If not, we just keep using macos-13 images for now.

YannickJadoul commented 2 months ago

Also check out the warning, is that a typo of "music-notation"?

It seems that extra's should contain dashes instead of underscores: see https://peps.python.org/pep-0685/ and https://packaging.python.org/en/latest/specifications/name-normalization/

As far as I can tell, these warnings (https://github.com/pypa/pip/issues/12431) just got fixed in pip, but maybe better still rename the extra to use - instead of _?