ktindiana / sphinxval

SPHINX validation code for solar energetic particle models
MIT License
3 stars 3 forks source link

Crashes on AWT when run with multiple models #50

Closed ktindiana closed 4 months ago

ktindiana commented 6 months ago

Current version of code creates AWT dictionaries with different lengths for different models depending on what they predict. This results in a crash when trying to convert to a dataframe due to different length arrays in the dictionaries due to different number of SEP events contained within the forecasts and possibly due to different predicted quantities. Need to rethink how to handle the AWT value. Either break up the AWT calculations for different quantities (All clear, peak flux) into different dictionaries or create one big massive dictionary that has lots of None values as place holders so that all arrays have the same length.

ktindiana commented 4 months ago

This bug was fixed. The AWT output file now contains a lot of empty space, which might be improved in the future, but it does not crash.