Removed noncopyable restriction on Conditional - this is not used on any other factors, and makes it harder to make unit tests. Should not make a difference for users since we already use exclusively pointer containers and using the linear side directly is fairly low-level anyway.
parent
1fed4fff8a
commit
081b805340
|
@ -36,13 +36,9 @@ namespace gtsam {
|
||||||
* Derived classes *must* redefine the Factor and shared_ptr typedefs to refer
|
* Derived classes *must* redefine the Factor and shared_ptr typedefs to refer
|
||||||
* to the associated factor type and shared_ptr type of the derived class. See
|
* to the associated factor type and shared_ptr type of the derived class. See
|
||||||
* IndexConditional and GaussianConditional for examples.
|
* IndexConditional and GaussianConditional for examples.
|
||||||
*
|
|
||||||
* We make it noncopyable so we enforce the fact that factors are
|
|
||||||
* kept in pointer containers. To be safe, you should make them
|
|
||||||
* immutable, i.e., practicing functional programming.
|
|
||||||
*/
|
*/
|
||||||
template<typename KEY>
|
template<typename KEY>
|
||||||
class Conditional: public gtsam::Factor<KEY>, boost::noncopyable {
|
class Conditional: public gtsam::Factor<KEY> {
|
||||||
|
|
||||||
private:
|
private:
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue