program exit, while trying to initialize a class

[dvir]

Hi

I have a runing error while trying to initialize a class.
this is my code:

main.C
-------------
#include "tdagent.H"
#include "simple_env .H"
#include <vector>

int main()
{
simple_env *env;
env = new simple_env();
env->create();
tdagent *agent;
agent = new tdagent(env);
....
}
tdagent.H
------------------

#ifndef TDGAENT_H
#define TDAGENT_H

#include "critic.H"
#include "actor.H"
#include "simple_env .H"

class tdagent
{

private:

ublas::vector<f loat> get_state();
ublas::vector<f loat> get_previous_st ate();
void set_previous_st ate(ublas::vect or<float> curr_state);
float get_reward();
simple_env* env;
critic* critic1;
actor* actor1;
float d;
state x;

public:
tdagent(simple_ env* environment);
~tdagent();
std::vector<flo at> find_next_actio n();
};

#endif

tdagent.C
------------------
#include "tdagent.H"

tdagent::tdagen t(simple_env* env)
: critic1(new critic(env->get_cov(),en v->get_state()) ),
actor1(new actor(env->get_cov(),en v->get_state(),
env->get_actions_nu m(),
env->get_min_reward (),env->get_max_reward ()))
{
x.prev=get_stat e();
}

.....

The code compiled fine.
But when I tried to run it I got:

"Aborted (core dumped)"

I tried to debug it using ddd, and I saw that the program exits at:
"agent = new tdagent(env);"
at main.C
when trying to step into this function I get to
/usr/lib/gcc/i686-pc-cygwin/3.4.4/include/c++/iostream

I'm using Cygwin g++ 3.4.4

I don't understand where is my mistake.
Hope you can help me find it.

Thanks for your help.
Dvir.

[Richard Herring]

In message <11************ **********@j55g 2000cwa.googleg roups.com>, dvir
<sc*****@gmail. com> writes

Hi

I have a runing error while trying to initialize a class.
this is my code:
[snip incomplete code]
I don't understand where is my mistake.
Hope you can help me find it.

That's unlikely, since you haven't shown any of the member functions of
simple_env, critic or actor.

Post a minimal _complete_ program that illustrates the problem.

--
Richard Herring

[dvir]

Ok here is a more complete version of the code.
hope its more clear now.

main.C
------------
#include "tdagent.H"
#include "simple_env .H"
#include <vector>

int main()
{
simple_env *env;
env = new simple_env();
env->create();
tdagent *agent;
agent = new tdagent(env);
....
}

simple_env.H
---------------------
#ifndef SIMPLE_ENV_H
#define SIMPLE_ENV_H

#include <math.h>
#include <boost/numeric/ublas/matrix.hpp>
#include <boost/numeric/ublas/vector.hpp>
#include <boost/numeric/ublas/io.hpp>
#include <boost/numeric/ublas/matrix_proxy.hp p>
#include <boost/numeric/ublas/vector_proxy.hp p>

#define PI 3.1415926535897 932384626433832 795

class simple_env
{
public:
simple_env();
~simple_env();
void create();
float preform(float u);
float get_reward();

boost::numeric: :ublas::vector< float> get_state();
boost::numeric: :ublas::vector< float> get_previous_st ate();
boost::numeric: :ublas::vector< float> get_final_state ();
boost::numeric: :ublas::vector< float> get_cov();
int get_actions_num ();
float get_min_reward( );
float get_max_reward( );

private:
float sigmoid(float u);
float reward;
boost::numeric: :ublas::vector< float> state;
boost::numeric: :ublas::vector< float> prev_state;
boost::numeric: :ublas::vector< float> final_state;
boost::numeric: :ublas::vector< float> cov;
};

#endif
simple_env.C
---------------------
#include "simple_env .H"

namespace ublas = boost::numeric: :ublas;
using namespace boost::numeric: :ublas;

simple_env::sim ple_env()
{

reward=0;
}

simple_env::~si mple_env()
{

}

void simple_env::cre ate()
{
cov.resize(3);
for (unsigned int i = 0; i < 3; ++ i)
cov (i) = 1;
vector<float> initial_state(3 );
for (unsigned int i = 0; i < initial_state.s ize (); ++ i)
initial_state (i) = 0;

final_state.res ize(3);
for (unsigned int i = 0; i < final_state.siz e (); ++ i)
final_state (i) = 5;

state=initial_s tate;

return;
}

ublas::vector<f loat> simple_env::get _cov()
{
return cov;
}
float simple_env::pre form(float u)
{

prev_state=stat e;
for (unsigned int i = 0; i < state.size (); ++ i)
state (i) += sigmoid(u);
reward=norm_2(s tate);

return reward;

}

float simple_env::get _reward()
{
return reward;
}

ublas::vector<f loat> simple_env::get _state()
{
return state;
}

ublas::vector<f loat> simple_env::get _final_state()
{
return final_state;
}
ublas::vector<f loat> simple_env::get _previous_state ()
{
return prev_state;
}

float simple_env::sig moid(float u)
{
float s;
s=PI/2*atanf(2/PI*u);
return s;
}

int simple_env::get _actions_num()
{
return 1;
}

float simple_env::get _min_reward()
{
return 0;
}

float simple_env::get _max_reward()
{
return 8.66;
}

tdagent.H
----------------
#ifndef TDGAENT_H
#define TDAGENT_H

#include "critic.H"
#include "actor.H"
#include "simple_env .H"

class tdagent
{

private:

ublas::vector<f loat> get_state();
ublas::vector<f loat> get_previous_st ate();
void set_previous_st ate(ublas::vect or<float> curr_state);
float get_reward();
simple_env* env;
critic* critic1;
actor* actor1;
float d;
state x;

public:
tdagent(simple_ env* environment);
~tdagent();
std::vector<flo at> find_next_actio n();
};

#endif

tdagent.C
-----------------
#include "tdagent.H"

tdagent::tdagen t(simple_env* env)
: critic1(new critic(env->get_cov(),en v->get_state()) ),
actor1(new actor(env->get_cov(),en v->get_state(),
env->get_actions_nu m(),
env->get_min_reward (),env->get_max_reward ()))
{
x.prev=get_stat e();
}

....

Thanks.
Dvir

[Paul]

"dvir" <sc*****@gmail. com> wrote in message
news:11******** **************@ c74g2000cwc.goo glegroups.com.. .

Ok here is a more complete version of the code.
hope its more clear now.

main.C
------------
#include "tdagent.H"
#include "simple_env .H"
#include <vector>

int main()
{
simple_env *env;
env = new simple_env();
env->create();
tdagent *agent;
agent = new tdagent(env);
...
}
1) Why are you dynamically creating these instances? Does that play into a
role in duplicating the problem?

And please remove the "..." as explained in the FAQ:

http://www.parashift.com/c++-faq-lit...t.html#faq-5.8
#include "tdagent.H"
#include "simple_env .H"
#include <vector>

int main()
{
simple_env env;
env.create();
tdagent agent(&env);
}

Is the above enough to duplicate the problem (that is if the rest of the
code is OK, which it isn't -- see next item).

2) vector<float> initial_state(3 );
for (unsigned int i = 0; i < initial_state.s ize (); ++ i)
initial_state (i) = 0; // This should have never compiled
There is no user-defined operator() for vector, but you're calling one.
What does this line do?

initial_state (i) = 0;

Shouldn't this be:

initial_state[i] = 0;

?
The code compiled fine.

It did? That's shocking, seeing the error you made in your program
concerning using ( ) instead of [ ] when accessing elements of the vector.

Here it is to you, as a reference to what I'm speaking of:

#include <vector>
int main()
{
std::vector<flo at> initial_state(3 );
for (unsigned int i = 0; i < initial_state.s ize(); ++i )
initial_state(i ) = 0;
}

----------------------------------------------------------------------------
----------------------------------
Your Comeau C/C++ test results are as follows:
Comeau C/C++ 4.3.3 (Aug 6 2003 15:13:37) for ONLINE_EVALUATI ON_BETA1
Copyright 1988-2003 Comeau Computing. All rights reserved.
MODE:strict errors C++

"ComeauTest .c", line 7: error: call of an object of a class type without
appropriate operator() or conversion functions to
pointer-to-function type
initial_state(i ) = 0;
^

1 error detected in the compilation of "ComeauTest .c".
----------------------------------------------------------------------------
------

If the compiler did compile the code you posted, no wonder it core dumped.

You also make the same mistake in other parts of the program, and that is
using ( ) instead of [ ] when accessing or setting elements of the vector.

Paul

[dvir]

Paul wrote:

2)

vector<float> initial_state(3 );
for (unsigned int i = 0; i < initial_state.s ize (); ++ i)
initial_state (i) = 0; // This should have never compiled

There is no user-defined operator() for vector, but you're calling one.
What does this line do?

initial_state (i) = 0;

Shouldn't this be:

initial_state[i] = 0;

I used boost::numeric: :ublas::vector from the boost library
(www.boost.org)
Because I'm using linear algebra functions.
With ublas::vector It's right to write:
vec(i)=0;
Therefore, it did compile fine.
My runing error was while trying to do:
agent = new tdagent(env);
I still don't understand what is wrong with that line.

Thanks,
Dvir