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

BattleTool and full message collection instead of summary #102

Merged
merged 26 commits into from
Nov 13, 2024

Conversation

KarmaKamikaze
Copy link
Owner

@KarmaKamikaze KarmaKamikaze commented Nov 1, 2024

  • We add the BattleTool, which will handle combat and battles between characters in the narrative
    • It is possible to do sparring fights and mock battle doing harmless damage
  • We provide an option to disable summarization of the narrative, instead sending the entire message collection each query
    • This was added to properly compare the ReAct version of ChatRPG to the non-ReAct version.
  • FindCharacter is better at finding the right character, using context messages, while returning no character when appropriate
  • Added examples to FindCharacter instructions
  • Switch to GPT-4o model
  • Removed markdown in responses
  • Fix environment and character loading after the initial campaign is presented by moving it to AfterRender
  • Made saving run as a task so player can input their next action faster
    • Added a progress spinner to indicate when the archivist is saving the campaign behind the scenes

summarize the narrative, instead sending the entire message collection
@KarmaKamikaze KarmaKamikaze added the enhancement New feature or request label Nov 1, 2024
@KarmaKamikaze KarmaKamikaze linked an issue Nov 1, 2024 that may be closed by this pull request
@@ -79,7 +79,7 @@ public ReActAgentChain(
string? gameSummary = null,
string inputKey = "input",
string outputKey = "text",
int maxActions = 10)
int maxActions = 20)
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pump Up Those Numbers!!!

@KarmaKamikaze KarmaKamikaze marked this pull request as ready for review November 8, 2024 10:41
This was linked to issues Nov 11, 2024
…ting. (This does make it seem a bit weird to the user though as they dont know when the game is finished updating the campaign with new characters.)
@KarmaKamikaze KarmaKamikaze merged commit c7c0137 into dev Nov 13, 2024
5 checks passed
@KarmaKamikaze KarmaKamikaze deleted the feature/BattleTool branch November 13, 2024 13:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
3 participants