Skip to content

Constrain spectrum MaxSpur annotations#61

Merged
Arcadia-1 merged 1 commit into
Arcadia-1:mainfrom
chenzc24:codex/fix-spectrum-maxspur-label-bounds
Jun 30, 2026
Merged

Constrain spectrum MaxSpur annotations#61
Arcadia-1 merged 1 commit into
Arcadia-1:mainfrom
chenzc24:codex/fix-spectrum-maxspur-label-bounds

Conversation

@chenzc24

Copy link
Copy Markdown
Contributor

Summary

  • Hide MaxSpur marker/text when the spur is outside the current plot viewport instead of forcing a misleading bottom-edge label.
  • Keep in-range MaxSpur labels inside the axes after caller x/y limit changes, including edge and high-spur cases.
  • Apply the same behavior to the Virtuoso spectrum plotter and clip harmonic labels to the axes.

Tests

  • uv run --with pytest pytest tests/unit/spectrum/test_plot_spectrum.py -q
  • uv run python src/adctoolbox/examples/03_generate_signals/exp_g01_generate_signal_demo.py

@Arcadia-1 Arcadia-1 merged commit 1e7f280 into Arcadia-1:main Jun 30, 2026
1 check 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.

2 participants