Skip to content

.Net Agents: Reducing the ChatCompletionAgent history size invalidates invocation logic #7321

New issue
New issue
Closed
#7570
Closed
.Net Agents: Reducing the ChatCompletionAgent history size invalidates invocation logic#7321
#7570
Assignees
crickman
Labels
.NETIssue or Pull requests regarding .NET codeagentsbugSomething isn't workingexperimentalAssociated with an experimental feature

Description

@crickman
Chris (crickman)
opened on Jul 17, 2024

Description
Currently ChatCompletionAgent assumes a forward only model when evaluating mutated chat-history.

https://github.com/microsoft/semantic-kernel/blob/main/dotnet/src/Agents/Core/ChatCompletionAgent.cs#L47

Should the history be reduced in size this will create an issue in maintaining the fidelity of the chat. This particulary impacts function calling.

** Work-around **
Caller doesn't modify the history during invocation (e.g. within a filter)

Additional context
To be addressed as part of history truncation feature: #7320

Activity

  1. Chris (crickman)
    added
    bugSomething isn't working
    on Jul 17, 2024
  2. Chris (crickman)
    added this to Semantic Kernelon Jul 17, 2024
  3. Mark Wallace (markwallace-microsoft)
    added
    .NETIssue or Pull requests regarding .NET code
    triage
    on Jul 17, 2024
  4. Chris (crickman)
    added
    agents
    experimentalAssociated with an experimental feature
    .NETIssue or Pull requests regarding .NET code
    and removed
    .NETIssue or Pull requests regarding .NET code
    on Jul 17, 2024
  5. Chris (crickman)
    moved this to Sprint: Planned in Semantic Kernelon Jul 17, 2024
  6. Chris (crickman)
    moved this from Sprint: Planned to Backlog: Planned in Semantic Kernelon Jul 17, 2024
  7. Chris (crickman)
    removed
    triage
    on Jul 17, 2024
  8. Chris (crickman)
    linked a pull request that will close this issue.Net Agents - ChatHistory Reducer Pattern #7570on Jul 31, 2024
  9. Chris (crickman)
    moved this from Backlog: Planned to Sprint: In Progress in Semantic Kernelon Aug 1, 2024
  10. Chris (crickman)
    self-assigned this
    on Aug 1, 2024
  11. Chris (crickman)
    moved this from Sprint: In Progress to Sprint: In Review in Semantic Kernelon Aug 1, 2024
  12. Chris (crickman)
    closed this as completedin #7570on Aug 7, 2024
  13. github-project-automation
    moved this from Sprint: In Review to Sprint: Done in Semantic Kernelon Aug 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

.NETIssue or Pull requests regarding .NET codeagentsbugSomething isn't workingexperimentalAssociated with an experimental feature

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions