Skip to content

Delegate external axes methods with guardrails #514

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

Merged
cvanelteren merged 3 commits into from
Jan 27, 2026
Merged

Delegate external axes methods with guardrails #514

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
ae40011
Delegate external axes methods with guardrails
cvanelteren Jan 27, 2026
085c629
Merge branch 'main' into external-axes-delegation
cvanelteren Jan 27, 2026
a210780
Simplify external axes delegation
cvanelteren Jan 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
30 changes: 12 additions & 18 deletions ultraplot/axes/container.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -61,6 +61,14 @@ class ExternalAxesContainer(CartesianAxes):
``external_padding=2`` or ``external_padding=0`` to disable padding entirely.
"""

_EXTERNAL_DELEGATE_BLOCKLIST = {
# Keep UltraPlot formatting/guide behaviors on the container.
"format",
"colorbar",
"legend",
"set_title",
}

def __init__(
self, *args, external_axes_class=None, external_axes_kwargs=None, **kwargs
):
Expand Down Expand Up @@ -799,28 +807,14 @@ def get_tightbbox(self, renderer, *args, **kwargs):

def __getattr__(self, name):
"""
Delegate attribute access to the external axes when not found on container.

This allows the container to act as a transparent wrapper, forwarding
plotting methods and other attributes to the external axes.
Delegate missing attributes to the external axes unless blocked.
"""
# Avoid infinite recursion for private attributes
# But allow parent class lookups during initialization
if name.startswith("_"):
# During initialization, let parent class handle private attributes
# This prevents interfering with parent class setup
if name in self._EXTERNAL_DELEGATE_BLOCKLIST:
raise AttributeError(
f"'{type(self).__name__}' object has no attribute '{name}'"
)

# Try to get from external axes if it exists
if hasattr(self, "_external_axes") and self._external_axes is not None:
try:
return getattr(self._external_axes, name)
except AttributeError:
pass

# Not found anywhere
if self._external_axes is not None:
return getattr(self._external_axes, name)
raise AttributeError(
f"'{type(self).__name__}' object has no attribute '{name}'"
)
Expand Down
Loading