René's URL Explorer Experiment


Title: pyflakes (pylint, ruff, ....) clean-up, CI? · Issue #1101 · python-control/python-control · GitHub

Open Graph Title: pyflakes (pylint, ruff, ....) clean-up, CI? · Issue #1101 · python-control/python-control

X Title: pyflakes (pylint, ruff, ....) clean-up, CI? · Issue #1101 · python-control/python-control

Description: Running pyflakes on main HEAD (0ff0452) reveals lots of possible issues; see end. Is there interest in getting and keeping the tree lint-clean? This would require some changes to style, e.g., dropping from foo import * (see output below)...

Open Graph Description: Running pyflakes on main HEAD (0ff0452) reveals lots of possible issues; see end. Is there interest in getting and keeping the tree lint-clean? This would require some changes to style, e.g., dropp...

X Description: Running pyflakes on main HEAD (0ff0452) reveals lots of possible issues; see end. Is there interest in getting and keeping the tree lint-clean? This would require some changes to style, e.g., dropp...

Opengraph URL: https://github.com/python-control/python-control/issues/1101

X: @github

direct link

Domain: github.com


Hey, it has json ld scripts:
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"pyflakes (pylint, ruff, ....) clean-up, CI?","articleBody":"Running pyflakes on main HEAD (0ff045263f) reveals lots of possible issues; see end.\n\n1. Is there interest in getting and keeping the tree lint-clean?\n\n    This would require some changes to style, e.g., dropping `from foo import *` (see output below), and some patience when the linter spits out an incomprehensible message.\n \n2. Are there strong preferences for a particular linter?\n\n    I use pyflakes often: it's simple, has a low false-positive rate, but is not configurable.\n\n    The other two I'm aware of:\n      - pylint, more extensive checks, configurable\n      - ruff; my crude, probably unfair thinking of it is \"pylint written in Rust\", but I think it may do more\n\n\npyflakes output:\n\n```\ncontrol/modelsimp.py:343:9: local variable 'order' is assigned to but never used\ncontrol/pzmap.py:14:1: 'math.pi' imported but unused\ncontrol/pzmap.py:18:1: 'numpy.cos' imported but unused\ncontrol/pzmap.py:18:1: 'numpy.exp' imported but unused\ncontrol/pzmap.py:18:1: 'numpy.linspace' imported but unused\ncontrol/pzmap.py:18:1: 'numpy.sin' imported but unused\ncontrol/pzmap.py:18:1: 'numpy.sqrt' imported but unused\ncontrol/pzmap.py:25:1: '.freqplot._freqplot_defaults' imported but unused\ncontrol/pzmap.py:28:1: '.lti.LTI' imported but unused\ncontrol/stochsys.py:25:1: '.iosys.InputOutputSystem' imported but unused\ncontrol/stochsys.py:25:1: '.iosys._process_indices' imported but unused\ncontrol/stochsys.py:29:1: redefinition of unused 'StateSpace' from line 23\ncontrol/stochsys.py:173:9: local variable 'NN' is assigned to but never used\ncontrol/stochsys.py:292:9: local variable 'NN' is assigned to but never used\ncontrol/stochsys.py:449:13: undefined name 'warnings'\ncontrol/stochsys.py:463:9: 'raise NotImplemented' should be 'raise NotImplementedError'\ncontrol/iosys.py:11:1: 'warnings.warn' imported but unused\ncontrol/nlsys.py:21:1: 'copy' imported but unused\ncontrol/nlsys.py:28:1: '.iosys._process_signal_list' imported but unused\ncontrol/nlsys.py:214:9: local variable 'dt' is assigned to but never used\ncontrol/nlsys.py:246:9: local variable 'dt' is assigned to but never used\ncontrol/nlsys.py:814:16: f-string is missing placeholders\ncontrol/nlsys.py:834:16: f-string is missing placeholders\ncontrol/nlsys.py:2477:5: '.statesp._convert_to_statespace' imported but unused\ncontrol/nlsys.py:2478:5: '.xferfcn.TransferFunction' imported but unused\ncontrol/nlsys.py:2540:12: f-string is missing placeholders\ncontrol/nlsys.py:2579:16: f-string is missing placeholders\ncontrol/nlsys.py:2642:24: f-string is missing placeholders\ncontrol/nlsys.py:2668:16: f-string is missing placeholders\ncontrol/nlsys.py:2745:24: f-string is missing placeholders\ncontrol/mateqn.py:40:1: 'numpy.copy' imported but unused\ncontrol/mateqn.py:40:1: 'numpy.dot' imported but unused\ncontrol/mateqn.py:40:1: 'numpy.atleast_2d' imported but unused\ncontrol/statesp.py:19:1: 'copy.deepcopy' imported but unused\ncontrol/statesp.py:25:1: 'numpy.array' imported but unused\ncontrol/statesp.py:25:1: 'numpy.ones' imported but unused\ncontrol/statesp.py:35:1: '.iosys._process_dt_keyword' imported but unused\ncontrol/statesp.py:35:1: '.iosys.iosys_repr' imported but unused\ncontrol/statesp.py:35:1: '.iosys.isdtime' imported but unused\ncontrol/nichols.py:24:1: '.freqplot._default_frequency_range' imported but unused\ncontrol/nichols.py:24:1: '.freqplot._freqplot_defaults' imported but unused\ncontrol/nichols.py:140:29: undefined name 'idx_sys'\ncontrol/phaseplot.py:41:1: '.exception.ControlNotImplemented' imported but unused\ncontrol/phaseplot.py:165:5: local variable 'passed_kwargs' is assigned to but never used\ncontrol/phaseplot.py:633:14: local variable 'single_color' is assigned to but never used\ncontrol/phaseplot.py:1094:13: f-string is missing placeholders\ncontrol/phaseplot.py:1096:19: undefined name 'ControlArgument'\ncontrol/phaseplot.py:1096:35: f-string is missing placeholders\ncontrol/phaseplot.py:1261:9: local variable 'xy' is assigned to but never used\ncontrol/phaseplot.py:1268:9: local variable 'bp' is assigned to but never used\ncontrol/statefbk.py:49:1: '.exception.ControlNotImplemented' imported but unused\ncontrol/statefbk.py:53:1: '.mateqn._check_shape' imported but unused\ncontrol/statefbk.py:55:1: '.statesp._convert_to_statespace' imported but unused\ncontrol/statefbk.py:278:5: local variable 'b' is assigned to but never used\ncontrol/timeresp.py:1841:5: '.lti.LTI' imported but unused\ncontrol/freqplot.py:14:1: 'os.path.commonprefix' imported but unused\ncontrol/freqplot.py:763:17: local variable 'mag_ylim' is assigned to but never used\ncontrol/freqplot.py:769:17: local variable 'phase_ylim' is assigned to but never used\ncontrol/freqplot.py:2219:38: undefined name 'arg'\ncontrol/__init__.py:74:1: 'from .iosys import *' used; unable to detect undefined names\ncontrol/__init__.py:75:1: 'from .nlsys import *' used; unable to detect undefined names\ncontrol/__init__.py:76:1: 'from .lti import *' used; unable to detect undefined names\ncontrol/__init__.py:77:1: 'from .statesp import *' used; unable to detect undefined names\ncontrol/__init__.py:78:1: 'from .xferfcn import *' used; unable to detect undefined names\ncontrol/__init__.py:79:1: 'from .frdata import *' used; unable to detect undefined names\ncontrol/__init__.py:82:1: 'from .timeresp import *' used; unable to detect undefined names\ncontrol/__init__.py:83:1: 'from .timeplot import *' used; unable to detect undefined names\ncontrol/__init__.py:85:1: 'from .bdalg import *' used; unable to detect undefined names\ncontrol/__init__.py:86:1: 'from .ctrlplot import *' used; unable to detect undefined names\ncontrol/__init__.py:87:1: 'from .delay import *' used; unable to detect undefined names\ncontrol/__init__.py:88:1: 'from .descfcn import *' used; unable to detect undefined names\ncontrol/__init__.py:89:1: 'from .dtime import *' used; unable to detect undefined names\ncontrol/__init__.py:90:1: 'from .freqplot import *' used; unable to detect undefined names\ncontrol/__init__.py:91:1: 'from .margins import *' used; unable to detect undefined names\ncontrol/__init__.py:92:1: 'from .mateqn import *' used; unable to detect undefined names\ncontrol/__init__.py:93:1: 'from .modelsimp import *' used; unable to detect undefined names\ncontrol/__init__.py:94:1: 'from .nichols import *' used; unable to detect undefined names\ncontrol/__init__.py:95:1: 'from .phaseplot import *' used; unable to detect undefined names\ncontrol/__init__.py:96:1: 'from .pzmap import *' used; unable to detect undefined names\ncontrol/__init__.py:97:1: 'from .rlocus import *' used; unable to detect undefined names\ncontrol/__init__.py:98:1: 'from .statefbk import *' used; unable to detect undefined names\ncontrol/__init__.py:99:1: 'from .stochsys import *' used; unable to detect undefined names\ncontrol/__init__.py:100:1: 'from .ctrlutil import *' used; unable to detect undefined names\ncontrol/__init__.py:101:1: 'from .canonical import *' used; unable to detect undefined names\ncontrol/__init__.py:102:1: 'from .robust import *' used; unable to detect undefined names\ncontrol/__init__.py:103:1: 'from .config import *' used; unable to detect undefined names\ncontrol/__init__.py:104:1: 'from .sisotool import *' used; unable to detect undefined names\ncontrol/__init__.py:105:1: 'from .passivity import *' used; unable to detect undefined names\ncontrol/__init__.py:106:1: 'from .sysnorm import *' used; unable to detect undefined names\ncontrol/__init__.py:109:1: '.phaseplot' imported but unused\ncontrol/__init__.py:110:1: '.phaseplot as pp' imported but unused\ncontrol/__init__.py:113:1: 'from .exception import *' used; unable to detect undefined names\ncontrol/__init__.py:124:13: 'ss' may be undefined, or defined from star imports: .bdalg, .canonical, .config, .ctrlplot, .ctrlutil, .delay, .descfcn, .dtime, .exception, .frdata, .freqplot, .iosys, .lti, .margins, .mateqn, .modelsimp, .nichols, .nlsys, .passivity, .phaseplot, .pzmap, .rlocus, .robust, .sisotool, .statefbk, .statesp, .stochsys, .sysnorm, .timeplot, .timeresp, .xferfcn\ncontrol/__init__.py:124:1: 'LTI' may be undefined, or defined from star imports: .bdalg, .canonical, .config, .ctrlplot, .ctrlutil, .delay, .descfcn, .dtime, .exception, .frdata, .freqplot, .iosys, .lti, .margins, .mateqn, .modelsimp, .nichols, .nlsys, .passivity, .phaseplot, .pzmap, .rlocus, .robust, .sisotool, .statefbk, .statesp, .stochsys, .sysnorm, .timeplot, .timeresp, .xferfcn\ncontrol/__init__.py:125:13: 'tf' may be undefined, or defined from star imports: .bdalg, .canonical, .config, .ctrlplot, .ctrlutil, .delay, .descfcn, .dtime, .exception, .frdata, .freqplot, .iosys, .lti, .margins, .mateqn, .modelsimp, .nichols, .nlsys, .passivity, .phaseplot, .pzmap, .rlocus, .robust, .sisotool, .statefbk, .statesp, .stochsys, .sysnorm, .timeplot, .timeresp, .xferfcn\ncontrol/__init__.py:125:1: 'LTI' may be undefined, or defined from star imports: .bdalg, .canonical, .config, .ctrlplot, .ctrlutil, .delay, .descfcn, .dtime, .exception, .frdata, .freqplot, .iosys, .lti, .margins, .mateqn, .modelsimp, .nichols, .nlsys, .passivity, .phaseplot, .pzmap, .rlocus, .robust, .sisotool, .statefbk, .statesp, .stochsys, .sysnorm, .timeplot, .timeresp, .xferfcn\ncontrol/__init__.py:126:17: 'bode_plot' may be undefined, or defined from star imports: .bdalg, .canonical, .config, .ctrlplot, .ctrlutil, .delay, .descfcn, .dtime, .exception, .frdata, .freqplot, .iosys, .lti, .margins, .mateqn, .modelsimp, .nichols, .nlsys, .passivity, .phaseplot, .pzmap, .rlocus, .robust, .sisotool, .statefbk, .statesp, .stochsys, .sysnorm, .timeplot, .timeresp, .xferfcn\ncontrol/__init__.py:126:1: 'LTI' may be undefined, or defined from star imports: .bdalg, .canonical, .config, .ctrlplot, .ctrlutil, .delay, .descfcn, .dtime, .exception, .frdata, .freqplot, .iosys, .lti, .margins, .mateqn, .modelsimp, .nichols, .nlsys, .passivity, .phaseplot, .pzmap, .rlocus, .robust, .sisotool, .statefbk, .statesp, .stochsys, .sysnorm, .timeplot, .timeresp, .xferfcn\ncontrol/__init__.py:127:20: 'nyquist_plot' may be undefined, or defined from star imports: .bdalg, .canonical, .config, .ctrlplot, .ctrlutil, .delay, .descfcn, .dtime, .exception, .frdata, .freqplot, .iosys, .lti, .margins, .mateqn, .modelsimp, .nichols, .nlsys, .passivity, .phaseplot, .pzmap, .rlocus, .robust, .sisotool, .statefbk, .statesp, .stochsys, .sysnorm, .timeplot, .timeresp, .xferfcn\ncontrol/__init__.py:127:1: 'LTI' may be undefined, or defined from star imports: .bdalg, .canonical, .config, .ctrlplot, .ctrlutil, .delay, .descfcn, .dtime, .exception, .frdata, .freqplot, .iosys, .lti, .margins, .mateqn, .modelsimp, .nichols, .nlsys, .passivity, .phaseplot, .pzmap, .rlocus, .robust, .sisotool, .statefbk, .statesp, .stochsys, .sysnorm, .timeplot, .timeresp, .xferfcn\ncontrol/__init__.py:128:20: 'nichols_plot' may be undefined, or defined from star imports: .bdalg, .canonical, .config, .ctrlplot, .ctrlutil, .delay, .descfcn, .dtime, .exception, .frdata, .freqplot, .iosys, .lti, .margins, .mateqn, .modelsimp, .nichols, .nlsys, .passivity, .phaseplot, .pzmap, .rlocus, .robust, .sisotool, .statefbk, .statesp, .stochsys, .sysnorm, .timeplot, .timeresp, .xferfcn\ncontrol/__init__.py:128:1: 'LTI' may be undefined, or defined from star imports: .bdalg, .canonical, .config, .ctrlplot, .ctrlutil, .delay, .descfcn, .dtime, .exception, .frdata, .freqplot, .iosys, .lti, .margins, .mateqn, .modelsimp, .nichols, .nlsys, .passivity, .phaseplot, .pzmap, .rlocus, .robust, .sisotool, .statefbk, .statesp, .stochsys, .sysnorm, .timeplot, .timeresp, .xferfcn\ncontrol/__init__.py:131:1: 'reset_defaults' may be undefined, or defined from star imports: .bdalg, .canonical, .config, .ctrlplot, .ctrlutil, .delay, .descfcn, .dtime, .exception, .frdata, .freqplot, .iosys, .lti, .margins, .mateqn, .modelsimp, .nichols, .nlsys, .passivity, .phaseplot, .pzmap, .rlocus, .robust, .sisotool, .statefbk, .statesp, .stochsys, .sysnorm, .timeplot, .timeresp, .xferfcn\ncontrol/rlocus.py:19:1: 'functools.partial' imported but unused\ncontrol/rlocus.py:21:1: 'matplotlib.pyplot as plt' imported but unused\ncontrol/rlocus.py:24:1: 'numpy.array' imported but unused\ncontrol/rlocus.py:24:1: 'numpy.imag' imported but unused\ncontrol/rlocus.py:24:1: 'numpy.real' imported but unused\ncontrol/rlocus.py:29:1: '.iosys.isdtime' imported but unused\ncontrol/rlocus.py:465:40: undefined name 'prevrow'\ncontrol/rlocus.py:467:32: undefined name 'prevrow'\ncontrol/rlocus.py:471:9: local variable 'prevrow' is assigned to but never used\ncontrol/lti.py:10:1: 'numpy.angle' imported but unused\ncontrol/canonical.py:11:1: 'numpy.zeros' imported but unused\ncontrol/canonical.py:11:1: 'numpy.shape' imported but unused\ncontrol/canonical.py:11:1: 'numpy.iscomplex' imported but unused\ncontrol/canonical.py:11:1: 'numpy.vstack' imported but unused\ncontrol/canonical.py:11:1: 'numpy.hstack' imported but unused\ncontrol/canonical.py:11:1: 'numpy.empty' imported but unused\ncontrol/canonical.py:11:1: 'numpy.finfo' imported but unused\ncontrol/canonical.py:11:1: 'numpy.float64' imported but unused\ncontrol/canonical.py:13:1: 'numpy.linalg.eig' imported but unused\ncontrol/dtime.py:51:1: '.statesp.StateSpace' imported but unused\ncontrol/sysnorm.py:15:1: 'scipy as sp' imported but unused\ncontrol/xferfcn.py:25:1: 'numpy.angle' imported but unused\ncontrol/xferfcn.py:25:1: 'numpy.float64' imported but unused\ncontrol/xferfcn.py:25:1: 'numpy.pi' imported but unused\ncontrol/xferfcn.py:34:1: '.iosys.isdtime' imported but unused\ncontrol/xferfcn.py:221:17: import 'poly' from line 25 shadowed by loop variable\ncontrol/xferfcn.py:1242:21: import 'poly' from line 25 shadowed by loop variable\ncontrol/xferfcn.py:1462:5: local variable 'kwargs' is assigned to but never used\ncontrol/frdata.py:18:1: 'numpy.angle' imported but unused\ncontrol/frdata.py:18:1: 'numpy.where' imported but unused\ncontrol/optimal.py:26:1: '.exception.ControlNotImplemented' imported but unused\ncontrol/optimal.py:27:1: '.iosys._process_indices' imported but unused\ncontrol/optimal.py:166:56: undefined name 'method'\ncontrol/optimal.py:1109:26: undefined name 'optimal_methods'\ncontrol/timeplot.py:14:1: 'matplotlib as mpl' imported but unused\ncontrol/timeplot.py:181:5: '.iosys.InputOutputSystem' imported but unused\ncontrol/timeplot.py:182:5: '.timeresp.TimeResponseData' imported but unused\ncontrol/exception.py:74:13: 'slycot' imported but unused\ncontrol/exception.py:88:13: 'pandas' imported but unused\ncontrol/exception.py:101:13: 'cvxopt' imported but unused\ncontrol/robust.py:45:1: 'from .exception import *' used; unable to detect undefined names\ncontrol/robust.py:47:1: 'from .statefbk import *' used; unable to detect undefined names\ncontrol/robust.py:106:5: local variable 'dico' is assigned to but never used\ncontrol/robust.py:111:15: 'ControlSlycot' may be undefined, or defined from star imports: .exception, .statefbk\ncontrol/robust.py:194:5: local variable 'dico' is assigned to but never used\ncontrol/robust.py:199:15: 'ControlSlycot' may be undefined, or defined from star imports: .exception, .statefbk\ncontrol/tests/timeplot_test.py:10:1: 'control.tests.conftest.mplcleanup' imported but unused\ncontrol/tests/timeplot_test.py:318:9: local variable 'combresp' is assigned to but never used\ncontrol/tests/timeplot_test.py:424:13: local variable 'shape' is assigned to but never used\ncontrol/tests/descfcn_test.py:213:9: local variable 'bias' is assigned to but never used\ncontrol/tests/descfcn_test.py:239:9: local variable 'cplt' is assigned to but never used\ncontrol/tests/bspline_test.py:14:1: 'scipy as sp' imported but unused\ncontrol/tests/bspline_test.py:18:1: 'control.optimal as opt' imported but unused\ncontrol/tests/bspline_test.py:221:9: local variable 'basis' is assigned to but never used\ncontrol/tests/matlab_test.py:139:9: local variable 'sys1' is assigned to but never used\ncontrol/tests/matlab_test.py:149:9: local variable 'sys1' is assigned to but never used\ncontrol/tests/matlab_test.py:159:9: local variable 'sys1' is assigned to but never used\ncontrol/tests/matlab_test.py:588:9: local variable 'unwrapped' is assigned to but never used\ncontrol/tests/xferfcn_test.py:1125:9: local variable 'array' is assigned to but never used\ncontrol/tests/trdata_test.py:217:9: local variable 'response_bad_kw' is assigned to but never used\ncontrol/tests/freqplot_test.py:10:1: 'control.tests.conftest.editsdefaults' imported but unused\ncontrol/tests/freqplot_test.py:10:1: 'control.tests.conftest.slycotonly' imported but unused\ncontrol/tests/freqplot_test.py:169:5: local variable 'lines_default' is assigned to but never used\ncontrol/tests/frd_test.py:6:1: 'sys as pysys' imported but unused\ncontrol/tests/frd_test.py:16:1: 'control.evalfr' imported but unused\ncontrol/tests/frd_test.py:186:9: local variable 'h2' is assigned to but never used\ncontrol/tests/frd_test.py:320:9: local variable 'chkpts' is assigned to but never used\ncontrol/tests/frd_test.py:343:13: local variable 'sys' is assigned to but never used\ncontrol/tests/frd_test.py:360:13: local variable 'sys' is assigned to but never used\ncontrol/tests/frd_test.py:532:13: local variable 'sys' is assigned to but never used\ncontrol/tests/nyquist_test.py:439:9: local variable 'response' is assigned to but never used\ncontrol/tests/nyquist_test.py:515:5: local variable 'cplt' is assigned to but never used\ncontrol/tests/nyquist_test.py:551:5: redefinition of unused 'indentsys' from line 277\ncontrol/tests/iosys_test.py:1415:13: local variable 'PC' is assigned to but never used\ncontrol/tests/iosys_test.py:1749:16: local variable 'record' is assigned to but never used\ncontrol/tests/iosys_test.py:1757:9: local variable 'h' is assigned to but never used\ncontrol/tests/iosys_test.py:1782:70: local variable 'record' is assigned to but never used\ncontrol/tests/iosys_test.py:1827:9: local variable 'h' is assigned to but never used\ncontrol/tests/iosys_test.py:1903:9: local variable 'resp_short' is assigned to but never used\ncontrol/tests/iosys_test.py:1907:9: local variable 'resp_bad' is assigned to but never used\ncontrol/tests/bdalg_test.py:365:9: local variable 'sys' is assigned to but never used\ncontrol/tests/stochsys_test.py:9:1: 'control.drss' imported but unused\ncontrol/tests/stochsys_test.py:9:1: 'control.ss' imported but unused\ncontrol/tests/stochsys_test.py:228:5: local variable 'P0' is assigned to but never used\ncontrol/tests/stochsys_test.py:247:9: local variable 'estim' is assigned to but never used\ncontrol/tests/stochsys_test.py:429:5: local variable 'W' is assigned to but never used\ncontrol/tests/stochsys_test.py:481:5: 'math.log' imported but unused\ncontrol/tests/docstrings_test.py:432:5: local variable 'docstring' is assigned to but never used\ncontrol/tests/namedio_test.py:11:1: 're' imported but unused\ncontrol/tests/namedio_test.py:296:9: local variable 'res' is assigned to but never used\ncontrol/tests/namedio_test.py:338:9: local variable 'sys' is assigned to but never used\ncontrol/tests/flatsys_test.py:522:9: local variable 'T' is assigned to but never used\ncontrol/tests/flatsys_test.py:632:9: local variable 'T' is assigned to but never used\ncontrol/tests/rlocus_test.py:137:9: undefined name '_RLClickDispatcher'\ncontrol/tests/rlocus_test.py:164:9: 'signal' imported but unused\ncontrol/tests/pzmap_test.py:114:9: local variable 'pzdata' is assigned to but never used\ncontrol/tests/pzmap_test.py:117:9: local variable 'fig' is assigned to but never used\ncontrol/tests/interconnect_test.py:18:1: 'scipy as sp' imported but unused\ncontrol/tests/interconnect_test.py:57:9: local variable 'sumblk' is assigned to but never used\ncontrol/tests/interconnect_test.py:349:9: local variable 'nlios' is assigned to but never used\ncontrol/tests/optimal_test.py:84:5: local variable 't' is assigned to but never used\ncontrol/tests/optimal_test.py:189:5: local variable 'yd' is assigned to but never used\ncontrol/tests/optimal_test.py:267:9: local variable 'ctrl' is assigned to but never used\ncontrol/tests/optimal_test.py:318:5: local variable 't' is assigned to but never used\ncontrol/tests/optimal_test.py:419:9: local variable 't' is assigned to but never used\ncontrol/tests/optimal_test.py:451:9: local variable 'res' is assigned to but never used\ncontrol/tests/optimal_test.py:558:9: local variable 'res' is assigned to but never used\ncontrol/tests/optimal_test.py:652:5: local variable 't' is assigned to but never used\ncontrol/tests/optimal_test.py:736:9: local variable 'curved_seg_length' is assigned to but never used\ncontrol/tests/optimal_test.py:797:9: local variable 'res' is assigned to but never used\ncontrol/tests/optimal_test.py:810:9: local variable 'mhe' is assigned to but never used\ncontrol/tests/modelsimp_test.py:6:1: 'math' imported but unused\ncontrol/tests/modelsimp_test.py:126:9: local variable 'mrk' is assigned to but never used\ncontrol/tests/conftest.py:3:1: 'os' imported but unused\ncontrol/tests/conftest.py:4:1: 'contextlib.contextmanager' imported but unused\ncontrol/tests/conftest.py:7:1: 'numpy as np' imported but unused\ncontrol/tests/lti_test.py:15:1: '.conftest.editsdefaults' imported but unused\ncontrol/tests/lti_test.py:32:13: local variable 'pole_list' is assigned to but never used\ncontrol/tests/lti_test.py:47:13: local variable 'zero_list' is assigned to but never used\ncontrol/tests/lti_test.py:298:13: local variable 'resp' is assigned to but never used\ncontrol/tests/phaseplot_test.py:22:1: 'control.tests.conftest.mplcleanup' imported but unused\ncontrol/tests/phaseplot_test.py:130:5: local variable 'out' is assigned to but never used\ncontrol/tests/timeresp_test.py:8:1: 'scipy as sp' imported but unused\ncontrol/tests/timeresp_test.py:1181:9: local variable 'uvec' is assigned to but never used\ncontrol/tests/timeresp_test.py:1306:9: local variable 'df' is assigned to but never used\ncontrol/tests/statesp_test.py:25:1: '.conftest.editsdefaults' imported but unused\ncontrol/tests/statesp_test.py:147:50: local variable 'w' is assigned to but never used\ncontrol/tests/statesp_test.py:754:9: local variable 'array' is assigned to but never used\ncontrol/tests/statesp_test.py:1101:66: redefinition of unused 'editsdefaults' from line 25\ncontrol/tests/statesp_test.py:1144:29: redefinition of unused 'editsdefaults' from line 25\ncontrol/tests/kwargs_test.py:16:1: 'matplotlib.pyplot as plt' imported but unused\ncontrol/tests/kwargs_test.py:110:53: undefined name 'np'\ncontrol/tests/kwargs_test.py:326:5: dictionary key 'StateSpace.__init__' repeated with different values\ncontrol/tests/kwargs_test.py:331:5: dictionary key 'StateSpace.__init__' repeated with different values\ncontrol/tests/nlsys_test.py:101:9: local variable 'resp_nl' is assigned to but never used\ncontrol/tests/config_test.py:336:9: '..statesp.StateSpace' imported but unused\ncontrol/tests/config_test.py:337:9: 'numpy.array' imported but unused\ncontrol/tests/discrete_test.py:267:9: local variable 'sys' is assigned to but never used\ncontrol/tests/discrete_test.py:304:9: local variable 'sys' is assigned to but never used\ncontrol/tests/discrete_test.py:343:9: local variable 'sys' is assigned to but never used\ncontrol/tests/discrete_test.py:423:13: local variable 'plant_d_warped' is assigned to but never used\ncontrol/tests/statefbk_test.py:10:1: 'math.atan' imported but unused\ncontrol/tests/statefbk_test.py:13:1: 'control.lqe' imported but unused\ncontrol/tests/statefbk_test.py:13:1: 'control.dlqe' imported but unused\ncontrol/tests/statefbk_test.py:823:13: local variable 'Ki' is assigned to but never used\ncontrol/tests/statefbk_test.py:938:1: redefinition of unused 'pi' from line 10\ncontrol/tests/statefbk_test.py:1204:5: local variable 'points' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:77:13: local variable 'default_labels' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:240:9: local variable 'cplt4' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:330:5: local variable 'sys1' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:331:5: local variable 'sys1c' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:332:5: local variable 'sys2' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:338:5: local variable 'expected_labels' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:341:13: local variable 'default_labels' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:490:5: local variable 'sys1' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:491:5: local variable 'sys1c' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:492:5: local variable 'sys2' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:498:5: local variable 'expected_title' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:534:52: undefined name 'resp_fnc'\ncontrol/tests/ctrlplot_test.py:534:64: undefined name 'plot_fnc'\ncontrol/tests/ctrlplot_test.py:585:14: local variable 'bot_row' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:589:13: local variable 'top_row' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:624:5: local variable 'sys1' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:625:5: local variable 'sys1c' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:626:5: local variable 'sys2' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:632:5: local variable 'expected_title' is assigned to but never used\ncontrol/tests/ctrlplot_test.py:635:13: local variable 'default_title' is assigned to but never used\ncontrol/matlab/__init__.py:54:1: 'scipy.signal.zpk2ss' imported but unused\ncontrol/matlab/__init__.py:54:1: 'scipy.signal.ss2zpk' imported but unused\ncontrol/matlab/__init__.py:54:1: 'scipy.signal.tf2zpk' imported but unused\ncontrol/matlab/__init__.py:54:1: 'scipy.signal.zpk2tf' imported but unused\ncontrol/matlab/__init__.py:55:1: 'numpy.linspace' imported but unused\ncontrol/matlab/__init__.py:55:1: 'numpy.logspace' imported but unused\ncontrol/matlab/__init__.py:64:1: 'from ..statesp import *' used; unable to detect undefined names\ncontrol/matlab/__init__.py:65:1: 'from ..xferfcn import *' used; unable to detect undefined names\ncontrol/matlab/__init__.py:66:1: 'from ..lti import *' used; unable to detect undefined names\ncontrol/matlab/__init__.py:67:1: 'from ..iosys import *' used; unable to detect undefined names\ncontrol/matlab/__init__.py:68:1: 'from ..frdata import *' used; unable to detect undefined names\ncontrol/matlab/__init__.py:69:1: 'from ..dtime import *' used; unable to detect undefined names\ncontrol/matlab/__init__.py:70:1: '..exception.ControlArgument' imported but unused\ncontrol/matlab/__init__.py:73:1: 'from ..ctrlutil import *' used; unable to detect undefined names\ncontrol/matlab/__init__.py:74:1: '..freqplot.gangof4' imported but unused\ncontrol/matlab/__init__.py:75:1: '..nichols.nichols' imported but unused\ncontrol/matlab/__init__.py:76:1: 'from ..bdalg import *' used; unable to detect undefined names\ncontrol/matlab/__init__.py:77:1: 'from ..pzmap import *' used; unable to detect undefined names\ncontrol/matlab/__init__.py:78:1: 'from ..statefbk import *' used; unable to detect undefined names\ncontrol/matlab/__init__.py:79:1: 'from ..delay import *' used; unable to detect undefined names\ncontrol/matlab/__init__.py:80:1: 'from ..modelsimp import *' used; unable to detect undefined names\ncontrol/matlab/__init__.py:81:1: 'from ..mateqn import *' used; unable to detect undefined names\ncontrol/matlab/__init__.py:82:1: '..margins.margin' imported but unused\ncontrol/matlab/__init__.py:83:1: '..rlocus.rlocus' imported but unused\ncontrol/matlab/__init__.py:84:1: '..dtime.c2d' imported but unused\ncontrol/matlab/__init__.py:85:1: '..sisotool.sisotool' imported but unused\ncontrol/matlab/__init__.py:86:1: '..stochsys.lqe' imported but unused\ncontrol/matlab/__init__.py:86:1: '..stochsys.dlqe' imported but unused\ncontrol/matlab/__init__.py:90:14: 'poles' may be undefined, or defined from star imports: ..bdalg, ..ctrlutil, ..delay, ..dtime, ..frdata, ..iosys, ..lti, ..mateqn, ..modelsimp, ..pzmap, ..statefbk, ..statesp, ..xferfcn\ncontrol/matlab/__init__.py:90:21: 'zeros' may be undefined, or defined from star imports: ..bdalg, ..ctrlutil, ..delay, ..dtime, ..frdata, ..iosys, ..lti, ..mateqn, ..modelsimp, ..pzmap, ..statefbk, ..statesp, ..xferfcn\ncontrol/matlab/__init__.py:91:12: 'frequency_response' may be undefined, or defined from star imports: ..bdalg, ..ctrlutil, ..delay, ..dtime, ..frdata, ..iosys, ..lti, ..mateqn, ..modelsimp, ..pzmap, ..statefbk, ..statesp, ..xferfcn\ncontrol/matlab/__init__.py:95:1: 'from .timeresp import *' used; unable to detect undefined names\ncontrol/matlab/__init__.py:96:1: 'from .wrappers import *' used; unable to detect undefined names\ncontrol/matlab/__init__.py:99:1: 'from ..config import *' used; unable to detect undefined names\ncontrol/matlab/__init__.py:100:1: 'use_matlab_defaults' may be undefined, or defined from star imports: ..bdalg, ..config, ..ctrlutil, ..delay, ..dtime, ..frdata, ..iosys, ..lti, ..mateqn, ..modelsimp, ..pzmap, ..statefbk, ..statesp, ..xferfcn, .timeresp, .wrappers\ncontrol/flatsys/bspline.py:11:1: 'scipy.interpolate.splev' imported but unused\ncontrol/flatsys/__init__.py:64:1: '.basis.BasisFamily' imported but unused\ncontrol/flatsys/__init__.py:65:1: '.poly.PolyFamily' imported but unused\ncontrol/flatsys/__init__.py:66:1: '.bezier.BezierFamily' imported but unused\ncontrol/flatsys/__init__.py:67:1: '.bspline.BSplineFamily' imported but unused\ncontrol/flatsys/__init__.py:70:1: '.systraj.SystemTrajectory' imported but unused\ncontrol/flatsys/__init__.py:71:1: '.flatsys.FlatSystem' imported but unused\ncontrol/flatsys/__init__.py:71:1: '.flatsys.flatsys' imported but unused\ncontrol/flatsys/__init__.py:72:1: '.linflat.LinearFlatSystem' imported but unused\ncontrol/flatsys/__init__.py:75:1: '.flatsys.point_to_point' imported but unused\ncontrol/flatsys/__init__.py:75:1: '.flatsys.solve_flat_ocp' imported but unused\ncontrol/flatsys/flatsys.py:248:5: '..iosys._process_iosys_keywords' imported but unused\ncontrol/flatsys/flatsys.py:486:19: undefined name 'ControlArgument'\ncontrol/flatsys/flatsys.py:520:13: redefinition of unused 'traj_cost' from line 499\ncontrol/flatsys/flatsys.py:724:5: local variable 'Tf' is assigned to but never used\ncontrol/flatsys/flatsys.py:725:46: undefined name 'T0'\n```","author":{"url":"https://github.com/roryyorke","@type":"Person","name":"roryyorke"},"datePublished":"2025-01-19T06:48:23.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":5},"url":"https://github.com/1101/python-control/issues/1101"}

route-pattern/_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format)
route-controllervoltron_issues_fragments
route-actionissue_layout
fetch-noncev2:9e90f07a-a581-24bd-6138-524b328befc8
current-catalog-service-hash81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114
request-idA8EA:2F1D49:2A45C79:3991D4C:6979B8E2
html-safe-nonceb4ee8aee839d528f98b4ee8e06c1223f5f59ff1a86696c2751a0546d969f7ccd
visitor-payloadeyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJBOEVBOjJGMUQ0OToyQTQ1Qzc5OjM5OTFENEM6Njk3OUI4RTIiLCJ2aXNpdG9yX2lkIjoiMzM3MTY4NjcyNTEzMzM4MzkwNiIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9
visitor-hmac43e5c913719e9a9d0c67c7e809f5bb3466361465af6a089e3ce418474680f568
hovercard-subject-tagissue:2797465238
github-keyboard-shortcutsrepository,issues,copilot
google-site-verificationApib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I
octolytics-urlhttps://collector.github.com/github/collect
analytics-location///voltron/issues_fragments/issue_layout
fb:app_id1401488693436528
apple-itunes-appapp-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/python-control/python-control/1101/issue_layout
twitter:imagehttps://opengraph.githubassets.com/0bb3b9bbb87a358afd72335a42971ca6ade946cde161ca87d24096cd8e0677db/python-control/python-control/issues/1101
twitter:cardsummary_large_image
og:imagehttps://opengraph.githubassets.com/0bb3b9bbb87a358afd72335a42971ca6ade946cde161ca87d24096cd8e0677db/python-control/python-control/issues/1101
og:image:altRunning pyflakes on main HEAD (0ff0452) reveals lots of possible issues; see end. Is there interest in getting and keeping the tree lint-clean? This would require some changes to style, e.g., dropp...
og:image:width1200
og:image:height600
og:site_nameGitHub
og:typeobject
og:author:usernameroryyorke
hostnamegithub.com
expected-hostnamegithub.com
Nonec049b65ec7e54cbf2521f5a560b6527714c612b0bd169188e2ea6e16f83bd5f4
turbo-cache-controlno-preview
go-importgithub.com/python-control/python-control git https://github.com/python-control/python-control.git
octolytics-dimension-user_id2285872
octolytics-dimension-user_loginpython-control
octolytics-dimension-repository_id22791752
octolytics-dimension-repository_nwopython-control/python-control
octolytics-dimension-repository_publictrue
octolytics-dimension-repository_is_forkfalse
octolytics-dimension-repository_network_root_id22791752
octolytics-dimension-repository_network_root_nwopython-control/python-control
turbo-body-classeslogged-out env-production page-responsive
disable-turbofalse
browser-stats-urlhttps://api.github.com/_private/browser/stats
browser-errors-urlhttps://api.github.com/_private/browser/errors
release87b137883e35e2766c3d0f6a257c4044f6390b83
ui-targetfull
theme-color#1e2327
color-schemelight dark

Links:

Skip to contenthttps://github.com/python-control/python-control/issues/1101#start-of-content
https://github.com/
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fpython-control%2Fpython-control%2Fissues%2F1101
GitHub CopilotWrite better code with AIhttps://github.com/features/copilot
GitHub SparkBuild and deploy intelligent appshttps://github.com/features/spark
GitHub ModelsManage and compare promptshttps://github.com/features/models
MCP RegistryNewIntegrate external toolshttps://github.com/mcp
ActionsAutomate any workflowhttps://github.com/features/actions
CodespacesInstant dev environmentshttps://github.com/features/codespaces
IssuesPlan and track workhttps://github.com/features/issues
Code ReviewManage code changeshttps://github.com/features/code-review
GitHub Advanced SecurityFind and fix vulnerabilitieshttps://github.com/security/advanced-security
Code securitySecure your code as you buildhttps://github.com/security/advanced-security/code-security
Secret protectionStop leaks before they starthttps://github.com/security/advanced-security/secret-protection
Why GitHubhttps://github.com/why-github
Documentationhttps://docs.github.com
Bloghttps://github.blog
Changeloghttps://github.blog/changelog
Marketplacehttps://github.com/marketplace
View all featureshttps://github.com/features
Enterpriseshttps://github.com/enterprise
Small and medium teamshttps://github.com/team
Startupshttps://github.com/enterprise/startups
Nonprofitshttps://github.com/solutions/industry/nonprofits
App Modernizationhttps://github.com/solutions/use-case/app-modernization
DevSecOpshttps://github.com/solutions/use-case/devsecops
DevOpshttps://github.com/solutions/use-case/devops
CI/CDhttps://github.com/solutions/use-case/ci-cd
View all use caseshttps://github.com/solutions/use-case
Healthcarehttps://github.com/solutions/industry/healthcare
Financial serviceshttps://github.com/solutions/industry/financial-services
Manufacturinghttps://github.com/solutions/industry/manufacturing
Governmenthttps://github.com/solutions/industry/government
View all industrieshttps://github.com/solutions/industry
View all solutionshttps://github.com/solutions
AIhttps://github.com/resources/articles?topic=ai
Software Developmenthttps://github.com/resources/articles?topic=software-development
DevOpshttps://github.com/resources/articles?topic=devops
Securityhttps://github.com/resources/articles?topic=security
View all topicshttps://github.com/resources/articles
Customer storieshttps://github.com/customer-stories
Events & webinarshttps://github.com/resources/events
Ebooks & reportshttps://github.com/resources/whitepapers
Business insightshttps://github.com/solutions/executive-insights
GitHub Skillshttps://skills.github.com
Documentationhttps://docs.github.com
Customer supporthttps://support.github.com
Community forumhttps://github.com/orgs/community/discussions
Trust centerhttps://github.com/trust-center
Partnershttps://github.com/partners
GitHub SponsorsFund open source developershttps://github.com/sponsors
Security Labhttps://securitylab.github.com
Maintainer Communityhttps://maintainers.github.com
Acceleratorhttps://github.com/accelerator
Archive Programhttps://archiveprogram.github.com
Topicshttps://github.com/topics
Trendinghttps://github.com/trending
Collectionshttps://github.com/collections
Enterprise platformAI-powered developer platformhttps://github.com/enterprise
GitHub Advanced SecurityEnterprise-grade security featureshttps://github.com/security/advanced-security
Copilot for BusinessEnterprise-grade AI featureshttps://github.com/features/copilot/copilot-business
Premium SupportEnterprise-grade 24/7 supporthttps://github.com/premium-support
Pricinghttps://github.com/pricing
Search syntax tipshttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
documentationhttps://docs.github.com/search-github/github-code-search/understanding-github-code-search-syntax
Sign in https://github.com/login?return_to=https%3A%2F%2Fgithub.com%2Fpython-control%2Fpython-control%2Fissues%2F1101
Sign up https://github.com/signup?ref_cta=Sign+up&ref_loc=header+logged+out&ref_page=%2F%3Cuser-name%3E%2F%3Crepo-name%3E%2Fvoltron%2Fissues_fragments%2Fissue_layout&source=header-repo&source_repo=python-control%2Fpython-control
Reloadhttps://github.com/python-control/python-control/issues/1101
Reloadhttps://github.com/python-control/python-control/issues/1101
Reloadhttps://github.com/python-control/python-control/issues/1101
python-control https://github.com/python-control
python-controlhttps://github.com/python-control/python-control
Notifications https://github.com/login?return_to=%2Fpython-control%2Fpython-control
Fork 447 https://github.com/login?return_to=%2Fpython-control%2Fpython-control
Star 2k https://github.com/login?return_to=%2Fpython-control%2Fpython-control
Code https://github.com/python-control/python-control
Issues 87 https://github.com/python-control/python-control/issues
Pull requests 8 https://github.com/python-control/python-control/pulls
Discussions https://github.com/python-control/python-control/discussions
Actions https://github.com/python-control/python-control/actions
Projects 0 https://github.com/python-control/python-control/projects
Wiki https://github.com/python-control/python-control/wiki
Security 0 https://github.com/python-control/python-control/security
Insights https://github.com/python-control/python-control/pulse
Code https://github.com/python-control/python-control
Issues https://github.com/python-control/python-control/issues
Pull requests https://github.com/python-control/python-control/pulls
Discussions https://github.com/python-control/python-control/discussions
Actions https://github.com/python-control/python-control/actions
Projects https://github.com/python-control/python-control/projects
Wiki https://github.com/python-control/python-control/wiki
Security https://github.com/python-control/python-control/security
Insights https://github.com/python-control/python-control/pulse
New issuehttps://github.com/login?return_to=https://github.com/python-control/python-control/issues/1101
New issuehttps://github.com/login?return_to=https://github.com/python-control/python-control/issues/1101
#1118https://github.com/python-control/python-control/pull/1118
pyflakes (pylint, ruff, ....) clean-up, CI?https://github.com/python-control/python-control/issues/1101#top
#1118https://github.com/python-control/python-control/pull/1118
https://github.com/roryyorke
https://github.com/roryyorke
roryyorkehttps://github.com/roryyorke
on Jan 19, 2025https://github.com/python-control/python-control/issues/1101#issue-2797465238
0ff0452https://github.com/python-control/python-control/commit/0ff045263f69802e32d7e5a69ac5c16142f7a96e
https://github.com
Termshttps://docs.github.com/site-policy/github-terms/github-terms-of-service
Privacyhttps://docs.github.com/site-policy/privacy-policies/github-privacy-statement
Securityhttps://github.com/security
Statushttps://www.githubstatus.com/
Communityhttps://github.community/
Docshttps://docs.github.com/
Contacthttps://support.github.com?tags=dotcom-footer

Viewport: width=device-width


URLs of crawlers that visited me.