From 36a404e067dd5404cf4456e814a2d6f35004bea4 Mon Sep 17 00:00:00 2001
From: Jürgen Knödlseder <jurgen.knodlseder@irap.omp.eu>
Date: Sat, 22 Sep 2018 21:59:36 +0200
Subject: [PATCH] Do no longer set missing boundaries in ctool::get_observations and ctool::setup_observations (#2686)

---
 ChangeLog                                |  4 +++-
 NEWS                                     | 10 +++++++++-
 doc/source/admin/release_history/1.6.rst |  3 +++
 src/support/ctool.cpp                    | 11 -----------
 4 files changed, 15 insertions(+), 13 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index f31d044..0f49e1a 100755
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,8 +1,10 @@
-2018-09-21
+2018-09-22
 
         * Version 1.6.0 released
         ========================
 
+        Do no longer set missing boundaries in ctool::get_observations and
+          ctool::setup_observations (#2686)
         Unnormalize background cube values in ctbkgcube (#2685)
         Account for varying energy thresholds in ctbin weight computation (#2685)
         Add ctbkgcube::cntcube() method (#2685)
diff --git a/NEWS b/NEWS
index e9d043e..f25c008 100755
--- a/NEWS
+++ b/NEWS
@@ -1,6 +1,6 @@
 New Features and Important Changes in ctools 1.6.0
 
-21 September 2018
+22 September 2018
 
 
 Introduction
@@ -140,6 +140,14 @@ None
 
 ctselect - CTA event selection
 ------------------------------
+Do no longer set missing energy and RoI boundaries in ctool::get_observations
+and ctool::setup_observations since this led to a problem in ctselect. For
+H.E.S.S. data, where the boundaries are not set, the boundaries were preset
+with the User parameters, which made event selection with the usepnt=no option
+impossible. It is not clear why the setting of missing boundaries was introduced
+but it looks like a dangerous thing to do. The setting has therefore been
+removed (#2686).
+
 Avoid an error in case that the RoI radius is identical to the existing RoI
 radius (#2682).
 
diff --git a/doc/source/admin/release_history/1.6.rst b/doc/source/admin/release_history/1.6.rst
index 77e08f3..972b70c 100644
--- a/doc/source/admin/release_history/1.6.rst
+++ b/doc/source/admin/release_history/1.6.rst
@@ -19,6 +19,9 @@ In particular, this release provides:
 Bug fixes
 ---------
 
+* [`2686 <https://cta-redmine.irap.omp.eu/issues/2686>`_] -
+  Do no longer set missing boundaries in ``ctool::get_observations`` and
+  ``ctool::setup_observations``
 * [`2682 <https://cta-redmine.irap.omp.eu/issues/2682>`_] -
   Avoid :ref:`ctselect` error in case that RoI radius is unchanged
 * [`2652 <https://cta-redmine.irap.omp.eu/issues/2652>`_] -
diff --git a/src/support/ctool.cpp b/src/support/ctool.cpp
index 7fae611..f7ff519 100755
--- a/src/support/ctool.cpp
+++ b/src/support/ctool.cpp
@@ -388,11 +388,6 @@ void ctool::sync_pfiles(void)
  * observations in the container. In case that no response information is
  * required, the @p response argument should be set to @p false. See the
  * set_response() method for details.
- *
- *
- * The method will also setup the observation boundaries for all CTA
- * observations in the container that contain event lists. For details, see
- * the set_obs_bounds() method.
  ***************************************************************************/
 void ctool::setup_observations(GObservations& obs,
                                const bool&    response,
@@ -480,9 +475,6 @@ void ctool::setup_observations(GObservations& obs,
         set_response(obs);
     }
 
-    // Setup boundaries for all CTA observations containing event lists
-    set_obs_bounds(obs);
-
     // Return
     return;
 }
@@ -1736,9 +1728,6 @@ GObservations ctool::get_observations(const bool& get_response)
 
             } // endif: response was required
 
-            // Set observation boundary parameters (emin, emax, rad)
-            set_obs_bounds(obs);
-
             // Signal that XML file should be used for storage
             m_use_xml = true;
 
--
libgit2 0.21.4