fix(mateqn): use np.finfo instead of bare finfo · python-control/python-control@c4e609e · GitHub

python-control / python-control Public

• Notifications You must be signed in to change notification settings
• Fork 462
• Star 2.1k

• Code
• Issues 81
• Pull requests 6
• Discussions
• Actions
• Projects
• Wiki
• Security and quality
• Insights

Additional navigation options

Commit c4e609e

Browse filesBrowse the repository at this point in the historyBrowse files

Петушков А.

committed

fix(mateqn): use np.finfo instead of bare finfo

1 parent 0af3dbf commit c4e609e
Copy full SHA for c4e609e

1 file changed

+2-2Lines changed: 2 additions & 2 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• control
  • mateqn.py

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎control/mateqn.py‎

Copy file name to clipboardExpand all lines: control/mateqn.py

+2-2Lines changed: 2 additions & 2 deletions

Original file line number	Diff line number	Diff line change
@@ -69,7 +69,7 @@ def _warn_ill_conditioned_E(E):
69	69	not measurably more accurate in that regime.
70	70	"""
71	71	condE = np.linalg.cond(E)
72		- if condE > 1.0 / np.sqrt(finfo(float).eps):
	72	+ if condE > 1.0 / np.sqrt(np.finfo(float).eps):
73	73	warnings.warn(
74	74	f"E is ill-conditioned (cond(E) = {condE:.2g}); the generalized "
75	75	"Lyapunov solution may have reduced accuracy. The problem itself "
@@ -383,7 +383,7 @@ def dlyap(A, Q, C=None, E=None, method=None):
383	383	# Solvability requires lam_A * lam_Q != 1 for all pairs of
384	384	# eigenvalues (the diagonals of the triangular factors)
385	385	if np.min(np.abs(np.outer(np.diag(Tq), np.diag(Ta)) - 1.)) \
386		- < finfo(float).eps * max(
	386	+ < np.finfo(float).eps * max(
387	387	1., np.abs(np.diag(Ta)).max()
388	388	* np.abs(np.diag(Tq)).max()):
389	389	raise ControlArgument(

0 commit comments

Comments

0 (0)