-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
[logalizer] rename plot to logplotter for clarity #1226
Conversation
👍 from me! 😄 |
@gautierhattenberger what do you think? |
Can we put a placeholder in that place instead, that prints out an error indicating the name change and what needs to be done? |
Done, if launched via the old |
@gautierhattenberger any comments? |
Fine for me |
[logalizer] rename plot to logplotter for clarity Was quite unclear that plot refers to the log plotter and plotter to the realtime plotter... Hence rename the plot executable to logplotter. If launched via the old plot it will print that plot was renamed to logplotter and then launch logplotter.
@flixr the usage message needs to be updated apropo:
It still says |
Thx, fixed. |
Was quite unclear that
plot
refers to the log plotter andplotter
to the realtime plotter...Hence rename the
plot
executable tologplotter