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 — Plotting nyquist plot from FrequencyResponseData fails · Issue #992 · python-control/python-control

Plotting nyquist plot from FrequencyResponseData fails #992

New issue
Copy link
New issue
Copy link
Closed
Closed
Plotting nyquist plot from FrequencyResponseData fails#992
Copy link
Labels
bug

Description

@wueestry
wueestry
opened on Apr 16, 2024
Issue body actions

When trying to create the nyquist plot from an FRD system

freq_response = ct.FRD(response, frequencies)
ct.nyquist_plot(freq_response)

the plotting fails with the following error
AttributeError: 'FrequencyResponseData' object has no attribute 'poles' during the call of _default_frequency_range

version: 0.10.0

Activity

  1. slivingston commented on Apr 16, 2024

    @slivingston
    slivingston
    on Apr 16, 2024
    Member
    More actions

    Is it possible to make a Nyquist plot given only frequency response data?

  2. murrayrm commented on Apr 17, 2024

    @murrayrm
    murrayrm
    on Apr 17, 2024
    Member
    More actions

    You can get a valid Nyquist plot from the frequency response in some instances and with some assumptions. Basically, if the open loop dynamics are assumed stable, the zero frequency gain is finite, and the high frequency gain is zero, it should work.

    The error above comes when it tries to compute the default frequency range for the plot, but in the case of frequency response data you don't actually need to do that.

    A quick fix is to change the call to be ct.nyquist_plot(freq_response, freq_response.omega), which bypasses the computation of the default frequency range.

    Reacted by Ryan Wüest
  3. murrayrm
    added
    bug
    on Apr 17, 2024
  4. murrayrm commented on Jun 13, 2024

    @murrayrm
    murrayrm
    on Jun 13, 2024
    Member
    More actions

    Fixed in #996.

  5. murrayrm
    closed this as completedon Jun 13, 2024
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

    bug

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions

      You are visitor 01402503 — 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.