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.

Sign up for GitHub

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

Jump to bottom

Adding setting title functionality to AnimationPlotter #919

Merged
merged 6 commits into from
Jan 8, 2024
Merged

Adding setting title functionality to AnimationPlotter #919

merged 6 commits into from
Jan 8, 2024

Conversation

jswright-dstl
Copy link
Contributor

@jswright-dstl jswright-dstl commented Jan 4, 2024 •
edited
Loading

This PR adds the functionality to set the title of an AnimationPlotter plot.
This solves Issue #858.
Usage:

plotter = AnimationPlotter(title="Plot Name")
plotter.plot_ground_truths(truth, [0, 2])
plotter.run()

@jswright-dstl
Add plot title capability to AnimationPlotter
14919dc 
The plot title can be set for the animation in the run() method;
plotter.run(plot_title="Plot Title")
@jswright-dstl
Add test for AnimationPlotter plot_title
e412e3b
@jswright-dstl
Add AnimationPlotter plot_title test for no title
1ec22fd
@jswright-dstl jswright-dstl requested a review from a team as a code owner January 4, 2024 16:22
@jswright-dstl jswright-dstl requested review from hpritchett-dstl, orosoman-dstl and sdhiscocks and removed request for a team, hpritchett-dstl and orosoman-dstl January 4, 2024 16:22
sdhiscocks
sdhiscocks reviewed Jan 5, 2024
View reviewed changes
stonesoup/plotter.py Outdated Show resolved Hide resolved
@sdhiscocks sdhiscocks merged commit 249ab46 into main Jan 8, 2024
6 checks passed
@sdhiscocks sdhiscocks deleted the animationplotter branch January 8, 2024 13:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mharris-dstl mharris-dstl mharris-dstl approved these changes

@sdhiscocks sdhiscocks sdhiscocks approved these changes

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@jswright-dstl @sdhiscocks @mharris-dstl