search

DrMarc / slab

python classes for working with sounds and conducting psychoacoustic experiments
MIT License
22 stars 7 forks source link

SoundCard as default requirement #37

Closed DrMarc closed 3 years ago

DrMarc commented 3 years ago

->> soundcard seems like a major functionality point of the library. from the docs i would expect that it is installed by default, but it's not and there isn't a notice given to the user that it hasn't been found. I assumed it was being used until i tried to import it manually. I would recommend adding it as a default or else as a clear optional dependency, and warn the user if it can't be found if it's the expected default behavior.

DrMarc commented 3 years ago

@OleBialas, SoundCard caused issues with testing on GitHub, if I remember correctly. Perhaps we could re-add it to the requirements and uninstall it in the GitHub workflow.