Merged
Conversation
…ng SQLAlchemy 2.0
The changes introduce async/await pattern and proper repository abstractions: - Add AsyncTyper to support async CLI commands - Create abstract Repository base class with async interface - Implement in-memory repositories for project and experiment runs - Split run repositories into separate interfaces - Remove SQL-specific repository implementation - Update runtime and services to use async/await pattern - Clean up model definitions and remove unused code
fix: Add type hints to plan_test.py
The test now properly asserts the number of experiments instead of just printing the project, and updates the import path to reflect new project structure
fix: Handle tilde expansion in logging configuration
The changes include: - Remove template string substitution in favor of direct replacement - Simplify event handling with generic Event type - Remove progress display temporarily - Hardcode log file path instead of making it configurable - Add type safety to event handlers
The changes include: - Replace manual DI with dishka container setup - Restructure CLI commands to use dependency injection - Add proper UI service integration with DI - Improve execution plan string representation - Update project dependencies and Python version requirement
The changes introduce a new messaging system to handle runtime events in a more decoupled way, replacing direct event handling with a message bus pattern. This improves separation of concerns and makes the system more maintainable.
rorybyrne
deleted the
5-feat-run-experiments-via-a-supplied-script-path
branch
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.