[Bug 600] New: another error in Simulate method of MeasurementModel

For more infomation about this bug, visit
Summary: another error in Simulate method of MeasurementModel
Product: BFL
Version: 0.6.1
Platform: All
OS/Version: All
Status: NEW
Severity: normal
Priority: P1
Component: core
AssignedTo: bfl [..] ...
ReportedBy: tinne [dot] delaet [..] ...
CC: bfl [..] ...
Estimated Hours: 0.0

Hi,

I have a question about version: BFL 0.6.1

The class MeasurementModel the following template method:

template MeasVar
MeasurementModel::Simulate(const StateVar& x,
int sampling_method,
void * sampling_args)
{
assert(_systemWithoutSensorParams == true);
_MeasurementPdf->ConditionalArgumentSet(0,x);
Sample Simulated(MeasurementSizeGet());
_MeasurementPdf->SampleFrom(Simulated, sampling_method,sampling_args);
MatrixWrapper::ColumnVector result = Simulated.ValueGet();
return result;
}

The line : " MatrixWrapper::ColumnVector result = Simulated.ValueGet();" is
maybe incorrect.
Should it not be: "MeasVar result = Simulated.ValueGet();" ?

With best regards,

Jürge

[Bug 600] another error in Simulate method of MeasurementModel

For more infomation about this bug, visit

Tinne De Laet <tinne [dot] delaet [..] ...> changed:

What |Removed |Added
--------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |FIXED

--- Comment #2 from Tinne De Laet <tinne [dot] delaet [..] ...> 2008-12-03 09:25:31 ---
Patch applied in revision 29765:

Sending model/measurementmodel.cpp
Transmitting file data .
Committed revision 29765.

[Bug 600] another error in Simulate method of MeasurementModel

For more infomation about this bug, visit

Tinne De Laet <tinne [dot] delaet [..] ...> changed:

What |Removed |Added
--------------------------------------------------------------------------
Status|NEW |ASSIGNED
AssignedTo|bfl [..] ... |tinne [dot] delaet [..] ...en.b
| |e

--- Comment #1 from Tinne De Laet <tinne [dot] delaet [..] ...> 2008-12-03 09:22:51 ---
Created an attachment (id=355)
--> (https://www.fmtc.be/bugzilla/orocos/attachment.cgi?id=355)
patch solves bug#600