search

davidljung / rl-glue

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

String Observations #74

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago 
It's come up before (RTS) and it will probably come up again.

For domains with crazy complex or structured observations, it might make
sense to have a third array in the observation, char*.  This will not break
backwards compat (people that don't use it won't notice it).

This will not take too much to implement.

Original issue reported on code.google.com by brian.ta...@gmail.com on 7 Sep 2008 at 6:19

GoogleCodeExporter commented 8 years ago

Original comment by brian.ta...@gmail.com on 7 Sep 2008 at 6:26

GoogleCodeExporter commented 8 years ago 
Good idea. I support it!

Original comment by whitiena...@gmail.com on 8 Sep 2008 at 5:00

GoogleCodeExporter commented 8 years ago 
I've updated the code in RL-Glue.  It's possible the only things I needed to 
change
were the definition of the RL_abstract_type_t in RL_common.h and 
rlCopyBufferToADT
rlCopyADTToBuffer in RL_network.c.

If that's the case, Andrew is my hero.

Original comment by brian.ta...@gmail.com on 8 Sep 2008 at 9:27

GoogleCodeExporter commented 8 years ago 
That was all I had to do.  Fixed a small bug from r793 and it now all seems to 
work
in r794.

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