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

Codegate not properly picking latest messages from Open Interpreter #820

Closed
yrobla opened this issue Jan 29, 2025 · 0 comments · Fixed by #829
Closed

Codegate not properly picking latest messages from Open Interpreter #820

yrobla opened this issue Jan 29, 2025 · 0 comments · Fixed by #829
Assignees

Comments

@yrobla
Copy link
Contributor

yrobla commented Jan 29, 2025

It uses a new role "tool" that was not considered by our engine when extracting messages, so it was not properly picking the latest messages for analysis

@yrobla yrobla self-assigned this Jan 29, 2025
yrobla added a commit that referenced this issue Jan 29, 2025
It uses a new message role tool, that contains some context for using
internal tools and the data it provides. So we need to consider it when
parsing messages, or when inserting context messages

Closes: #820
yrobla added a commit that referenced this issue Jan 29, 2025
It uses a new message role tool, that contains some context for using
internal tools and the data it provides. So we need to consider it when
parsing messages, or when inserting context messages

Closes: #820
yrobla added a commit that referenced this issue Jan 31, 2025
* fix: consider the tool role when being in open interpreter

It uses a new message role tool, that contains some context for using
internal tools and the data it provides. So we need to consider it when
parsing messages, or when inserting context messages

Closes: #820

* fix lint

* fix tests

* change logic for open interpreter

* fix linting
yrobla added a commit that referenced this issue Jan 31, 2025
* fix: consider the tool role when being in open interpreter

It uses a new message role tool, that contains some context for using
internal tools and the data it provides. So we need to consider it when
parsing messages, or when inserting context messages

Closes: #820

* fix lint

* fix tests

* change logic for open interpreter

* fix linting

* fix: open intepreter not properly working with ollama

It needs fixes in cli commands

* fixes in chunk and cli formattig
lukehinds pushed a commit that referenced this issue Jan 31, 2025
* fix: consider the tool role when being in open interpreter

It uses a new message role tool, that contains some context for using
internal tools and the data it provides. So we need to consider it when
parsing messages, or when inserting context messages

Closes: #820

* fix lint

* fix tests

* change logic for open interpreter

* fix linting
lukehinds pushed a commit that referenced this issue Jan 31, 2025
* fix: consider the tool role when being in open interpreter

It uses a new message role tool, that contains some context for using
internal tools and the data it provides. So we need to consider it when
parsing messages, or when inserting context messages

Closes: #820

* fix lint

* fix tests

* change logic for open interpreter

* fix linting

* fix: open intepreter not properly working with ollama

It needs fixes in cli commands

* fixes in chunk and cli formattig
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants