charieswei
/
TradingAgents
mirror of https://github.com/TauricResearch/TradingAgents.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix: write only current trade_date state to log file, not full history 

Each log file is named for a specific trade_date, so it should only
contain that date's state. Writing self.log_states_dict caused O(N^2)
storage and I/O as every daily file redundantly included all previously
processed dates.
Suggested by GitHub Copilot review on PR #499.
This commit is contained in:
tiffanychum 2026-04-02 01:23:25 +08:00
parent ce241998c6
commit 6eceb84d98
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
tradingagents/graph/trading_graph.py
View File

@ -264,7 +264,7 @@ class TradingAgentsGraph:
log_path = directory / f"full_states_log_{trade_date}.json"
with open(log_path, "w", encoding="utf-8") as f:
json.dump(self.log_states_dict, f, indent=4)
json.dump(self.log_states_dict[str(trade_date)], f, indent=4)
def reflect_and_remember(self, returns_losses):
"""Reflect on decisions and update memory based on returns."""