add --gds option to watch command and include output in watch 1

sleyzerzon / soar

Automatically exported from code.google.com/p/soar

1 stars 0 forks source link

add --gds option to watch command and include output in watch 1 #76

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago

For a beginning Soar programmer (me) it can be a surprise to find that a state 
has been removed by the GDS. Usually something doesn't work so you type "p s2" 
and find that there's nothing there. 

 Soar does print messages about GDS state removals but only when WM changes are being traced.  I propose adding a --gds (-g) option to the watch command to control these messages directly. Also, I think these messages should be enabled at watch level 1. 

Thoughts? I've already implemented this in JSoar and find it helpful, but I'm 
willing to be persuaded otherwise.

Original issue reported on code.google.com by dave...@gmail.com on 12 Jun 2010 at 12:35

GoogleCodeExporter commented 8 years ago

Original comment by dave...@gmail.com on 12 Jun 2010 at 12:52

Added labels: Type-Enhancement
Removed labels: Type-Defect

GoogleCodeExporter commented 8 years ago

John is on board with this. I'll make it so.

Original comment by voigtjr@gmail.com on 16 Jun 2010 at 7:18

Changed state: Started

GoogleCodeExporter commented 8 years ago

Thanks. Maybe we can discuss it a little more in person. Now that I've used it 
a week in JSoar, it's a little chatty in "watch 1". Every time an operator 
retracts, a message is printed. Maybe there's a middle ground where it will 
only print when it's "unexpected". I'm not sure how to quantify that, but I 
know it's not when an operator produces a normal result.

Having the separate --gds trace category is definitely useful though. Maybe 
just not in watch level 1.

Original comment by dave...@gmail.com on 16 Jun 2010 at 7:55

GoogleCodeExporter commented 8 years ago

Just to follow up on our chat. I think we decided that I was seeing the 
messages because I was doing something wrong and that keeping them in watch 1 
is what we want.

Original comment by dave...@gmail.com on 17 Jun 2010 at 3:33

GoogleCodeExporter commented 8 years ago

Added to watch 2 after email from John:

I changed my mind. Watch 1 has always been just a trace of decisions – no 
warnings, etc. (that I know of). I think having the gds flag is fine and it can 
be on at watch 2 and above (and Dave can turn it on for himself all the time).

John

Original comment by voigtjr@gmail.com on 21 Jun 2010 at 3:45

Changed state: Fixed