Skip to content
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

ERROR: stressberry-plot: error: unrecognized arguments: --hide-legend --line-width 2 #82

Open
Samk19 opened this issue Aug 16, 2020 · 4 comments · May be fixed by #111
Open

ERROR: stressberry-plot: error: unrecognized arguments: --hide-legend --line-width 2 #82

Samk19 opened this issue Aug 16, 2020 · 4 comments · May be fixed by #111

Comments

@Samk19
Copy link

Samk19 commented Aug 16, 2020

Getting this Error stressberry-plot: error: unrecognized arguments: --hide-legend --line-width 2

When running command
MPLBACKEND=Agg /home/pi/.local/bin/stressberry-plot mytest.out -f -d 300 -f -l 400 1600 -t 30 90 -o mytest.png --hide-legend --not-transparent --line-width 2

Works when remove —hide-legend and —line-width command but the graph is not clear

@chunyianliew
Copy link

I am running in the same issue.

The two parameters have been removed in this commit when introducing cleanplotlib.
And in the next commit cleanplotlib was replaced by dufte and in that commit the legend label plotting has become hardcoded.

For combined graphs that is not such a big issue, as it will only display the temperature y axis on the left. But when plotting the CPU frequency of one measurement together with the CPU frequency the temperature legend label overlaps the y axis showing the CPU frequency intervals.

flirc_20210117_1300
geekworm_20210117_1300
combined_20210117_1300

@shortbloke
Copy link
Contributor

In my fork of this project I've reinstated --hide-legend and --line-width options https://github.com/shortbloke/stressberry

I've also added a --legacy-style to switch back to the previous type of graph and legend. Finally, I'm experimenting with adding a --color-blind-friendly option that changes the color pallet used to be more easily interpreted by those with colour blindness, though this is limited to use with --legacy-style.

More than happy to do a PR if @nschloe wants to allow the mixing of old and new styles.

@p1r473
Copy link

p1r473 commented Aug 21, 2021

@shortbloke please create a PR!

This was referenced Aug 21, 2021
@shortbloke
Copy link
Contributor

@p1r473 Done. Not sure if it will be accepted.

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 a pull request may close this issue.

4 participants