Skip to content

Commit

Permalink
Merge pull request #550 from hsanter/hsanter-patch-1
Browse files Browse the repository at this point in the history
fixed 1d location subsetting in obs_sequence_tool
  • Loading branch information
hkershaw-brown authored Oct 5, 2023
2 parents 713dd3f + a59dbff commit 689317f
Showing 1 changed file with 2 additions and 4 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -221,13 +221,13 @@ program obs_sequence_tool
! the values if so. Note that if any of the values are not missing_r8, all of
! them must have good (and consistent) values. i don't have code in here yet
! to idiotproof this, but i should add it.

restrict_by_location = .false.
if ((minval(min_box).ne.missing_r8) .or. (maxval(min_box).ne.missing_r8) .or. &
(minval(max_box).ne.missing_r8) .or. (maxval(max_box).ne.missing_r8)) then
restrict_by_location = .true.
min_loc = set_location(min_box)
max_loc = set_location(max_box)
else
restrict_by_location = .false.
endif

! 3d sphere box check - locations module dependent, but an important one.
Expand Down Expand Up @@ -302,8 +302,6 @@ program obs_sequence_tool
max_box(3) = 0.0_r8
max_box(4) = -2.0_r8 !! FIXME: VERTISUNDEF, but not all loc mods have it
max_loc = set_location(max_box)
else
restrict_by_location = .false.
endif

!%! ! SPECIAL: cut off all GPS obs below the given height
Expand Down

0 comments on commit 689317f

Please sign in to comment.