Lee Miller

This is related to #998 right @poweihuang0817 ?

This seems like an ActionPlanner bug. It is placing `SoP_High_Memory` into the Plan.State which gets overridden by the input when the plan is executed (`Yes, proceed`). LIke `SequentialPlanner`, this should...

There's definitely work we want to do to improve the visibility of outputs from functions. In the short term, I'd try using the `Plan` object. ` var prompt = "Write...

Closing old issues, please re-open if you have more to add to the issue.

This is great, will be glad to help with a closer review when appropriate. Some thoughts below on your questions: > in which namespace it should reside? For `IterativePlanner`, I'd...

> Hi Comunity, > > I belive I have a good first candidate, so please let me know what else do I need to adapt / fix. I added two...

I think a key requirement of being a `Planner` is that ` Plan` object is created, either semantically or otherwise. I definitely think having this iterative/mrkl approach is worth incorporating....

For sake of visibility, some proposed changes can be seen in the following commit (and others in the branch it's contained in). https://github.com/lemillermicrosoft/semantic-kernel/commit/88a7379b36621d681107ac6cfe13f24e7b471d1f

Hey @kaza -- I've taken a stab at breaking up the various changes into smaller PRs. Let me know if you have preference/desire in proceeding forward with this PR or...

Closing for now. Please don't hesitate to re-open as appropriate @kaza