-
Notifications
You must be signed in to change notification settings - Fork 0
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
fix: Refactor GitHub plugin actions to improve state management and update… #123
fix: Refactor GitHub plugin actions to improve state management and update… #123
Conversation
snobbee
commented
Dec 18, 2024
… model class usage - Removed unnecessary client references in the logging-addict character file. - Updated various GitHub plugin actions to use ModelClass.SMALL instead of ModelClass.LARGE for efficiency. - Update template file to include character content and PR examples.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome thanks for addressing the discussed changes can we confirm that the context gets properly configured with the relevant files and that the issues incorporate the context? Thank you!
@monilpat result below. I still need to figure why the character file content is not added in the context, also memory and files do not appear for the later I might need to fix the The github issue got created though with a logging related feature request see here Sifchain/stf-nomic-game#38
|
Thanks for doing this - so this is where I got to as well it is not interpolating the character details and files which is the main thing so getting it added to the state properly and added to the context is the key here |
I managed to get the code in a place where the memories from files are retrieved from the create issue action, the issue was around the room id, the memory id was set a room id making it impossible to retrieve the list of files. I used a simple repo with 2 files to avoid hitting the context window limit but will continue my tests with a bigger repo. You can see in the results, that the template contains now the character data, the list of files and their content and the recent messages (memory) I also added few more important info about the character. |
Awesome thanks so much for getting this to a working state. Yes please feel free to refine this and get it to generate more useful content with code inlined and such and add more examples of issues that are fully fleshed out! Thanks for doing this! |
b323a21
into
realitySpiral/contextualizedActions