Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Feature Request] Refactor code to simplify the concepts #163

Closed
7 of 11 tasks
Obs01ete opened this issue Jun 14, 2023 · 2 comments
Closed
7 of 11 tasks

[Feature Request] Refactor code to simplify the concepts #163

Obs01ete opened this issue Jun 14, 2023 · 2 comments
Assignees
Labels
Agent Related to camel agents API Modifies the API enhancement New feature or request Message Related to camel messages Role Related to camel roles

Comments

@Obs01ete
Copy link
Collaborator

Obs01ete commented Jun 14, 2023

Motivation

We need to clean up code so that the API is easier to study and use. List of required changes:

Solution

No response

Alternatives

No response

Additional context

No response

@Obs01ete
Copy link
Collaborator Author

I think mostly the refectoring is done. The remaining open points we will address later on.

@dandansamax
Copy link
Collaborator

dandansamax commented Aug 31, 2023

Why should we remove role from ChatMessage? I'm planning to replace ChatAgent.stored_messages with memory module. It's inconvenient for memory API design if not all message-related information stores in BaseMessage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Agent Related to camel agents API Modifies the API enhancement New feature or request Message Related to camel messages Role Related to camel roles
Projects
Archived in project
Development

No branches or pull requests

3 participants