|Re: [eigen] LinSpaced() API|
[ Thread Index |
| More lists.tuxfamily.org/eigen Archives
- To: eigen@xxxxxxxxxxxxxxxxxxx
- Subject: Re: [eigen] LinSpaced() API
- From: Hauke Heibel <hauke.heibel@xxxxxxxxxxxxxx>
- Date: Thu, 22 Jul 2010 13:41:47 +0200
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=googlemail.com; s=gamma; h=domainkey-signature:mime-version:received:received:in-reply-to :references:date:message-id:subject:from:to:content-type :content-transfer-encoding; bh=uICfOAe9xn+0n6s1RxDDqPJknRGQLm7szTisacjb2d0=; b=xz7tDYXPLbPmQDU67f2xE+YGQDA8Vr+vulROVNjHynmMHb8e5dkk9eR4+5dX20x5d2 21n1Yh8JZgNnlRt5fvc/tmlNx93N/njW5/oKdpD/o3yUoWcqtlm634Kzb7m1cq1LX0Y2 NDHkC6iqKs7e70aiVzEVm0DCU7RNUGViM1iHE=
- Domainkey-signature: a=rsa-sha1; c=nofws; d=googlemail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type:content-transfer-encoding; b=oyUfoLcA+ZiQDrgZZZPRXMtjhD7j9XOTJiwFUJfuA838yeYLV2otfxt6HjQ0Th0suH 8ffROrm/Z8Uyrof4WOBic66xatg7HcTnsbSmigzEQ8099t/PVVXNQBofMNpsFeCgyLH7 pPoDD2OUqeJXnfkG6yZwOC3k6iWZJth9/EDbY=
I wrote those functions....
On Thu, Jul 22, 2010 at 1:13 PM, Jitse Niesen <jitse@xxxxxxxxxxxxxxxxx> wrote:
> I'm adding some bits to Tutorial page 5 (advanced initialization).
Thank you for doing this since I am hardly finding any time at the moment.
> doing this, I noticed the order of the arguments for the static methods
> Constant() and LinSpaced() for vectors:
> Constant(size, value) versus LinSpaced(low, high, size)
> I guess that the order of arguments for LinSpaced() mirrors the order for
> the Matlab function linspace. However, it does seem inconsistent when
> compared to the order of arguments for Constant(). I think that consistency
> within Eigen is more important than consistency with Matlab, so what about
> changing the order to LinSpaced(size, low, high)?
That's true and I am fine with switching. Last chance for breaking the
> Additionally, I note that there is no fixed-size variant without the size
> argument. Is there any reason for that?
I cannot recall why I did not implement this. It should be simply to
add an overload. I can do both changes right now.