Skip to content

Commit 6cfa4df

Browse files
authored
Merge pull request #119 from datakind/develop
fix: increased api limits
2 parents da5b64e + 7f1aac2 commit 6cfa4df

3 files changed

Lines changed: 6 additions & 13 deletions

File tree

src/webapp/databricks.py

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -206,7 +206,6 @@ def fetch_table_data(
206206
inst_name: str,
207207
table_name: str,
208208
warehouse_id: str,
209-
limit: int = 1000,
210209
) -> List[Dict[str, Any]]:
211210
"""
212211
Executes a SELECT * query on the specified table within the given catalog and schema,
@@ -226,7 +225,7 @@ def fetch_table_data(
226225
fully_qualified_table = (
227226
f"`{catalog_name}`.`{schema_name}_silver`.`{table_name}`"
228227
)
229-
sql_query = f"SELECT * FROM {fully_qualified_table} LIMIT {limit}"
228+
sql_query = f"SELECT * FROM {fully_qualified_table}"
230229

231230
try:
232231
# Execute the SQL statement

src/webapp/routers/data.py

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -1057,7 +1057,6 @@ def get_top_features(
10571057
inst_name=f"{query_result[0][0].name}",
10581058
table_name=f"sample_inference_{run_id}_features_with_most_impact",
10591059
warehouse_id=env_vars["SQL_WAREHOUSE_ID"],
1060-
limit=500,
10611060
)
10621061

10631062
return rows
@@ -1101,7 +1100,6 @@ def get_support_overview(
11011100
inst_name=f"{query_result[0][0].name}",
11021101
table_name=f"sample_inference_{run_id}_support_overview",
11031102
warehouse_id=env_vars["SQL_WAREHOUSE_ID"],
1104-
limit=500,
11051103
)
11061104

11071105
return rows
@@ -1144,7 +1142,6 @@ def get_training_support_overview(
11441142
inst_name=f"{query_result[0][0].name}",
11451143
table_name=f"sample_training_{run_id}_support_overview",
11461144
warehouse_id=env_vars["SQL_WAREHOUSE_ID"],
1147-
limit=500,
11481145
)
11491146

11501147
return rows
@@ -1187,7 +1184,6 @@ def get_feature_value(
11871184
inst_name=f"{query_result[0][0].name}",
11881185
table_name=f"sample_inference_{run_id}_shap_feature_importance",
11891186
warehouse_id=env_vars["SQL_WAREHOUSE_ID"],
1190-
limit=500,
11911187
)
11921188

11931189
return rows
@@ -1230,7 +1226,6 @@ def get_confusion_matrix(
12301226
inst_name=f"{query_result[0][0].name}",
12311227
table_name=f"sample_training_{run_id}_confusion_matrix",
12321228
warehouse_id=env_vars["SQL_WAREHOUSE_ID"],
1233-
limit=500,
12341229
)
12351230

12361231
return rows
@@ -1273,7 +1268,6 @@ def get_roc_curve(
12731268
inst_name=f"{query_result[0][0].name}",
12741269
table_name=f"sample_training_{run_id}_roc_curve",
12751270
warehouse_id=env_vars["SQL_WAREHOUSE_ID"],
1276-
limit=500,
12771271
)
12781272

12791273
return rows

src/webapp/validation_schemas/pdp_schema_extension.json

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -575,7 +575,7 @@
575575
"checks": []
576576
},
577577
"core_competency_completed": {
578-
"dtype": "float64",
578+
"dtype": "category",
579579
"coerce": true,
580580
"nullable": true,
581581
"required": true,
@@ -599,23 +599,23 @@
599599
"checks": []
600600
},
601601
"course_instructor_employment_status": {
602-
"dtype": "float64",
602+
"dtype": "category",
603603
"coerce": true,
604604
"nullable": true,
605605
"required": true,
606606
"aliases": [],
607607
"checks": []
608608
},
609609
"course_instructor_rank": {
610-
"dtype": "float64",
610+
"dtype": "category",
611611
"coerce": true,
612612
"nullable": true,
613613
"required": false,
614614
"aliases": [],
615615
"checks": []
616616
},
617617
"credential_engine_identifier": {
618-
"dtype": "float64",
618+
"dtype": "string",
619619
"coerce": true,
620620
"nullable": true,
621621
"required": true,
@@ -671,7 +671,7 @@
671671
"checks": []
672672
},
673673
"institution_id": {
674-
"dtype": "float64",
674+
"dtype": "string",
675675
"coerce": true,
676676
"nullable": true,
677677
"required": true,

0 commit comments

Comments
 (0)