cosmoharrigan / rl-glue

Automatically exported from code.google.com/p/rl-glue
0 stars 0 forks source link

Safe way to refactor the header system for core RL-Glue #53

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
For some reason, every agent and environment that supports RL-Glue is
required to re-specify all of the agent_ or env_ methods in it's header file.

These are also defined in RLGlue.h file.  I think maybe we can move them to
some shared files:

agent_common.h
env_common.h

Both of these could include RL_common.h

Then, agents and environments still only have to include 1 file (same as
now), and they work.  Old agents still work.  BUT!  Added bonus.  We can
extern "C" the agent and environments in the common files, so any new
agents or environments that use them will be able to be dynamically loaded
through RL-Viz with no source code changes. Hooray!

Original issue reported on code.google.com by brian.ta...@gmail.com on 17 Jan 2008 at 7:01

GoogleCodeExporter commented 9 years ago
I think this is done, but needs to be pushed out in a release.

Original comment by brian.ta...@gmail.com on 29 Jan 2008 at 6:29

GoogleCodeExporter commented 9 years ago

Original comment by brian.ta...@gmail.com on 29 Jan 2008 at 6:30

GoogleCodeExporter commented 9 years ago
I feel like this needs to be looked at to be sure.

Original comment by brian.ta...@gmail.com on 2 Sep 2008 at 10:10

GoogleCodeExporter commented 9 years ago
This is done.  See here:
http://glue.rl-community.org/changes

Have also added the appropriate macros so that the code can be included and 
linked
from C or C++.

Original comment by brian.ta...@gmail.com on 7 Sep 2008 at 12:22