adafruit / Adafruit_CircuitPython_PyPortal

CircuitPython driver for Adafruit PyPortal.
MIT License
45 stars 56 forks source link

Fixing error, WaveFile moved to audiocore #82

Closed fede2cr closed 3 years ago

fede2cr commented 3 years ago

I was testing the new json library and with this particular branch of circuitpython, but also with the latest from main, i was getting the following error while testing with the PyPortal:

Traceback (most recent call last): File "code.py", line 35, in File "/lib/adafruit_pyportal.py", line 284, in init File "/lib/adafruit_pyportal.py", line 282, in init File "/lib/adafruit_pyportal.py", line 633, in play_file AttributeError: 'module' object has no attribute 'WaveFile'

Checking the docs, it looks like WaveFile moved to audiocore. This couple of line fixes the problem and allow the PyPortal object to be created, and example pyportal code now works ok.

fede2cr commented 3 years ago

Note: This is intended to be applied for a 6.x release. If you use it as is on 5.3.x it does not know the module audiocore.

fede2cr commented 3 years ago

I just added the change to fede2cr:master. I don't see github re-doing the checks after this commit/push. Not sure if I did something wrong, or am I just being impatient with github.

FoamyGuy commented 3 years ago

That is weird I see your new commit/push on your branch, but it's not showing up here.

As far as I can tell you did make that new commit / push correctly. My guess is perhaps github and/or the actions system is having some kind of issue at the moment. Hopefully it will resolve itself shortly.

FoamyGuy commented 3 years ago

I did a bit more testing and it looks like 5.0.0 was the oldest version that this works with. On 4.1.2 it crashes because audiocore does not exist yet.

I think this is okay to merge and release now. When we release it we should probably bump the major version and make a note that it's incompatible with Circuit Python versions below 5.0.0.

I'm going to leave it for now to see if anyone has thoughts or objections but we can merge it soon if there are no concerns.

tannewt commented 3 years ago

Also note, we no longer make 4.x bundles so folks won't get this version that way.