Skip to content

Commit 6870b4f

Browse files
committed
refactor: remove analytics_dir from sceptre.py
This results in .../exp/elk/ instead of .../exp/analytics/elk/ for elk-related files. This reduces complexity a bit, and less nesting in the experiment directory. As far as I can tell, nothing directly uses the files in the experiment directory, beyond than the code in sceptre.py for injections.
1 parent 628a799 commit 6870b4f

File tree

1 file changed

+7
-9
lines changed

1 file changed

+7
-9
lines changed

src/python/phenix_apps/apps/sceptre/sceptre.py

Lines changed: 7 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -15,15 +15,13 @@ def __init__(self):
1515

1616
self.eprint(self.stage)
1717

18-
self.startup_dir = f"{self.exp_dir}/startup"
19-
self.sceptre_dir = f"{self.exp_dir}/sceptre"
20-
self.analytics_dir = f"{self.exp_dir}/analytics" # TODO: unused?
21-
self.elk_dir = f"{self.analytics_dir}/elk"
22-
23-
os.makedirs(self.startup_dir, exist_ok=True)
24-
os.makedirs(self.sceptre_dir, exist_ok=True)
25-
os.makedirs(self.analytics_dir, exist_ok=True)
26-
os.makedirs(self.elk_dir, exist_ok=True)
18+
self.startup_dir = f"{self.exp_dir}/startup"
19+
self.sceptre_dir = f"{self.exp_dir}/sceptre"
20+
self.elk_dir = f"{self.exp_dir}/elk"
21+
22+
os.makedirs(self.startup_dir, exist_ok=True)
23+
os.makedirs(self.sceptre_dir, exist_ok=True)
24+
os.makedirs(self.elk_dir, exist_ok=True)
2725

2826
self.mako_path = utils.abs_path(__file__, "templates") # type: str
2927

0 commit comments

Comments
 (0)