Nice spotting Ari... In my skim read of Gama's report I missed this. Since all AtomSpace/TLB calls now go through the AtomTableWrapper PLN will continue to function assuming that the Handle class can be hashed for the real->pln/fake handle map and vice versa.
J On Wed, Oct 22, 2008 at 12:28 AM, Ari Heljakka <[EMAIL PROTECTED]> wrote: > Hi Gama, > >> * Started investigating another suggestion from the Petaverse port: >> replace the Handle typedef with a real class. This should help us find >> out several semantic bugs in the code that rely on the assumption that >> Handle == unsigned long. > > I remind that the vtree system relies on the assumption that Handles can be > converted to (long) integers. There is a natural way to get rid of this > assumption by extending the Vertex variant to contain also your new Handle > class (instead of containing the Handle-integer) and then rewrite the code > that relies on the integer assumption, correspondingly. > > The code that relies on this assumption includes most significantly PLN, and > less significantly the space-time schemata used by the reasoning module, and > maybe some of the older MOSES code. > > -- > Ari Heljakka > CTO > Dream Broker Ltd > > Tekniikantie 14 [EMAIL PROTECTED] > 02150 Espoo +358 40 568 2420 > Finland www.dreambroker.fi > > _______________________________________________ > Mailing list: https://launchpad.net/~opencog-dev > Post to : [email protected] > Unsubscribe : https://launchpad.net/~opencog-dev > More help : https://help.launchpad.net/ListHelp > _______________________________________________ Mailing list: https://launchpad.net/~opencog-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~opencog-dev More help : https://help.launchpad.net/ListHelp
