Skip to content

Commit

Permalink
Fix order in which transformers are returned (#1239)
Browse files Browse the repository at this point in the history
  • Loading branch information
pvk-developer authored and amontanez24 committed Mar 23, 2023
1 parent d4bba60 commit ffca169
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 2 deletions.
5 changes: 5 additions & 0 deletions sdv/single_table/base.py
Original file line number Diff line number Diff line change
Expand Up @@ -352,6 +352,11 @@ def get_transformers(self):
"Use 'auto_assign_transformers' or 'fit' to create them."
)

field_transformers = {
column_name: field_transformers.get(column_name)
for column_name in self.metadata._columns
}

return field_transformers

def get_info(self):
Expand Down
8 changes: 6 additions & 2 deletions tests/unit/single_table/test_base.py
Original file line number Diff line number Diff line change
Expand Up @@ -146,14 +146,18 @@ def test_get_transformers(self):
'name': 'FrequencyEncoder',
'salary': 'FloatFormatter'
}
instance.metadata._columns = {
'salary': {'sdtype': 'numerical'},
'name': {'sdtype': 'categorical'}
}

# Run
result = BaseSingleTableSynthesizer.get_transformers(instance)

# Assert
assert result == {
'name': 'FrequencyEncoder',
'salary': 'FloatFormatter'
'salary': 'FloatFormatter',
'name': 'FrequencyEncoder'
}

def test_get_transformers_raises_an_error(self):
Expand Down

0 comments on commit ffca169

Please sign in to comment.