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 — Remove redundant _isstatic calculation in xferfcn · Issue #1090 · python-control/python-control

Remove redundant _isstatic calculation in xferfcn #1090

New issue
Copy link
New issue
Copy link
Closed
#1117
Closed
Remove redundant _isstatic calculation in xferfcn#1090
#1117
Copy link

Description

@murrayrm
murrayrm
opened on Jan 4, 2025
Issue body actions

... there is already an _isstatic method that is also short-circuiting. Can we use the same code for both - perhaps by moving the code to a separate function that tests num and den, and a method that calls it?

Originally posted by @sawyerbfuller in #1082 (comment)

Activity

  1. murrayrm
    linked a pull request that will close this issueupdate use/computation of sys._isstatic() #1117on Feb 2, 2025
  2. murrayrm
    changed the title [-]Use _isstatic in xferfcn [title needs updating][/-] [+]Remove redundant _isstatic calculation in xferfcn[/+] on Feb 2, 2025
  3. murrayrm
    closed this as completedin #1117on Feb 8, 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

    No type

    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.