GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — Regression: root-locus plots may be coarse, and no longer react to zoom events · Issue #1151 · python-control/python-control

Regression: root-locus plots may be coarse, and no longer react to zoom events #1151

New issue
Copy link
New issue
Copy link
Closed
Bug
#1153
Closed
Regression: root-locus plots may be coarse, and no longer react to zoom events#1151
Bug
#1153
Copy link

Description

@repagh
repagh
opened on Jun 4, 2025
Issue body actions

Root-locus (and sisotool root-locus) used to recompute their points when zooming in on the root-locus plot (typically to get to the dominant pole group). At zoom-in, the new xlim and ylim are used to recompute the RL lines.

Currently root-locus does not recompute, and zoomed-in plots show (sometimes unusably) coarse edgy root-locuses. The recompute was removed in commit

57dac87da3c5885ef4912fa4484a0dff811e0b9c    Initial refactoring of root_locus_{map,plot}

I am currently trying to figure out how to bring recalculation of the plot back.

Example:

import control.matlab as ml
from matplotlib import pyplot as plt

s = ml.tf('s')
H = (s+1)/s**2/(s**2+2*s+7)/(s+15)
ml.rlocus(H)
plt.show()

Image

Activity

  1. murrayrm commented on Jun 5, 2025

    @murrayrm
    murrayrm
    on Jun 5, 2025
    Member
    More actions

    Thanks for pointing this out @repagh. I remember getting rid of that function because I wasn't able to get it to work with the new structure. In particular, when we separated out the computation of the response from the plotting of that response, it didn't seem easy to go back and generate more detail. But there may be a way to have a response function have a callback to regenerate the plot with new resolution, then pass that information on via the response object so that a plotting function can do this same.

    This might be a good time to think about interactive plots in general and start putting in some generic mechanisms. I think it should always be possible to hover/clock on the graph and have something useful happen (eg, hovering or clicking on a point on a Bode plot would show the frequency, magnitude, and phase at that point).

  2. repagh
    added the
    Bug
    issue type on Jun 6, 2025
  3. sawyerbfuller commented on Jun 10, 2025

    @sawyerbfuller
    sawyerbfuller
    on Jun 10, 2025
    Contributor
    More actions

    Showing where the calculated points are when you hover over them in interactive mode would be particularly helpful in root locus plots because you can only click on the points.

  4. murrayrm
    closed this as completedin #1153on Jun 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    Bug

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      You are visitor 01402520 — GeezerVison does not store the pages you look up.
      Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.