Skip to content

Allow customisation of the plot utilities in the realisation module#104

Merged
lispandfound merged 5 commits intomasterfrom
plot_utils_kwargs
May 7, 2025
Merged

Allow customisation of the plot utilities in the realisation module#104
lispandfound merged 5 commits intomasterfrom
plot_utils_kwargs

Conversation

@lispandfound
Copy link
Contributor

The following functions can now be customised to allow different plotting styles:

  • plot_stations
  • plot_sources
  • plot_domain
  • plot_rrup_polygon

This is so researchers can create custom styles for plots in papers.

Copilot AI review requested due to automatic review settings April 30, 2025 22:44
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR enables customisation of plot styling in the realisation module by allowing users to override default plotting parameters using additional keyword arguments.

  • plot_stations, plot_sources, and plot_domain now accept **kwargs for custom styles.
  • plot_rrup_polygon adds optional parameters to customize both the polygon and label styles.

@lispandfound lispandfound requested a review from Copilot May 6, 2025 03:22
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This pull request introduces enhancements to the plot utility functions in the realisation module, allowing researchers to customise the styling of plots in their outputs.

  • Added **kwargs parameters to plot_stations, plot_sources, and plot_domain for user-defined style customisations.
  • Added optional style arguments (rrup_polygon_args and label_args) to plot_rrup_polygon for more flexible plotting of RRup bounding polygons.

@lispandfound lispandfound merged commit f2672cd into master May 7, 2025
5 checks passed
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 this pull request may close these issues.

4 participants