Repository navigation
Confusing! There are many variables with similar names, inconsistently named. #44
Description
Pay attention:
dataTrueValues_dictandmonthsForPredicted_dictdon't have a consistent naming, despite having the same role;dataForPrediction_dictandmonthsForPrediction_dicthave a consistent naming, and have the same role.
Then, predictValues_dict is created, inside NeuralNetwork.use_neural_network(). These are tha values that indeed have been predicted.
All these names are really confusing!
Activity
A-Infor commentedon Jul 11, 2025
I think that the variables could be renamed like this:
dataTrueValues_dict→spei_provided_inputsmonthsForPredicted_dict→months_for_provided_inputsdataForPrediction_dict→spei_expected_outputsmonthsForPrediction_dict→months_for_expected_outputspredictValues_dict→spei_predicted_valuesCloses #60, by fixing the fix #44: inputs and outputs got swapped whe…