From 8f3ba1d9473195c1cbafe967a127800d90fde3ee Mon Sep 17 00:00:00 2001 From: Michael Orlitzky Date: Thu, 4 Mar 2021 17:43:59 -0500 Subject: [PATCH] eja: improve a gram_schmidt() error message. --- mjo/eja/eja_utils.py | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/mjo/eja/eja_utils.py b/mjo/eja/eja_utils.py index 2402d9f..d4e9990 100644 --- a/mjo/eja/eja_utils.py +++ b/mjo/eja/eja_utils.py @@ -230,7 +230,14 @@ def gram_schmidt(v, inner_product=None): """ if inner_product is None: inner_product = lambda x,y: x.inner_product(y) - norm = lambda x: inner_product(x,x).sqrt() + def norm(x): + ip = inner_product(x,x) + # Don't expand the given field; the inner-product's codomain + # is already correct. For example QQ(2).sqrt() returns sqrt(2) + # in SR, and that will give you weird errors about symbolics + # when what's really going wrong is that you're trying to + # orthonormalize in QQ. + return ip.parent()(ip.sqrt()) v = list(v) # make a copy, don't clobber the input -- 2.43.2