Skip to content

Commit d1a4afe

Browse files
committed
handle poc test failure
1 parent 0cff4d2 commit d1a4afe

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

sdv/multi_table/hma.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -258,7 +258,7 @@ def _print_estimate_warning(self):
258258
for table, est_cols in self._estimate_num_columns(self.metadata, distributions).items():
259259
entry = []
260260
entry.append(table)
261-
entry.append(metadata_columns[table])
261+
entry.append(sum(metadata_columns[table]))
262262
total_est_cols += est_cols
263263
entry.append(est_cols)
264264
print_table.append(entry)

tests/integration/utils/test_poc.py

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -70,12 +70,12 @@ def test_simplify_schema(capsys):
7070
# Assert
7171
expected_message_before = re.compile(
7272
r'PerformanceAlert: Using the HMASynthesizer on this metadata schema is not recommended\.'
73-
r' To model this data, HMA will generate a large number of columns\. \(173818 columns\)\s+'
73+
r' To model this data, HMA will generate a large number of columns\. \(135934 columns\)\s+'
7474
r'Table Name\s*#\s*Columns in Metadata\s*Est # Columns\s*'
7575
r'match_stats\s*24\s*24\s*'
76-
r'matches\s*39\s*412\s*'
77-
r'players\s*5\s*378\s*'
78-
r'teams\s*1\s*173004\s*'
76+
r'matches\s*39\s*364\s*'
77+
r'players\s*5\s*330\s*'
78+
r'teams\s*1\s*135216\s*'
7979
r'We recommend simplifying your metadata schema using '
8080
r"'sdv.utils.poc.simplify_schema'\.\s*"
8181
r'If this is not possible, please visit '

0 commit comments

Comments
 (0)