Performance improvement - #19
Merged
Merged
Conversation
…tion - Added a new async trader service in `src/trader/__init__.py` to handle trading signals using aio-pika. - Replaced synchronous trading signal processing with an asynchronous approach, improving performance and scalability. - Introduced PGSQL as database. - Updated the message dispatcher to validate incoming messages against a JSON schema. - Modified the web server's message sending function to publish messages to the `trading-signals` queue instead of `trading-action`. - Ensured that the RabbitMQ queues are declared with the correct names and durability settings.
… support handling
…ient Co-authored-by: Copilot <copilot@github.com>
… tests Co-authored-by: Copilot <copilot@github.com>
…daily profit history methods - Added `get_closed_trade_history` and `get_daily_profit_history` methods to `DbPositionManager` and `AsyncDbPositionManager` for fetching closed trades and daily profit/loss data. - Updated `handle_daily_stats` in `position_monitor` and `trader` services to utilize new methods for generating daily trading reports. - Refactored message generation to use `build_daily_trading_report_message` for improved report formatting. - Introduced new tests for the trading report message and related calculations in `test_message_helper.py`.
…c and add comprehensive unit tests
…or improved performance
…ll related tests for TradingOrchestrator, InstrumentService, OrderService, PositionService, and PerformanceMonitor.
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
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.