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

Test audresample==1.3.0 #309

Closed
wants to merge 4 commits into from
Closed

Test audresample==1.3.0 #309

wants to merge 4 commits into from

Conversation

hagenw
Copy link
Member

@hagenw hagenw commented May 30, 2023

Test new audresample with dedicated wheel packages.

@hagenw
Copy link
Member Author

hagenw commented May 30, 2023

It is working as expected for Linux and Windows:

image

image

But it does not find the wheel for MacOS, but installs from sources:

image

@hagenw
Copy link
Member Author

hagenw commented May 30, 2023

Before it was always installing the universal wheel as expected:

image

image

image

@hagenw
Copy link
Member Author

hagenw commented May 30, 2023

The reason it did not install the dedicated wheel for Mac OS is that it looks for a wheel compatible with Mac OS version 10, so it seems we need to provide packages for version 10 and 11 as well.

@hagenw
Copy link
Member Author

hagenw commented May 30, 2023

We have updated the supported MacOS version in audeering/audresample#44, but still experience an installation from sources here:

image

Even though it is listing the actual tag of the wheel we published as supported tag:

image

@hagenw
Copy link
Member Author

hagenw commented May 30, 2023

We still used wrong tags when publishing the wheels, which is fixed in audeering/audresample#46

@hagenw
Copy link
Member Author

hagenw commented May 30, 2023

Now we are also installing the wheel under MacOS:

image

@hagenw
Copy link
Member Author

hagenw commented May 30, 2023

Everything is now working as expected and we can install audresample from it's corresponding wheels and find the library we need when importing it.

@hagenw hagenw closed this May 30, 2023
@hagenw hagenw deleted the test-audresample branch May 3, 2024 11:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant