Skip to content

Commit 5d0ca7a

Browse files
committed
Fix lint
1 parent a9f3296 commit 5d0ca7a

File tree

1 file changed

+3
-11
lines changed

1 file changed

+3
-11
lines changed

sdgym/result_explorer/result_explorer.py

Lines changed: 3 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -25,11 +25,6 @@ def _validate_local_path(path):
2525
}
2626

2727

28-
def _get_baseline_synthesizer(modality):
29-
"""Return the appropriate baseline synthesizer for the given modality."""
30-
return _BASELINE_BY_MODALITY.get(modality, SYNTHESIZER_BASELINE)
31-
32-
3328
def _resolve_effective_path(path, modality):
3429
"""Append the modality folder to the given base path if provided."""
3530
if not modality:
@@ -51,11 +46,10 @@ def _resolve_effective_path(path, modality):
5146
class ResultsExplorer:
5247
"""Explorer for SDGym benchmark results, supporting both local and S3 storage."""
5348

54-
def _create_results_handler(self, original_path, effective_path, baseline_synthesizer):
49+
def _create_results_handler(self, original_path, effective_path):
5550
"""Create the appropriate results handler for local or S3 storage."""
51+
baseline_synthesizer = _BASELINE_BY_MODALITY.get(self.modality, SYNTHESIZER_BASELINE)
5652
if is_s3_path(original_path):
57-
# Use original path to obtain client (keeps backwards compatibility),
58-
# but handler should operate on the modality-specific effective path.
5953
s3_client = _get_s3_client(
6054
original_path, self.aws_access_key_id, self.aws_secret_access_key
6155
)
@@ -71,9 +65,8 @@ def __init__(self, path, modality, aws_access_key_id=None, aws_secret_access_key
7165
self.modality = modality.lower()
7266
self.aws_access_key_id = aws_access_key_id
7367
self.aws_secret_access_key = aws_secret_access_key
74-
baseline_synthesizer = _get_baseline_synthesizer(modality)
7568
effective_path = _resolve_effective_path(path, modality)
76-
self._handler = self._create_results_handler(path, effective_path, baseline_synthesizer)
69+
self._handler = self._create_results_handler(path, effective_path)
7770

7871
def list(self):
7972
"""List all runs available in the results directory."""
@@ -112,7 +105,6 @@ def load_synthetic_data(self, results_folder_name, dataset_name, synthesizer_nam
112105

113106
def load_real_data(self, dataset_name):
114107
"""Load the real data for a given dataset."""
115-
# Keep strict validation for single_table to preserve existing behavior
116108
if (self.modality is None or self.modality == 'single_table') and (
117109
dataset_name not in DEFAULT_SINGLE_TABLE_DATASETS
118110
):

0 commit comments

Comments
 (0)