Skip to content

Moved audioio.WaveFile -> audiocore.WaveFile #1374

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

Merged
merged 4 commits into from
Jan 13, 2021
Merged

Moved audioio.WaveFile -> audiocore.WaveFile #1374

merged 4 commits into from
Jan 13, 2021

Conversation

evaherrada
Copy link
Collaborator

Also moved audioio.RawSample -> audiocore.RawSample

Fixes #1342

@evaherrada evaherrada requested a review from tannewt December 17, 2020 15:31
Copy link
Member

@ladyada ladyada left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

scanned thru, didnt test

@evaherrada
Copy link
Collaborator Author

I haven't ever done a whole lot with audio, so there were a lot of projects that I couldn't test since I didn't have the hardware, but here's what I did test:

  • Circuit_Playground_Express_Rocket_Lamp
  • Make_It_Talk (wasn't initially working, although I don't think this had anything to do with the audioio to audiocore switch)

If you'd like me to test more, I can see about grabbing a crickit and a propmaker featherwing since a majority of the projects I modified use one of those.

@ladyada
Copy link
Member

ladyada commented Jan 12, 2021

im ok with that, @tannewt any thoughts?

@dhalbert
Copy link
Contributor

I was going to submit exactly the same fix for a single one of these, then discovered this PR. Very happy this is being done.

Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One question. Everything else looks good.

Copy link
Member

@tannewt tannewt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Needs a merge

@evaherrada evaherrada merged commit 602ac4d into adafruit:master Jan 13, 2021
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.

audioio.WaveFile -> audiocore.WaveFile
4 participants