sonar sensor problem(hcsr04) - time_pulse_us

[crouse12]

The following code for hcsr04 can work on microibit v1, but it cannot work on microbit v2.
Python editor version for my test is 2.1.0-beta.7.
I think time_pulse_us or sleep_us has some bugs.

[microbit-carlos]

What's the result you get in a v1 vs v2?

[crouse12]

microbit v1 show distance on led screen.
However, microbit v2 does not show anything.
I found that makecode has the same issue.

microsoft/pxt-microbit#3731

[crouse12]

I find v0.2.23 of codal-microbit-v2 now released.
It solves pulse problem.
Does micropython update, too?

microsoft/pxt-microbit#3731

[dpgeorge]

Yes MicroPython is now updated to codal v0.2.23.

[jaustin]

@crouse12 could you please test with the build from https://github.com/microbit-foundation/micropython-microbit-v2/actions/runs/419939816 (this is an automatically built version from a commit on master, so isn't exactly what the next release will look like, but does have the updated CODAL).

[crouse12]

I test time_pulse_us on microbit v2 using the following two MicroPython Versions.
However, they cannot work.
I think there are something wrong on time_pulse_us.

[jaustin]

@dpgeorge does the patch from @xmeow look okay to you? Would be great to close this off with the next release if so?

[microbit-carlos]

Looks like PR #58 is adding the CODAL method to the MicroPython pin API. I think ultimately what we need is to make sure machine.time_pulse_us works on V2.

MakeCode also had an issue in this area, and the feature was originally implemented in the MakeCode code too, so the functionality has been moved to CODAL.
It'd be good to use CODAL in this case as well to unify the implementation.

[dpgeorge]

machine.time_pulse_us() has been fixed by b7a76e4 . So this function should now work the same as on V1.

It'd be good to use CODAL in this case as well to unify the implementation.

If the built-in MicroPython one works, is it ok to just keep that the way it is?

[jaustin]

I think if it's not a trivial switch over, I'm happy to leave things as they are with the V2 implementation.

[microbit-carlos]

Great, closing this as fixed. 🎉