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

Message class should be immutable/dataclass #124

Closed
ErikBjare opened this issue Sep 18, 2024 · 1 comment
Closed

Message class should be immutable/dataclass #124

ErikBjare opened this issue Sep 18, 2024 · 1 comment

Comments

@ErikBjare
Copy link
Owner

I've had a few issues with logs getting mutated by preprocessing before sending off to LLMs, not good.

There has been a TODO above the class for a while, should get around to it.

@ErikBjare ErikBjare changed the title Message class should be immutable Message class should be immutable/dataclass Sep 18, 2024
@ErikBjare
Copy link
Owner Author

Completed in #125

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant