Skip to content

Commit

Permalink
Per #1843, correct definition of SI from RMSE/ME to RMSE/OBAR.
Browse files Browse the repository at this point in the history
  • Loading branch information
JohnHalleyGotway committed Jul 29, 2021
1 parent 1d849fb commit 76fa862
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion met/docs/Users_Guide/appendixC.rst
Original file line number Diff line number Diff line change
Expand Up @@ -501,7 +501,7 @@ Scatter Index (SI)

Called "SI" in CNT output :numref:`table_PS_format_info_CNT`

SI is the ratio of the root mean squared error to the mean error (overall bias), :math:`\text{SI} = \text{RMSE} / \text{ME}`.
SI is the ratio of the root mean squared error to the average observation value, :math:`\text{SI} = \text{RMSE} / \text{OBAR}`.

Smaller values of SI indicate better agreement between the model and observations (less scatter on scatter plot).

Expand Down
4 changes: 2 additions & 2 deletions met/src/libcode/vx_statistics/compute_stats.cc
Original file line number Diff line number Diff line change
Expand Up @@ -122,7 +122,7 @@ void compute_cntinfo(const SL1L2Info &s, bool aflag, CNTInfo &cnt_info) {

// Compute Scatter Index (SI)
if(!is_eq(cnt_info.me.v, 0.0)) {
cnt_info.si.v = cnt_info.rmse.v / cnt_info.me.v;
cnt_info.si.v = cnt_info.rmse.v / cnt_info.obar.v;
}
else {
cnt_info.si.v = bad_data_double;
Expand Down Expand Up @@ -354,7 +354,7 @@ void compute_cntinfo(const PairDataPoint &pd, const NumArray &i_na,
// Compute Scatter Index (SI)
//
if(!is_eq(cnt_info.me.v, 0.0)) {
cnt_info.si.v = cnt_info.rmse.v / cnt_info.me.v;
cnt_info.si.v = cnt_info.rmse.v / cnt_info.obar.v;
}
else {
cnt_info.si.v = bad_data_double;
Expand Down

0 comments on commit 76fa862

Please sign in to comment.