New user self-registration is currently disabled. Please email eigen-core-team @ if you need an account.
Bug 1286 - Automatically adjust the prototype of functors for CwiseNullaryOp
Automatically adjust the prototype of functors for CwiseNullaryOp
Product: Eigen
Classification: Unclassified
Component: Core - general
3.3 (current stable)
All All
: Normal Unknown
Assigned To: Nobody
Depends on:
Blocks: 457 720
  Show dependency treegraph
Reported: 2016-08-31 12:46 UTC by Gael Guennebaud
Modified: 2016-08-31 13:14 UTC (History)
3 users (show)

patch (19.07 KB, patch)
2016-08-31 12:46 UTC, Gael Guennebaud
no flags Details | Diff

Description Gael Guennebaud 2016-08-31 12:46:29 UTC
Created attachment 726 [details]

Current functors passed to NullaryExpr must implement both operator()(i) and operator()(i,j) plus specialize internal::functor_has_linear_access<> for non trivial functors. This is rather painful and cumbersome to document (hence, this is currently not really documented!).

The proposed patch automatically detect whether the functor expose one or multiple of the following operators:


and then a little helper augments the input functors with the missing operators.

With this change, no need to provide operator()(i,j) if nor i nor j are required (e.g.,  for random, and constant).

I'd like to push this change in 3.3 as it considerably help in documenting and using NullaryExpr.
Comment 1 Christoph Hertzberg 2016-08-31 13:14:50 UTC
I'm all for this!

Note You need to log in before you can comment on or make changes to this bug.