search

sleyzerzon / soar

Automatically exported from code.google.com/p/soar
1 stars 0 forks source link

Possibly remove deep copy RHS function #4

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago 
Description From Karen Coulter 2004-12-17 14:05:50 (-) [reply]
this functionality is either not general enough to be part of the supported
Soar
distribution, or needs Soar Consortium approval before being added.

-- "deepCopy" is a new RHS function that is deeply embedded in the kernel,
and is commented as a "HACK that should be doable through the external
kernel interface"    It creates a copy of a wme, makes a fake pref, and 
then
deallocates the wme, all in the matcher (create_instantiation) and also
adds references to objects.  I haven't investigated the cost of this code,
but it has potential to be significant.  I think this is the sort of change 
that should go to the Soar Consortium if SoarTech wants it to stay in the
distribution.  Certainly they can add it back in to their versions easily.

Nov 29:  ==>> should go thru Consortium for sure.  There may be other ways 
to
implement this, and alternatives should be evaluated.  Remove from kernel.

-- O_REJECTS_FIRST compiler directive added.  I think we should ask
SoarTech whether they still use this or whether it was for some 
experiments.
It doesn't cost anything since it's a compiler directive, but it is one 
more
option to support and understand.

Nov 29:  yep, we should take it out. check w/ SoarTech if even being used
regularly.
------- Comment #1 From Bob Marinier 2004-12-17 20:18:32 (-) [reply] ------
-
We added O_REJECTS_FIRST in Soar 8.5.0, and I don't think we want it taken 
out.
 It allows us to write fewer productions in some cases.  See the 8.5.0 
release
notes for details.
------- Comment #2 From Karen Coulter 2005-11-10 11:47:30 (-) [reply] -----
--
*** Bug 586 has been marked as a duplicate of this bug. ***
------- Comment #3 From Bob Marinier 2005-11-10 11:49:02 (-) [reply] ------
-
Sorry, forgot this already existed.
------- Comment #4 From Jonathan Voigt 2009-05-11 11:10:10 (-) [reply] ----

---
Can't remove this, Soar Tech uses it a lot. Possibly re-implement for next
release.
------- Comment #5 From Jonathan Voigt 2009-05-11 12:13:07 (-) [reply] ----

---
http://code.google.com/p/jsoar/source/browse/trunk/jsoar-
core/src/main/java/org/jsoar/kernel/rhs/functions/DeepCopy.java

Original issue reported on code.google.com by voigtjr@gmail.com on 23 Jul 2009 at 4:04

GoogleCodeExporter commented 9 years ago

Original comment by voigtjr@gmail.com on 23 Jul 2009 at 5:29

GoogleCodeExporter commented 9 years ago

Original comment by voigtjr@gmail.com on 23 Feb 2010 at 7:41