Skip to content

Use the latest "pluggable monitor" gRPC APIs #769

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

Closed
fstasi opened this issue Jan 24, 2022 · 8 comments
Closed

Use the latest "pluggable monitor" gRPC APIs #769

fstasi opened this issue Jan 24, 2022 · 8 comments
Assignees
Labels
conclusion: resolved Issue was resolved topic: CLI Related to Arduino CLI topic: code Related to content of the project itself topic: serial monitor Related to the Serial Monitor type: enhancement Proposed improvement

Comments

@fstasi
Copy link
Contributor

fstasi commented Jan 24, 2022

The IDE2 needs to use the latest pluggable monitor gRPC APIs, rather than the legacy ones, which are deprecated

@ghost ghost assigned silvanocerza Jan 24, 2022
@fstasi fstasi transferred this issue from arduino/arduino-cli Jan 24, 2022
@per1234 per1234 changed the title IDE 2 has to use the latest "pluggable monitor" gRPC APIs (was Pluggable monitor IDE-2) Use the latest "pluggable monitor" gRPC APIs Jan 24, 2022
@per1234 per1234 added topic: code Related to content of the project itself topic: serial monitor Related to the Serial Monitor type: enhancement Proposed improvement topic: CLI Related to Arduino CLI labels Jan 24, 2022
@PaulStoffregen
Copy link

Nightly build from Feb 2, 2022 is not running my pluggable monitor tool.

screenshot1

It does work when run with arduino-cli from command line

screenshot2

Here's my config info, just in case

screenshot3

Also just in case, here's a peek at the JSON from my pluggable discovery tool for this case

screenshot4

@fstasi
Copy link
Contributor Author

fstasi commented Feb 2, 2022

@PaulStoffregen for the IDE2 to fully support your pluggable monitor we need to tackle a couple of issues

  • update IDE2 to CLI 0.21.0 (that's not released yet)
  • start using the new pluggable monitor APIs (as described by this very issue)

Expect updated on these 👆 anytime soon. Keeping you posted

@PaulStoffregen
Copy link

Thanks! Looking forward to trying the new IDE when it's updated.

@PaulStoffregen
Copy link

Hoping we can start testing IDE 2.0 nightly with pluggable serial monitor soon.

@fstasi fstasi assigned umbynos and unassigned silvanocerza Mar 21, 2022
@fstasi fstasi modified the milestones: arduino-ide rc5, arduino-ide rc6 Mar 24, 2022
@fstasi fstasi assigned AlbyIanna and unassigned umbynos Apr 1, 2022
@PaulStoffregen
Copy link

@AlbyIanna - Any chance for pluggable monitor to be implemented by RC7?

@AlbyIanna
Copy link
Contributor

AlbyIanna commented May 2, 2022

Hi @PaulStoffregen! Sadly, this was delayed for multiple reasons, the main one being the complexity of the change (a pretty big refactor was necessary). BUT we're working on it! I can't make promises, but I believe it will be working soon.

If you're curious, this is the branch we're working on: https://github.com/arduino/arduino-ide/tree/pluggable-monitor

@PaulStoffregen
Copy link

@AlbyIanna - Glad it's happening. Do you have a Teensy board? If not and you want one for testing, just say the word and I can send you one. As far as I know, Teensy is the only 3rd party board (so far) using the new pluggable monitor API.

@AlbyIanna
Copy link
Contributor

Thank you so much @PaulStoffregen 😄 but I believe @ubidefeo can lend me one of his Teensy boards for testing!

@fstasi fstasi removed this from the arduino-ide rc6 milestone May 9, 2022
@fstasi fstasi self-assigned this May 10, 2022
@AlbyIanna AlbyIanna mentioned this issue Jun 7, 2022
4 tasks
@fstasi fstasi closed this as completed Jun 8, 2022
@per1234 per1234 added the conclusion: resolved Issue was resolved label Jun 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
conclusion: resolved Issue was resolved topic: CLI Related to Arduino CLI topic: code Related to content of the project itself topic: serial monitor Related to the Serial Monitor type: enhancement Proposed improvement
Projects
None yet
Development

No branches or pull requests

6 participants