Skip to content

Commit 8291be2

Browse files
xgchenaathewsey
authored andcommitted
fix: support specifying a facet by its column index
Currently the Clarify BiasConfig only accepts facet name. Actually Clarify analysis configuration supports both name and index. This commit adds the same support to BiasConfig.
1 parent f9b88ba commit 8291be2

File tree

1 file changed

+0
-13
lines changed

1 file changed

+0
-13
lines changed

tests/unit/test_clarify.py

Lines changed: 0 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -82,19 +82,6 @@ def test_invalid_data_config():
8282
)
8383

8484

85-
def test_s3_data_distribution_type_ignorance():
86-
data_config = DataConfig(
87-
s3_data_input_path="s3://input/train.csv",
88-
s3_output_path="s3://output/analysis_test_result",
89-
label="Label",
90-
headers=["Label", "F1", "F2", "F3", "F4"],
91-
dataset_type="text/csv",
92-
joinsource="F4",
93-
s3_data_distribution_type="ShardedByS3Key",
94-
)
95-
assert data_config.s3_data_distribution_type == "FullyReplicated"
96-
97-
9885
def test_bias_config():
9986
label_values = [1]
10087
facet_name = "F1"

0 commit comments

Comments
 (0)