Skip to content

Add serial monitor settings on Home panel. #17

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.

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

Conversation

rudeb0t
Copy link
Collaborator

@rudeb0t rudeb0t commented Dec 31, 2024

Re #15

@thelastoutpostworkshop
Copy link
Owner

Good Job!

@thelastoutpostworkshop thelastoutpostworkshop merged commit 78eab28 into thelastoutpostworkshop:main Dec 31, 2024
@rudeb0t rudeb0t deleted the serial-monitor-settings-panel branch January 1, 2025 13:35
@thelastoutpostworkshop
Copy link
Owner

@rudeb0t I was wondering if you see a use case where the port to upload and the port in serial monitoring could be different ?
I was thinking of having the port for upload and serial monitor to be the same,

@rudeb0t
Copy link
Collaborator Author

rudeb0t commented Jan 3, 2025

@rudeb0t I was wondering if you see a use case where the port to upload and the port in serial monitoring could be different ? I was thinking of having the port for upload and serial monitor to be the same,

On macOS this is the case. /dev/cu.* is where you upload and /dev/tty.* is where you connect to the serial port. Initially I had it set up to use the same port /dev/cu.SLAB_USBtoUART for both upload and serial monitor. Whenever I upload a sketch, the serial monitor fails to connect to the port saying it's not connected to the device. After a bit of searching, I found this where it says /dev/cu.* and /dev/tty.* are different things. The subtle difference is /dev/cu.* does not wait for CDC and this causes the vscode serial monitor extension to fail to connect to it saying the device is not connected. The /dev/cu.* ports are also not listed in the dropdown in the extension:
image
And so even though it's more work, I added a separate serial monitor port field.

@thelastoutpostworkshop
Copy link
Owner

That makes sense, thanks for the explanation.

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