Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Added labels to legend #7453

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
WilhelmWiens wants to merge 3 commits into arduino:master from WilhelmWiens:serialPlotterLabels

Conversation

Copy link
Contributor

@WilhelmWiens WilhelmWiens commented Apr 10, 2018

Supports CSV style header and colon separated :.
It is enough to send the label once they are kept until a new label is send.
Multiply values should be separated by space character. : : ...
If can send values only or mixed width labels like this:
: : ...
If you send one value you have to send al values. It will corrupt the visualisation.
!!! NOT ALLOWED !!! : : ...
Labels can be set without sending data:
: : ...

When using the CSV header style you just send space separated. OBS number only headers do not work. they are only allowed with colon separated style.

Copy link
Member

@ArduinoBot build this please

@facchinm facchinm added Component: IDE user interface The Arduino IDE's user interface IDE 1.9.x Beta Related to the Arduino IDE Beta Build labels Apr 13, 2018
Copy link

jottr commented Mar 3, 2019

@facchinm will this be merged at some point?

@facchinm facchinm added this to the Release 1.8.9 milestone Mar 4, 2019
Copy link
Member

facchinm commented Mar 4, 2019

I think we can merge it, but some documentation must be added too

Copy link
Collaborator

per1234 commented Mar 4, 2019
edited
Loading

@facchinm I'm guessing you mean user-targeted documentation rather than just comments in the source code. I completely agree that it's important do document this feature, otherwise the users will not know it's there.

There is some that mentions Serial Plotter, but I don't consider any of this to be a substitute for documentation of the plotter itself:

Built-in examples

Library examples

Library reference

Hardware

Other

I feel that adding a demonstration of the label feature to either of those built-in examples might overly complicate them since they're in the "Basics" section.

It would be more appropriate in the library examples, but that is hiding the label documentation pretty deeply.

I have thought it would be good to provide some formal documentation of the Serial Plotter (arduino/docs-content#668). Where do you envision the documentation of the new label functionality being added? I was thinking the IDE guide was the best place for it, but perhaps the more detailed documentation of the Serial Plotter (allowed delimiters, labels) will be too much for that page and deserves a page of its own, which would be linked to from the summary description of the Serial Plotter on the IDE guide page.

I'm happy to help with this. I would be limited to providing suggested text for the documentation, which would need to be implemented by someone else, since I don't have edit access to the documentation content. Even so, that would take care of some of the job.

facchinm and s-light reacted with heart emoji

Copy link
Member

facchinm commented Mar 4, 2019

Serial plotter surely needs some love, both from the "inline" documentation and from the website doc. We should coordinate with @SimonePDA to find the right spot where these documentation should be added but I really like the way you want to tackle it!

Copy link
Contributor Author

Hi
I have uploaded a small protocol description for the label functionality.
WilhelmWiens@cdd66b6

Copy link
Contributor Author

The legend is right now only shown when there is more than one data set. Should I change that so it is always shown ?

Copy link
Contributor Author

Hi
Thank you for the feedback.
One way to get an arm build is to clone the repository to your am bord and run it from there, but i have newer tried it by myself.

I think it is an good idear to add '|' sign as additional part seperator.

I think the arduino team has do decide what diretion the serialPlotter should take.

Open PR on SerialPlotter
Open issues on SerialPlotter

Regards

Copy link
Collaborator

per1234 commented Mar 19, 2019

Where can I find an ARM Build to test?

@rin67630 assuming you mean 32-bit ARM Linux, it's the last link in the list of builds provided by ArduinoBot:
#7453 (comment)

The only changes made to this PR since that build are the addition of documentation files, so it is functionally identical.

Copy link

I think it is an good idear to add '|' sign as additional part seperator.

It currently works well with " | " (pipe sign between spaces), no need to write additional code.
If possible, ignoring what is between /* ... */ would have been more useful.

Copy link
Member

Rebased and manually merged, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Component: IDE user interface The Arduino IDE's user interface IDE 1.9.x Beta Related to the Arduino IDE Beta Build SerialPlotter Tools > Serial Plotter
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AltStyle によって変換されたページ (->オリジナル) /