You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We haven't taken a look at this code in a long time, but when doing multilake calibrations or different types of calibrations, the compare_to_field workflow is pretty slow. This is probably a low priority for the time being, but probably some easy wins using more modern packages to do this type of matching/interpolating within the output matrix.
Before starting on this it would be good to benchmark with an example.
The text was updated successfully, but these errors were encountered:
@jread-usgs rewrote resample_to_field using join function from dplyr. Improves speed about 25%. The rest of the time sink is in get_var
Also, I was trying to figure out how to update my branch with the new commits on the USGS ggoverhaul fork... and I'm not sure how, but managed to commit directly to it, instead of to mine 🤦♂. Since I've already messed up, going to leave it as is and not damage anything further.
We haven't taken a look at this code in a long time, but when doing multilake calibrations or different types of calibrations, the
compare_to_field
workflow is pretty slow. This is probably a low priority for the time being, but probably some easy wins using more modern packages to do this type of matching/interpolating within the output matrix.Before starting on this it would be good to benchmark with an example.
The text was updated successfully, but these errors were encountered: