Skip to content

Conversation

@smebberson
Copy link
Contributor

This is a simple example showing how to display the current range on an ESP32-S3 based device. A link to this could be placed next to the current link (https://github.com/m5stack/M5Stack/blob/master/examples/Unit/ToF_VL53L0X/ToF_VL53L0X.ino) on https://docs.m5stack.com/en/unit/tof (under the Arduino heading) informing users this is an example that will work on ESP32-S*, ESP32-C* devices.

@GOB52
Copy link
Contributor

GOB52 commented Feb 3, 2025

@smebberson

Thank you for PR.
Can you please set the merge destination to the develop branch?

By the way, the sample you added looks like it could be integrated into PlotToSerial.
I maybe add a screen display section and integrate.

@smebberson smebberson changed the base branch from main to develop February 3, 2025 02:51
@smebberson
Copy link
Contributor Author

@GOB52, I've set the merge destination to be the develop branch :)

Yeah, it probably could be added in there too. I think a super simple example without the PlotToSerial stuff would also be beneficial (as is included in the PR).

Thanks for reviewing.

@GOB52
Copy link
Contributor

GOB52 commented Feb 3, 2025

@smebberson
I made a revised proposal based on your PR #2.
What do you think?

@smebberson
Copy link
Contributor Author

@GOB52

Looks great :) Thank you!

@GOB52
Copy link
Contributor

GOB52 commented Feb 3, 2025

Moving to #2
This PR is closed.

@GOB52 GOB52 closed this Feb 3, 2025
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.

2 participants