Skip to content

Commit c639936

Browse files
jeremymanningclaude
andcommitted
Fix code formatting and linting issues
- Ran black formatter on clustrix/ and tests/ directories - Fixed 31 flake8 errors in tests/ including: - Removed unused imports (F401 errors) - Converted lambda assignments to proper functions (E731 errors) - Fixed redefinition of unused imports (F811 errors) - Removed trailing whitespace - All 223 tests continue to pass - Documentation builds successfully (36 warnings about missing pages) - Note: mypy has 45 type annotation errors that require extensive work 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 4522d41 commit c639936

File tree

14 files changed

+89
-15
lines changed

14 files changed

+89
-15
lines changed

clustrix/config.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -90,26 +90,26 @@ def save_to_file(self, config_path: str) -> None:
9090
"""Save this configuration instance to a file."""
9191
config_path = Path(config_path)
9292
config_data = asdict(self)
93-
93+
9494
with open(config_path, "w") as f:
9595
if config_path.suffix.lower() in [".yml", ".yaml"]:
9696
yaml.dump(config_data, f, default_flow_style=False)
9797
else:
9898
json.dump(config_data, f, indent=2)
99-
99+
100100
@classmethod
101101
def load_from_file(cls, config_path: str) -> "ClusterConfig":
102102
"""Load configuration from a file and return a new instance."""
103103
config_path = Path(config_path)
104104
if not config_path.exists():
105105
raise FileNotFoundError(f"Configuration file not found: {config_path}")
106-
106+
107107
with open(config_path, "r") as f:
108108
if config_path.suffix.lower() in [".yml", ".yaml"]:
109109
config_data = yaml.safe_load(f)
110110
else:
111111
config_data = json.load(f)
112-
112+
113113
return cls(**config_data)
114114

115115

5.13 KB
Binary file not shown.
3.07 KB
Binary file not shown.

docs/build/html/_modules/clustrix/config.html

Lines changed: 34 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -219,7 +219,39 @@ <h1>Source code for clustrix.config</h1><div class="highlight"><pre>
219219
<span class="k">if</span> <span class="bp">self</span><span class="o">.</span><span class="n">module_loads</span> <span class="ow">is</span> <span class="kc">None</span><span class="p">:</span>
220220
<span class="bp">self</span><span class="o">.</span><span class="n">module_loads</span> <span class="o">=</span> <span class="p">[]</span>
221221
<span class="k">if</span> <span class="bp">self</span><span class="o">.</span><span class="n">pre_execution_commands</span> <span class="ow">is</span> <span class="kc">None</span><span class="p">:</span>
222-
<span class="bp">self</span><span class="o">.</span><span class="n">pre_execution_commands</span> <span class="o">=</span> <span class="p">[]</span></div>
222+
<span class="bp">self</span><span class="o">.</span><span class="n">pre_execution_commands</span> <span class="o">=</span> <span class="p">[]</span>
223+
224+
<div class="viewcode-block" id="ClusterConfig.save_to_file">
225+
<a class="viewcode-back" href="../../api/config.html#clustrix.config.ClusterConfig.save_to_file">[docs]</a>
226+
<span class="k">def</span> <span class="nf">save_to_file</span><span class="p">(</span><span class="bp">self</span><span class="p">,</span> <span class="n">config_path</span><span class="p">:</span> <span class="nb">str</span><span class="p">)</span> <span class="o">-&gt;</span> <span class="kc">None</span><span class="p">:</span>
227+
<span class="w"> </span><span class="sd">&quot;&quot;&quot;Save this configuration instance to a file.&quot;&quot;&quot;</span>
228+
<span class="n">config_path</span> <span class="o">=</span> <span class="n">Path</span><span class="p">(</span><span class="n">config_path</span><span class="p">)</span>
229+
<span class="n">config_data</span> <span class="o">=</span> <span class="n">asdict</span><span class="p">(</span><span class="bp">self</span><span class="p">)</span>
230+
231+
<span class="k">with</span> <span class="nb">open</span><span class="p">(</span><span class="n">config_path</span><span class="p">,</span> <span class="s2">&quot;w&quot;</span><span class="p">)</span> <span class="k">as</span> <span class="n">f</span><span class="p">:</span>
232+
<span class="k">if</span> <span class="n">config_path</span><span class="o">.</span><span class="n">suffix</span><span class="o">.</span><span class="n">lower</span><span class="p">()</span> <span class="ow">in</span> <span class="p">[</span><span class="s2">&quot;.yml&quot;</span><span class="p">,</span> <span class="s2">&quot;.yaml&quot;</span><span class="p">]:</span>
233+
<span class="n">yaml</span><span class="o">.</span><span class="n">dump</span><span class="p">(</span><span class="n">config_data</span><span class="p">,</span> <span class="n">f</span><span class="p">,</span> <span class="n">default_flow_style</span><span class="o">=</span><span class="kc">False</span><span class="p">)</span>
234+
<span class="k">else</span><span class="p">:</span>
235+
<span class="n">json</span><span class="o">.</span><span class="n">dump</span><span class="p">(</span><span class="n">config_data</span><span class="p">,</span> <span class="n">f</span><span class="p">,</span> <span class="n">indent</span><span class="o">=</span><span class="mi">2</span><span class="p">)</span></div>
236+
237+
238+
<div class="viewcode-block" id="ClusterConfig.load_from_file">
239+
<a class="viewcode-back" href="../../api/config.html#clustrix.config.ClusterConfig.load_from_file">[docs]</a>
240+
<span class="nd">@classmethod</span>
241+
<span class="k">def</span> <span class="nf">load_from_file</span><span class="p">(</span><span class="bp">cls</span><span class="p">,</span> <span class="n">config_path</span><span class="p">:</span> <span class="nb">str</span><span class="p">)</span> <span class="o">-&gt;</span> <span class="s2">&quot;ClusterConfig&quot;</span><span class="p">:</span>
242+
<span class="w"> </span><span class="sd">&quot;&quot;&quot;Load configuration from a file and return a new instance.&quot;&quot;&quot;</span>
243+
<span class="n">config_path</span> <span class="o">=</span> <span class="n">Path</span><span class="p">(</span><span class="n">config_path</span><span class="p">)</span>
244+
<span class="k">if</span> <span class="ow">not</span> <span class="n">config_path</span><span class="o">.</span><span class="n">exists</span><span class="p">():</span>
245+
<span class="k">raise</span> <span class="ne">FileNotFoundError</span><span class="p">(</span><span class="sa">f</span><span class="s2">&quot;Configuration file not found: </span><span class="si">{</span><span class="n">config_path</span><span class="si">}</span><span class="s2">&quot;</span><span class="p">)</span>
246+
247+
<span class="k">with</span> <span class="nb">open</span><span class="p">(</span><span class="n">config_path</span><span class="p">,</span> <span class="s2">&quot;r&quot;</span><span class="p">)</span> <span class="k">as</span> <span class="n">f</span><span class="p">:</span>
248+
<span class="k">if</span> <span class="n">config_path</span><span class="o">.</span><span class="n">suffix</span><span class="o">.</span><span class="n">lower</span><span class="p">()</span> <span class="ow">in</span> <span class="p">[</span><span class="s2">&quot;.yml&quot;</span><span class="p">,</span> <span class="s2">&quot;.yaml&quot;</span><span class="p">]:</span>
249+
<span class="n">config_data</span> <span class="o">=</span> <span class="n">yaml</span><span class="o">.</span><span class="n">safe_load</span><span class="p">(</span><span class="n">f</span><span class="p">)</span>
250+
<span class="k">else</span><span class="p">:</span>
251+
<span class="n">config_data</span> <span class="o">=</span> <span class="n">json</span><span class="o">.</span><span class="n">load</span><span class="p">(</span><span class="n">f</span><span class="p">)</span>
252+
253+
<span class="k">return</span> <span class="bp">cls</span><span class="p">(</span><span class="o">**</span><span class="n">config_data</span><span class="p">)</span></div>
254+
</div>
223255

224256

225257

@@ -351,7 +383,7 @@ <h1>Source code for clustrix.config</h1><div class="highlight"><pre>
351383
&copy; Copyright 2025, Contextual Dynamics Laboratory
352384
</div>
353385
<div class="text-center text-white-50 font-italic mt-3">
354-
<small>Last updated: Jun 25, 2025</small>
386+
<small>Last updated: Jun 26, 2025</small>
355387
<small>
356388
Created using
357389
<a class="text-white" href="https://github.com/wagtail/sphinx_wagtail_theme" rel="nofollow" target="_blank">

docs/build/html/_modules/index.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -160,7 +160,7 @@ <h1>All modules for which code is available</h1>
160160
&copy; Copyright 2025, Contextual Dynamics Laboratory
161161
</div>
162162
<div class="text-center text-white-50 font-italic mt-3">
163-
<small>Last updated: Jun 25, 2025</small>
163+
<small>Last updated: Jun 26, 2025</small>
164164
<small>
165165
Created using
166166
<a class="text-white" href="https://github.com/wagtail/sphinx_wagtail_theme" rel="nofollow" target="_blank">

0 commit comments

Comments
 (0)