Anohter question about MeasurementModel<MeasVar, StateVar>::Simulate: return type correct?

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
_______________________________________________
I hereby promise not to top-post on the
BFL mailing list
BFL [..] ...
http://lists.mech.kuleuven.be/mailman/listinfo/bfl

Disclaimer: http://www.kuleuven.be/cwis/email_disclaimer.htm

Anohter question about MeasurementModel<MeasVar, StateVar>::Simu

On Monday 01 December 2008 20:10:17 Jurge van Eijck wrote:
> 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();" ?
Again, I think you are right. Opinion of the author??

Tinne
_______________________________________________
I hereby promise not to top-post on the
BFL mailing list
BFL [..] ...
http://lists.mech.kuleuven.be/mailman/listinfo/bfl

Disclaimer: http://www.kuleuven.be/cwis/email_disclaimer.htm

Anohter question about MeasurementModel<MeasVar, StateVar>::Simu

On Tue, Dec 2, 2008 at 9:14 AM, Tinne De Laet
<tinne [dot] delaet [..] ...> wrote:
> On Monday 01 December 2008 20:10:17 Jurge van Eijck wrote:
>> 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();" ?
> Again, I think you are right. Opinion of the author??

Same as before. Unfortunately, I can't track this one through the svn
history since it seems this has been lost in the conversion from the
actsens repository. The last (totally harmless) commit was by Wim,
but I can't retrieve the rest of the diffs :-(

Klaas
_______________________________________________
I hereby promise not to top-post on the
BFL mailing list
BFL [..] ...
http://lists.mech.kuleuven.be/mailman/listinfo/bfl

Disclaimer: http://www.kuleuven.be/cwis/email_disclaimer.htm

Anohter question about MeasurementModel<MeasVar, StateVar>::Simu

...
> >>
> >> The line : " MatrixWrapper::ColumnVector result = Simulated.ValueGet();"
> >> is maybe incorrect.
> >> Should it not be: "MeasVar result = Simulated.ValueGet();" ?
> >
> > Again, I think you are right. Opinion of the author??
>
> Same as before. Unfortunately, I can't track this one through the svn
> history since it seems this has been lost in the conversion from the
> actsens repository. The last (totally harmless) commit was by Wim,
> but I can't retrieve the rest of the diffs :-(
I opened a bugreport (bug#600) for this error.

Tinne

Disclaimer: http://www.kuleuven.be/cwis/email_disclaimer.htm

_______________________________________________
I hereby promise not to top-post on the
BFL mailing list
BFL [..] ...
http://lists.mech.kuleuven.be/mailman/listinfo/bfl

Disclaimer: http://www.kuleuven.be/cwis/email_disclaimer.htm