uparekh01151 commited on
Commit
1853581
·
1 Parent(s): e6b0a87

Fix DatasetManager to handle different use case requirements

Browse files
Files changed (1) hide show
  1. src/evaluator.py +18 -3
src/evaluator.py CHANGED
@@ -60,13 +60,28 @@ class DatasetManager:
60
  loader_path = os.path.join(dataset_path, "loader.py")
61
  cases_path = os.path.join(dataset_path, "cases.yaml")
62
 
63
- if all(os.path.exists(p) for p in [schema_path, loader_path, cases_path]):
 
 
 
 
 
 
 
 
 
 
 
 
 
 
 
64
  # Use the full path as the dataset name
65
  full_name = f"{use_case}/{dataset_name}"
66
  datasets[full_name] = DatasetConfig(
67
  name=full_name,
68
- schema_path=schema_path,
69
- loader_path=loader_path,
70
  cases_path=cases_path
71
  )
72
 
 
60
  loader_path = os.path.join(dataset_path, "loader.py")
61
  cases_path = os.path.join(dataset_path, "cases.yaml")
62
 
63
+ # Check requirements based on use case
64
+ required_files = []
65
+ if use_case == "sql_generation":
66
+ # SQL generation needs all three files
67
+ required_files = [schema_path, loader_path, cases_path]
68
+ elif use_case == "code_generation":
69
+ # Code generation needs loader and cases
70
+ required_files = [loader_path, cases_path]
71
+ elif use_case == "documentation":
72
+ # Documentation only needs cases
73
+ required_files = [cases_path]
74
+ else:
75
+ # Default: require all files
76
+ required_files = [schema_path, loader_path, cases_path]
77
+
78
+ if all(os.path.exists(p) for p in required_files):
79
  # Use the full path as the dataset name
80
  full_name = f"{use_case}/{dataset_name}"
81
  datasets[full_name] = DatasetConfig(
82
  name=full_name,
83
+ schema_path=schema_path if os.path.exists(schema_path) else None,
84
+ loader_path=loader_path if os.path.exists(loader_path) else None,
85
  cases_path=cases_path
86
  )
87