From b7044ca4b45f3d40676495856be647ed578790b8 Mon Sep 17 00:00:00 2001 From: Joshua Bloom Date: Mon, 13 Apr 2026 07:21:24 -0400 Subject: [PATCH] fix(main): remove dangling data_logger reference left over from pkl removal MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit PR #10 (feat/hdf5-data-pipeline) removed the pkl writer import but missed two lines that still constructed a data_writer per episode. Every job crashes immediately at episode 0 with: NameError: name 'data_logger' is not defined The two lines were dead code — data_writer was constructed but never actually written to anywhere else in Main.py after the pkl removal. Surfaced by a smoke test of the new diagnostic logging stack on Stelaris main, which submits a real job through the dispatcher to RL-CT master. Co-Authored-By: Claude Opus 4.6 (1M context) --- rl_code/Main.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/rl_code/Main.py b/rl_code/Main.py index 08a9c29..bd41c12 100644 --- a/rl_code/Main.py +++ b/rl_code/Main.py @@ -172,8 +172,6 @@ exp_done, episode_done, reached_goal = Utility.parse_status(msgs[0]) socket.set_episode(ep_counter) log.info("Episode %d starting", ep_counter) - data_file_name = 'Data_Episode_'+str(ep_counter)+'.pkl' - data_writer = data_logger(data_file_path+data_file_name) if not exp_done: time_steps = 0