Issue 162 in soar: incorrect parse of gp with dot

3 views
Skip to first unread message

so...@googlecode.com

unread,
Jun 16, 2014, 7:54:27 AM6/16/14
to soar-...@googlegroups.com
Status: New
Owner: ----
Labels: Type-Defect Priority-Medium

New issue 162 by garfield...@gmail.com: incorrect parse of gp with dot
http://code.google.com/p/soar/issues/detail?id=162

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).


--
You received this message because this project is configured to send all
issue notifications to this address.
You may adjust your notification preferences at:
https://code.google.com/hosting/settings

so...@googlecode.com

unread,
Jul 5, 2014, 3:36:23 AM7/5/14
to soar-...@googlegroups.com

Comment #1 on issue 162 by garfield...@gmail.com: incorrect parse of gp
with dot
http://code.google.com/p/soar/issues/detail?id=162

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

so...@googlecode.com

unread,
Jul 5, 2014, 1:04:23 PM7/5/14
to soar-...@googlegroups.com
Updates:
Status: Fixed

Comment #2 on issue 162 by maz...@gmail.com: incorrect parse of gp with dot
http://code.google.com/p/soar/issues/detail?id=162

Thanks for updating us that this is resolved!

so...@googlecode.com

unread,
Jul 5, 2014, 8:44:04 PM7/5/14
to soar-...@googlegroups.com

Comment #3 on issue 162 by garfield...@gmail.com: incorrect parse of gp
with dot
http://code.google.com/p/soar/issues/detail?id=162

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.
Reply all
Reply to author
Forward
0 new messages