Removed instruction fallback to a string (#61)

* In evaluate, removed empty 'instructions' fallback and added code to ensure task_definition is validated

* Removed task_definition validation in the decorator. It should be handled in the sdk-backend.

Author: devvratbhardwaj

.gitignore

@@ -31,3 +31,9 @@ pip-log.txt
 .metaflow
 .metaflow/
 .ipynb_checkpoints
+
+## Local testing
+venv
+tests/test_local_eval.py
+tests/fixed_test_eval.csv
+tests/test_evaluation_dataset_may_2025.csv

aimon/decorators/evaluate.py

@@ -284,9 +284,7 @@ def evaluate(
             "evaluation_id": am_eval.id,
             "evaluation_run_id": eval_run.id,
         }
-        if "instructions" in payload and not payload["instructions"]:
-            payload["instructions"] = ""
-
+        
         results.append(EvaluateResponse(record['output'], client.analyze.create(body=[payload])))
 
     return results