sleyzerzon / soar

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

incorrect parse of gp with dot #162

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Using Soar 9.3.2 on Windows 7.

The production below does not parse at all, leading me to believe that dot 
notation with gp patterns is not supposed to be supported:

gp {gp*fail4
(state <s> ^[att1 att2 <foo>].[bar baz qux] val)  #can't use it with dots like 
this
-->
(<s> ^foo <foo>)
}

For good measure, though, I tried the following:

gp {gp*test1
(state <s> ^foo.[bar baz qux] val)
-->
(<s> ^foo <foo>)
}

Soar accepts this as legal, so I would expect there to be three productions 
using ^foo.bar, ^foo.baz and ^foo.qux. Instead, it produces three productions 
with ^bar, ^baz and ^qux, which is clearly incorrect. The one below yields 
unexpected results as well:

gp {gp*test2
(state <s> ^[bar baz qux].foo val)
-->
(<s> ^foo <foo>)
}

This gives three productions, all using ^foo (so they are identical 
productions).

Original issue reported on code.google.com by garfield...@gmail.com on 16 Jun 2014 at 11:54

GoogleCodeExporter commented 9 years ago
Dot notation with gp patterns is supported and works properly in 9.3.4, so this 
issue can be closed.

Original comment by garfield...@gmail.com on 5 Jul 2014 at 7:36

GoogleCodeExporter commented 9 years ago
Thanks for updating us that this is resolved!

Original comment by maz...@gmail.com on 5 Jul 2014 at 5:04

GoogleCodeExporter commented 9 years ago
Sure. This is fixed as near as I can tell, anyway, since my tests all work on 
9.3.4. I can't point to a specific commit or anything in the change log. 

Original comment by garfield...@gmail.com on 6 Jul 2014 at 12:43