Skip to content

audiobusio.PDMIn.record hangs CPX (SAMD21) if a wav file has been played #4600

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.

Sign up for GitHub

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

Jump to bottom
Closed
kevinjwalters opened this issue Apr 13, 2021 · 2 comments
Closed

audiobusio.PDMIn.record hangs CPX (SAMD21) if a wav file has been played #4600

kevinjwalters opened this issue Apr 13, 2021 · 2 comments
Labels
atmel-samd audio bug
Milestone
Long term

Comments

@kevinjwalters
Copy link

I just noticed this on 6.2.0 on a CPX with fresh (20210413) libraries, mic.record() hangs in a reproducible way under some circumstances. I tried a few things based on my first discovery of this. It looks like playing a wav file to the speaker does it.

Adafruit CircuitPython 6.2.0 on 2021-04-05; Adafruit CircuitPlayground Express with samd21g18
>>>
>>> import board
>>> import audiobusio
>>> import array
>>> mic = audiobusio.PDMIn(board.MICROPHONE_CLOCK, board.MICROPHONE_DATA,
...                        sample_rate=16000, bit_depth=16)
>>> NUM_SAMPLES = 160
>>> samples = array.array('H', [0] * NUM_SAMPLES)
>>> mic.record(samples, len(samples))
160
>>> mic.record(samples, len(samples))
160
>>> mic.record(samples, len(samples))
160
>>> mic.record(samples, len(samples))
160
>>> mic.record(samples, len(samples))
160
>>> from adafruit_circuitplayground import cp
>>> mic.record(samples, len(samples))
160
>>> mic.record(samples, len(samples))
160
>>> dir(cp)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "adafruit_circuitplayground/express.py", line 82, in _unsupported
NotImplementedError: This feature is not supported on Circuit Playground Express.
>>> mic.record(samples, len(samples))
160
>>> mic.record(samples, len(samples))
160
>>> import gc
>>> gc.collect() ; print(gc.mem_free())
12944
>>> gc.collect() ; print(gc.mem_free())
12944
>>> mic.record(samples, len(samples))
160
>>> mic.record(samples, len(samples))
160
>>> cp.play_file("num/one.wav")  ### 16k 16-bit short mono sample
>>> mic.record(samples, len(samples))  ### CIRCUITPY drive no longer works, serial console non-responsive

I see another issue with record() reported in #4261
@tannewt tannewt added this to the Long term milestone Apr 13, 2021
@sta-c0000
Copy link

Confirmed: Circuit Playground Express can now freeze/hang after using mic.record
using: adafruit-circuitpython-circuitplayground_express-en_US-6.2.0.uf2

Also CircuitPlaygroundExpress_SoundMeter.py apparently no longer works for me now because only 32 samples are returned by mic.record() instead of the expected 160 (as explained in bug #4261), even then, after fixing that code for count, it's prone to freeze/hang later with more code, for example, adding this at the end now crashes the CPX when switch is moved to False (doesn't with 5.x):

    if not switch.value: # setup earlier as DigitalInOut(board.D7) with Direction.INPUT Pull.UP
        import supervisor
        supervisor.reload() # attempting to reload main.py

Power cycle or CPX reset button appear to be only way back.

@dhalbert
Copy link
Collaborator

This should be fixed by #5842.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
atmel-samd audio bug
Projects
None yet
Milestone
Long term
Development

No branches or pull requests
4 participants
@tannewt @dhalbert @kevinjwalters @sta-c0000