|Re: [eigen] What about HyperPlane ?|
[ Thread Index |
| More lists.tuxfamily.org/eigen Archives
- To: eigen@xxxxxxxxxxxxxxxxxxx
- Subject: Re: [eigen] What about HyperPlane ?
- From: "Gael Guennebaud" <gael.guennebaud@xxxxxxxxx>
- Date: Thu, 28 Aug 2008 17:42:02 +0200
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:received:received:message-id:date:from:to :subject:in-reply-to:mime-version:content-type :content-transfer-encoding:content-disposition:references; bh=zK02nQeIllJITdEW/DYQgr5jiVPnzSwQCGtoFW0kzFU=; b=w5FDAheEdHKa9wfy50hD+N4a6zXoNAecmT8et/UGEOEbRMdykzuAH3SJaQ9WQlxJ7X awSJfK50IZUlDmNavulAUyU2DV54+lpYfQRTjvX/uDavQLzVvLjUC+CXeyLRZyJIYwNc ZW6SHZLqltIYlMuwDkHqw4v6nn5ybDtIsBtyE=
- Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=message-id:date:from:to:subject:in-reply-to:mime-version :content-type:content-transfer-encoding:content-disposition :references; b=w4GFr2Lx5DpKjusedEackq9jS6ttGMm31yL3HS/JatVfcw49jFsjPzkMhzWXcmDz1t 0vwkGJbvhHUH98JhRwyMrmEg4j53SGZPdYH3wWcDsTqoQPh87ff0T935LQhX2TImehTj 37expqvdiq455EiwZT0RaZX1KXk+O/Xmf1Sgs=
On Thu, Aug 28, 2008 at 4:17 PM, Benoît Jacob <jacob@xxxxxxxxxxxxxxx> wrote:
> Why not; There is a question: do you think the hyperplane equation should have
> normalized normal vector?
definitely yes ! my class currently enforces unit normal.
> For most use cases, this is very useful. Like computing the distance from your
> hyperplane to points. It is much faster if the normal vector can be assumed
> to be normalized. For that reason a hyperplane class is much more useful with
> normalized normal vector.
> This will probably require to normalize in fitHyperplane() but that's no big
> deal, the cost is not much compared to the eigensolver used.
fitHyperplane() already returns a normalized normal because it is and
eigen vector. (hm... this remark might be quite confusing ! for the
lost reader: don't think Eigen's vectors cannot be non-unit, only
eigen ones are normalized ;) )
> Notify me before changing the fitHyperplane() API because this affects
> Avogadro :) We need to coordinate.
yes I know that, so I'll mark the old one with a deprecated gcc
warning for a few weeks....