From 9d840ca71487ed14473d1d5f9f18c417a83bb284 Mon Sep 17 00:00:00 2001
From: Craig Warren <craig_warren@me.com>
Date: Thu, 14 Jul 2022 15:41:04 +0100
Subject: [PATCH] Correction to snapshot bounds checks

---
 gprMax/cmds_multiuse.py | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/gprMax/cmds_multiuse.py b/gprMax/cmds_multiuse.py
index 0a45dfe0..cccdd8f5 100644
--- a/gprMax/cmds_multiuse.py
+++ b/gprMax/cmds_multiuse.py
@@ -977,13 +977,17 @@ class Snapshot(UserObjectMulti):
                              'parameters.')
             raise
 
-        p1, p2 = uip.check_box_points(p1, p2, self.params_str())
+        try:
+            p3 = uip.round_to_grid_static_point(p1)
+            p4 = uip.round_to_grid_static_point(p2)
+            p1, p2 = uip.check_box_points(p1, p2, self.params_str())
+        except ValueError:
+            logger.exception(self.params_str() + ' point is outside the domain.')
+            raise
         xs, ys, zs = p1
         xf, yf, zf = p2
-        dx, dy, dz = uip.discretise_point(dl)
 
-        p3 = uip.round_to_grid_static_point(p1)
-        p4 = uip.round_to_grid_static_point(p2)
+        dx, dy, dz = uip.discretise_static_point(dl)
 
         # If number of iterations given
         try: