Remove problematic string_non_nominal soft constraint.

Author: haldenl

asp/assign_weights.lp

@@ -14,7 +14,6 @@ soft_weight(same_field_gte3,same_field_gte3_weight).
 soft_weight(count_twice,count_twice_weight).
 soft_weight(shape_cardinality,shape_cardinality_weight).
 soft_weight(number_nominal,number_nominal_weight).
-soft_weight(string_non_nominal,string_non_nominal_weight).
 soft_weight(bin_cardinality,bin_cardinality_weight).
 soft_weight(quant_bin,quant_bin_weight).
 soft_weight(agg_dim,agg_dim_weight).

asp/soft.lp

@@ -37,11 +37,6 @@ soft(shape_cardinality,E) :- channel(E,shape), discrete_cardinality(E,C), C > 5.
 % @constraint Numbers should not be nominal.
 soft(number_nominal,E) :- type(E,nominal), field(E,F), fieldtype(F,number).
 
-% @constraint Prefer nominal string fields
-soft(string_non_nominal,E) :- type(E,ordinal), field(E,F), fieldtype(F,string).
-soft(string_non_nominal,E) :- type(E,temporal), field(E,F), fieldtype(F,string).
-soft(string_non_nominal,E) :- type(E,quantitative), field(E,F), fieldtype(F,string).
-
 % @constraint Binned quantitative field should not have too low cardinality.
 soft(bin_cardinality,E) :- type(E,quantitative), bin(E,_), enc_cardinality(E,C), C < 15.
 

asp/weights.lp

@@ -14,7 +14,6 @@
 #const count_twice_weight = 50.
 #const shape_cardinality_weight = 5.
 #const number_nominal_weight = 10.
-#const string_non_nominal_weight = 2.
 #const bin_cardinality_weight = 5.
 #const quant_bin_weight = 1.
 #const agg_dim_weight = 2.

draco/cli.py

@@ -95,7 +95,7 @@ def main():  # pragma: no cover
     else:
         logger.info(f"Processing query: {args.query.name} ...")
         if args.type == QueryType.asp:
-            draco_query = args.query.read().split('\n')
+            draco_query = args.query.read().split("\n")
         else:
             query_spec = json.load(args.query)
             d = args.base or os.path.dirname(args.query.name)