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

(WIP) Semantic logging #75 #79

Open
wants to merge 3 commits into
base: development
Choose a base branch
from
Open

Conversation

Neverbolt
Copy link
Collaborator

@Neverbolt Neverbolt commented Aug 6, 2024

resolves #75

The goal here is to turn the very much ad-hoc and split logging (spread across the log_db and console) into a semantic logging utility.

  • Remove all references to log_db and console and replace them with semantic functions defined in Logger
  • Add configuration to run information
  • Make log be a global field, so that it doesn't need to be passed around manually
  • Position logging of LLM response streaming better (not sure if that is in the logger yet, the LLM or a third place entirely)
  • Handle multiple "conversations"
  • Add additional output possibilities for status changes
  • Add section definitions in logging for eg rounds or sub-parts of a round
  • Update log viewer to use the newly constructed logging tables
  • Remove old database tables and columns
  • Add spinner back in

MR should only be merged when the (WIP) is resolved

@Neverbolt Neverbolt self-assigned this Aug 6, 2024
@Neverbolt Neverbolt force-pushed the log_infrastructure branch 2 times, most recently from 5edb6f7 to 5b9cc54 Compare August 6, 2024 09:13
Also changes default value for DbStorage.connection_string fronm ":memory:" to "wintermute.sqlite3"
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

Successfully merging this pull request may close these issues.

1 participant