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

If I try to do stereo without having seewave installed, I should get a nice error. #2

Open
tlevine opened this issue Jun 12, 2013 · 1 comment

Comments

@tlevine
Copy link
Contributor

tlevine commented Jun 12, 2013

This is what I get now. I would like a nicer error that explains that I need seewave for stereo.

  sequence(list(piano$A2), list(beat.dynamics[1:100]), stereo = T)
Loading required package: seewave
  |======================================================================================================================================================| 100%
Warning message:
In library(package, lib.loc = lib.loc, character.only = TRUE, logical.return = TRUE,  :
  there is no package called ‘seewave’
Calls: sequence ... tryCatchList -> tryCatchOne -> doTryCatch -> library
@abelsonlive
Copy link
Contributor

Am just adding a dependency to seewave and removing the stereo arg. All the built-in waves are stereo for now, so it shouldn't be a big issue. Eventually there should be a test for whether a wave is stereo.

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

No branches or pull requests

2 participants