Repository navigation
DOC: avoid czt example aspect warning - #25569
Conversation
github-actions
Bot
added
scipy.signal
Documentation
|
Following up on this small documentation-only fix after two weeks. The rendered-docs, docs-build, refguide, SciPy-build, and benchmark checks are green, and the branch is conflict-free. Is the change still useful, or is there anything I should adjust to help review? Thanks. |
|
@AMBRA7592, take a look at the updated plot after your changes: and compare to the plot before your change: See the difference? |
|
Yes, I see it: the reference arc was drawn on the lower semicircle while the sampled CZT points lie on the upper semicircle. That also expanded the automatic y-range and changed the focused semicircle plot into a square full-circle composition. I corrected the Arc to theta1=0, theta2=180 in 639bd23. A local Agg render now retains the original x/y limits and emits no warnings. Thanks for catching this. |
Reference issue
Closes gh-25532.
What does this implement/fix?
This updates the
scipy.signal.cztdocstring example so the plot no longer triggers Matplotlib's fixed-limit/equal-aspect message during docs builds.The example now draws the upper unit-circle reference with
matplotlib.patches.Arc, creates the axes explicitly withfig, ax = plt.subplots(), and uses an equal aspect ratio with only the x limits fixed.Additional information
Validation run locally:
git diff --checkpython3 -m py_compile scipy/signal/_czt.pyAI Generation Disclosure
OpenAI Codex was used to help prepare this PR, including the modified
scipy.signal.cztdocstring example and the PR description. I reviewed the final diff and validation results before submitting.