Skip to content

Commit 186b43d

Browse files
committed
style: Format files from GPU memory fix merge
1 parent 3ec55de commit 186b43d

File tree

3 files changed

+17
-5
lines changed

3 files changed

+17
-5
lines changed

smoltrace/core.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121
"ignore", message=".*attention mask is not set.*", category=UserWarning, module="transformers.*"
2222
)
2323

24+
2425
def _cleanup_gpu_memory(verbose: bool = False):
2526
"""Frees GPU memory between test iterations to prevent OOM.
2627
@@ -37,7 +38,9 @@ def _cleanup_gpu_memory(verbose: bool = False):
3738
if verbose:
3839
allocated = torch.cuda.memory_allocated() / 1024**2
3940
reserved = torch.cuda.memory_reserved() / 1024**2
40-
print(f"[GPU] After cleanup: {allocated:.0f} MiB allocated, {reserved:.0f} MiB reserved")
41+
print(
42+
f"[GPU] After cleanup: {allocated:.0f} MiB allocated, {reserved:.0f} MiB reserved"
43+
)
4144
except ImportError:
4245
pass
4346

tests/test_core.py

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -524,7 +524,10 @@ def test_evaluate_single_test_failure_missing_keyword(mocker):
524524
mock_agent.run.return_value = "The weather is nice" # Missing "temperature"
525525

526526
# Mock analyze_streamed_steps
527-
mocker.patch("smoltrace.core.analyze_streamed_steps", return_value=(["get_weather"], True, 2, "The weather is nice"))
527+
mocker.patch(
528+
"smoltrace.core.analyze_streamed_steps",
529+
return_value=(["get_weather"], True, 2, "The weather is nice"),
530+
)
528531

529532
test_case = {
530533
"id": "test_003",

tests/test_core_additional.py

Lines changed: 9 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -228,7 +228,10 @@ def test_evaluate_single_test_success(mocker, capsys):
228228
}
229229

230230
# Mock analyze_streamed_steps
231-
mocker.patch("smoltrace.core.analyze_streamed_steps", return_value=(["get_weather"], True, 2, "The weather in Paris is sunny"))
231+
mocker.patch(
232+
"smoltrace.core.analyze_streamed_steps",
233+
return_value=(["get_weather"], True, 2, "The weather in Paris is sunny"),
234+
)
232235

233236
# Call with verbose=True to test print statements (lines 271-275, 340-343)
234237
result = evaluate_single_test(
@@ -275,7 +278,9 @@ def test_evaluate_single_test_with_tracer(mocker):
275278
mock_tracer.start_as_current_span.return_value.__enter__ = Mock(return_value=mock_span)
276279
mock_tracer.start_as_current_span.return_value.__exit__ = Mock(return_value=None)
277280

278-
mocker.patch("smoltrace.core.analyze_streamed_steps", return_value=(["tool1"], True, 1, "Response"))
281+
mocker.patch(
282+
"smoltrace.core.analyze_streamed_steps", return_value=(["tool1"], True, 1, "Response")
283+
)
279284

280285
evaluate_single_test(
281286
mock_agent,
@@ -308,7 +313,8 @@ def test_evaluate_single_test_with_multiple_expected_tools(mocker):
308313
}
309314

310315
mocker.patch(
311-
"smoltrace.core.analyze_streamed_steps", return_value=(["tool1", "tool2", "tool3"], True, 3, "Response")
316+
"smoltrace.core.analyze_streamed_steps",
317+
return_value=(["tool1", "tool2", "tool3"], True, 3, "Response"),
312318
)
313319

314320
result = evaluate_single_test(

0 commit comments

Comments
 (0)