Skip to content

ModelInferenceLogsDatabase cleanup#1930

Open
rithvik-doshi wants to merge 24 commits intodevfrom
remove-model-id
Open

ModelInferenceLogsDatabase cleanup#1930
rithvik-doshi wants to merge 24 commits intodevfrom
remove-model-id

Conversation

@rithvik-doshi
Copy link
Contributor

@rithvik-doshi rithvik-doshi commented Jan 6, 2026

  • Remove unused fields from the room table, add agent type to messages
  • Add checks to physical relation and determine discrepancies in columns in the owl creator.
    • Accordingly, generate owl and let the model utils init function know if there are any changes made so that migrations can take place.
  • Fixes unit tests

To test, restart server, ensure database changes are made, and test playground and ensure things operate normally.

Expected changes:

  • removal of model_id, agent_id, agent_type columns from room table
  • addition of agent_type column to message table
  • all unit tests should pass

Note: Removes support for usage of room.model_id or room.agent_id across the platform. For usage tracking, use message.agent_id

@rithvik-doshi rithvik-doshi requested a review from a team as a code owner January 6, 2026 17:03
@rithvik-doshi rithvik-doshi marked this pull request as draft January 9, 2026 13:42
@rithvik-doshi rithvik-doshi changed the title Remove model id and agent id from room table ModelInferenceLogsDatabase cleanup Jan 20, 2026
@rithvik-doshi rithvik-doshi marked this pull request as ready for review January 21, 2026 15:17
@rithvik-doshi rithvik-doshi self-assigned this Jan 24, 2026
@rithvik-doshi rithvik-doshi marked this pull request as draft January 24, 2026 15:10
@rithvik-doshi rithvik-doshi marked this pull request as ready for review January 26, 2026 18:35
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.

3 participants