Unification of plot functions#509
Unification of plot functions#509JuleAdrian wants to merge 13 commits intoPedestrianDynamics:mainfrom
Conversation
…plot density, speed and acceleration
…ents marker and marker_size
Codecov Report✅ All modified and coverable lines are covered by tests. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
ThoChat
left a comment
ThoChat
left a comment
There was a problem hiding this comment.
Everything looks good, just minor cosmetic changes are required for the user guide.
| "cell_type": "markdown", | ||
| "metadata": {}, | ||
| "source": [ | ||
| "### Note:\n", |
There was a problem hiding this comment.
The "note" format should be change to match the other one used in the User guide: ":::{note}".
| "source": [ | ||
| "### Note:\n", | ||
| "\n", | ||
| "For all functions that plot time series, it is also possible to plot the time in Seconds on the x-axis instead of the frame. You can choose this option with the keyword argument x_axis. Then, also a framerate needs to be defined as in the following example:" |
There was a problem hiding this comment.
For all functions that plot time series, it is also possible to plot the time in seconds on the x-axis instead of the frame. You can choose this option with the keyword argument x_axis. Then, a framerate also needs to be defined, as in the following example:
Minor changes: lowercase for "second" and change of place of the "also" in the last sentence.
2 participants
Closes #466
Closes #449
Most of the plot_functions were adjusted. Now, we have similar options in all plot_functions.
As a default, the title is empty
The following options can be set for all plot_funcitons: title, x_label, y_label, line width and color of borders, measurement_areas_, measurement_lines, lines that are ploted
When timeseries are plotted, you have the option to either plot frame or time at the x-axis
all plot_functions are used in the user_guide notebook