Closed jds485 closed 1 year ago
@jds485 Code looks fine to me. I didn't look at the targets since they may still be running but I think the code changes are ok to merge.
The moving window targets are still not finished building. I think they're close to finishing so I'll restart building them again today. But after they're done I'm going to comment the targets out again.
Draft - target building in progress
This PR adds targets that create clusters based on annual flow metric values.
5_EDA.R
shows as completely different. I think this is a result of line endings being different from this commit to the previous. I added 1 new target to5_EDA.R
to plot flow metrics for each of the clusters that were discovered using raw flow metric values.Also updated:
dplyr::do
todplyr::summarize
, which contributed to some of the speed up. Most of the speed up resulted from using the output of another function as input to this function when seasonal metrics are requested.sbatch
command line flagsCloses #139