| Re: [eigen] eigen sqrt compilation error |
[ Thread Index |
Date Index
| More lists.tuxfamily.org/eigen Archives
]
- To: eigen@xxxxxxxxxxxxxxxxxxx
- Subject: Re: [eigen] eigen sqrt compilation error
- From: Benoit Jacob <jacob.benoit.1@xxxxxxxxx>
- Date: Fri, 19 Jun 2009 15:10:27 +0200
- Dkim-signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=gamma; h=domainkey-signature:mime-version:received:in-reply-to:references :date:message-id:subject:from:to:content-type :content-transfer-encoding; bh=cyxFZk1wFi9vdmrYbvdWghIbtElmhdfBeZ1IU229538=; b=XkwHak3cdx7Ph3wofqiz55LqZUf2IVaZYG3yNwsZujlL8Rpr+p3piuwlIniPCwdsXv jR2oUkBLt5uwajwUuz1wRemiG96jZ4O5nNcWuVlCQowiIxgdc14dg/VLdev/dBPO5IYJ H4mlIORk0lc9y+WCcA4w6IYgty1Spn3ZqUBr4=
- Domainkey-signature: a=rsa-sha1; c=nofws; d=gmail.com; s=gamma; h=mime-version:in-reply-to:references:date:message-id:subject:from:to :content-type:content-transfer-encoding; b=WqMkKIH55Ic0xer0LDmdDKY6Zf2NW6XbH16FvsnEURb8Soqy85v3VcvS0F+Jemw7QV mBBnTmfasNpmjsxqu9DNJJDCnyycOl87e9N+2LTu7u+Lzk6z0FFQoQk7QgutmNU3JNqr zVtTwkaobJilMDgUqiXtOPV37m9xu1iywS7wg=
2009/6/19 Patrick Mihelich <patrick.mihelich@xxxxxxxxx>
>
> On Fri, Jun 19, 2009 at 12:37 AM, Moritz Lenz <mlenz@xxxxxxxxxxxxxxxxxxxxxxxx> wrote:
>>
>> Considering the number of times people make this error (forgetting to include it), what about including it by default? Is there a very good reason not to do it?
>
> I wonder this too. I ran into this error when I started using Eigen, and it seems to pop up regularly on the list. What exactly is the cost of including <Eigen/Array> in the core? How much does it increase compile times? Documentation is good, but it is still too easy to use the Eigen API incorrectly like this, and the error messages are no help.
>
> Awhile back I suggested a trick based on default arguments, which is a little ugly but does massively improve the error messages. Here it is again:
>
> // Changes to MatrixBase
> template<typename Derived> class MatrixBase
> {
> private:
> struct ERROR_YOU_MUST_INCLUDE_Eigen_Array {};
>
> public:
> const PartialRedux<Derived, Horizontal> rowwise(ERROR_YOU_MUST_INCLUDE_Eigen_Array) const;
> };
>
> // Changes to implementation in Array module
> // Defining the default argument here changes the function signature, allowing matrix.rowwise()
> template<typename Derived>
> const PartialRedux<Derived,Horizontal> MatrixBase<Derived>::rowwise(ERROR_YOU_MUST_INCLUDE_Eigen_Array = ERROR_YOU_MUST_INCLUDE_Eigen_Array()) const
> {
> /* implementation */
> }
>
> Then when the user includes <Eigen/Array>, he effectively gets the expected zero-arg version of rowwise. If he forgets to include it, no zero-arg version exists, so mat.rowwise() should produce a compiler error along the lines of:
>
> error: no matching function for call to 'MatrixBase<Derived>::rowwise()'
> note: candidates are: const PartialRedux<Derived, 0> MatrixBase<Derived>::rowwise(MatrixBase<Derived>::ERROR_YOU_MUST_INCLUDE_Eigen_Array) const
>
> As usual you can wrap the dummy argument in a macro to selectively hide it from doxygen.
My main reticence with this is that it is pretty heavy and we have to
do this at many places!
Benoit