Skip to content

Commit

Permalink
fixed the sst problem
Browse files Browse the repository at this point in the history
  • Loading branch information
AndrewEichmann-NOAA committed Jan 27, 2024
1 parent d8ca56d commit ca3196e
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 4 deletions.
2 changes: 1 addition & 1 deletion parm/soca/obs/obs_list.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
observers:
- !INC ${OBS_YAML_DIR}/adt_rads_all.yaml
#- !INC ${OBS_YAML_DIR}/sst_metopb_l3u.yaml
- !INC ${OBS_YAML_DIR}/sst_metopb_l3u.yaml
- !INC ${OBS_YAML_DIR}/icec_amsr2_north.yaml
- !INC ${OBS_YAML_DIR}/icec_amsr2_south.yaml
7 changes: 4 additions & 3 deletions utils/obsproc/Ghrsst2Ioda.h
Original file line number Diff line number Diff line change
Expand Up @@ -212,9 +212,10 @@ namespace gdasapp {

// Replace datime by its mean
// TODO(ASGM): Remove when the time reading is fixed
int64_t mean = iodaVars.datetime_.sum() / iodaVars.datetime_.size();
iodaVars.datetime_.setConstant(mean);

if (iodaVars.datetime_.size() > 0) {
int64_t mean = iodaVars.datetime_.sum() / iodaVars.datetime_.size();
iodaVars.datetime_.setConstant(mean);
}
return iodaVars;
};
}; // class Ghrsst2Ioda
Expand Down

3 comments on commit ca3196e

@apchoiCMD
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ha, nice finds! Any others are fine?

@AndrewEichmann-NOAA
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't look too closely, only Ghrsst2Ioda - if they have the same lines, then they should probably be changed too

@apchoiCMD
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't look too closely, only Ghrsst2Ioda - if they have the same lines, then they should probably be changed too

Thanks @AndrewEichmann-NOAA !

Please sign in to comment.