Closed jsadler2 closed 2 years ago
The combined_metrics function currently takes pred_train, pred_val, and pred_test. I think it would make more sense if it took a dictionary with the partition id (e.g., train) as the keys and the prediction data file as the value.
combined_metrics
pred_train
pred_val
pred_test
train
That would greatly simplify combined_metrics because then the function would simply iterate through that dictionary.
The
combined_metrics
function currently takespred_train
,pred_val
, andpred_test
. I think it would make more sense if it took a dictionary with the partition id (e.g.,train
) as the keys and the prediction data file as the value.