ktindiana / sphinxval

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

To optimize: Perform unmatching step only on models with CME or flare triggers #7

Closed ktindiana closed 9 months ago

ktindiana commented 1 year ago

The current version of SPHINX will go through all forecasts for each SEP event in the unmatching step. This shouldn't take nearly as much time as the matching, but it may be a time sink. The unmatching step only makes sense for models with flare or CME triggers. So during matching, should note which models use those triggers and then check unmatching only for those models.

ktindiana commented 9 months ago

This has been (already) implemented in SPHINX. When performing the matching step, a "uses_eruptions" boolean is set for each model and unmatching will proceed only if True.