Skip to content

Commit

Permalink
Per #1706, update aggr_ecnt_lines() to also add an entry to the skip_…
Browse files Browse the repository at this point in the history
…ba array of booleans. This is used to keep track of whether or not the point should be excluded from the statistics and is used in the Ensemble-Stat. This same aggregation code in the vx_statistics library is used by Stat-Analysis. Since Stat-Analysis failed to populate that array, it led to an array parsing error in ECNTInfo::set() when trying to compute the sum of the weights. (#1707)
  • Loading branch information
JohnHalleyGotway authored Mar 10, 2021
1 parent 3317778 commit bd8a63c
Showing 1 changed file with 1 addition and 0 deletions.
1 change: 1 addition & 0 deletions met/src/tools/core/stat_analysis/aggr_stat_line.cc
Original file line number Diff line number Diff line change
Expand Up @@ -2577,6 +2577,7 @@ void aggr_ecnt_lines(LineDataFile &f, STATAnalysisJob &job,
m[key].ens_pd.var_oerr_na.add(square(cur.spread_oerr));
m[key].ens_pd.var_plus_oerr_na.add(square(cur.spread_plus_oerr));
m[key].ens_pd.wgt_na.add(cur.total);
m[key].ens_pd.skip_ba.add(false);

//
// Store the summary statistics
Expand Down

0 comments on commit bd8a63c

Please sign in to comment.