Skip to content

Commit

Permalink
Updated imosImpossibleDepthQC to consider adding zNominalMargin (defa…
Browse files Browse the repository at this point in the history
…ult value is 15m) to the site_nominal_depth as a possible maximum depth rather than adding 20% of the site_nominal_depth. This is to be able to deal with shallow sites that can face large tides (ex. DARWIN).
  • Loading branch information
ggalibert committed Jul 12, 2017
1 parent be90f10 commit 924d048
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion AutomaticQC/imosImpossibleDepthQC.m
Original file line number Diff line number Diff line change
Expand Up @@ -311,7 +311,7 @@
% possibleMin shouldn't be above the surface (~global range value)
% possibleMax cannot be below the site depth
possibleMin = max(possibleMin, imosParameters('DEPTH', 'valid_min')); % value from global range
possibleMax = min(possibleMax, siteNominalDepth + 20*siteNominalDepth/100); % we allow +20% to the nominal site depth
possibleMax = min(possibleMax, siteNominalDepth + zNominalMargin); % we allow the same flexibility for nominal site depth as for instrument nominal_depth

if any(strcmpi(paramName, {'PRES', 'PRES_REL'}))
if ~isempty(sample_data.geospatial_lat_min) && ~isempty(sample_data.geospatial_lat_max)
Expand Down

0 comments on commit 924d048

Please sign in to comment.