set_analog_period(20) giving invalid value

[mytechnotalent]

As discussed @microbit-carlos:

When setting:
set_analog_period(20)

I am getting:
ValueError: invalid period

I have tried other values but same result on V2. 12/21 firmware on the v/beta web editor.

[microbit-mark]

micro:bit support #42332

Also raised by Paralax on micro:bit support. This works on V1 but not on V2

[dpgeorge]

Which pin do you set the period on?

Note that the ValueError: invalid period error is just passed on from CODAL.

[mytechnotalent]

@dpgeorge I used pin0 and pin1 both together and separately.

[mytechnotalent]

If they are passed on from CODAL it might be beyond the scope of this repo perhaps @dpgeorge @microbit-carlos?

[dpgeorge]

The issue is that the CODAL does not allow changing the analog period on pins that are not configured in analog-out mode. But I think this is how v1 DAL behaved so I don't believe it's an issue with CODAL.

Should be fixed for MicroPython by 6e0273f

[jaustin]

@mytechnotalent did you test this with the CI build? If so does your thumbs-up mean "works for me"?

[mytechnotalent]

@jaustin I promise to be more explicit next time my apologies again.

I git checkout 6e0273f and build and flashed and what the ValueError is in fact fixed. Tested and good to merge to main.

>>> from microbit import pin1
>>> pin1.set_analog_period(20)
>>> from microbit import pin0
>>> pin0.set_analog_period(20)
>>> 

[microbit-carlos]

Thanks Kevin!
I'll mark this as fixed.

[rbryx]

Hi,
I still have this issue. I try to both pin1.set_analog_period(20) and pin1.set_analog_period_microseconds(20) in both Thonny an Mu and still have the error.
My workaround is to pin1.write_analog(0) before trying to pin1.set_analog_period(20).
I’m using version 2.0.0-beta.5 .

Sorry for this question, I’m new to robotics, should I be able tu use all values (0 to 1023) after setting pin1.set_analog_period(20)? Because it seems to me that I can only use 0 to 300 (or similar), the remaining values produce nothing or odd behavior in kitronik servo motors.

[microbit-carlos]

Are you sure you are getting the same error ValueError: invalid period? If I flash this programme with the latest version in the Python Editor I do not get any errors:

# Add your Python code here. E.g.
from microbit import *

pin1.set_analog_period(20)

Could you provide a minimum programme that throws your error? Maybe it's related to some other parts of the code.

[rbryx]

Sorry, my fault!!!
I was so sure I was using beta-5... three days spent on this... so frustrating!

Again, sorry.