From bdc9edf6b109d1da5452309747bdd7635e6c8132 Mon Sep 17 00:00:00 2001
From: Utkarsh <Utkarsh.Yenurkar@Sony.com>
Date: Fri, 27 Dec 2024 14:49:32 +0530
Subject: [PATCH] Update FrustumVisual

Signed-off-by: Utkarsh <Utkarsh.Yenurkar@Sony.com>
---
 include/gz/rendering/FrustumVisual.hh         | 34 ++-----------------
 .../gz/rendering/base/BaseFrustumVisual.hh    |  2 +-
 ogre2/src/Ogre2FrustumVisual.cc               |  5 +++
 src/FrustumVisual.cc                          |  2 +-
 4 files changed, 9 insertions(+), 34 deletions(-)

diff --git a/include/gz/rendering/FrustumVisual.hh b/include/gz/rendering/FrustumVisual.hh
index bd1771a30..0d49eec3f 100644
--- a/include/gz/rendering/FrustumVisual.hh
+++ b/include/gz/rendering/FrustumVisual.hh
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2015 Open Source Robotics Foundation
+ * Copyright (C) 2024 Open Source Robotics Foundation
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -72,24 +72,7 @@ namespace gz
       /// \brief Destructor
       public: virtual ~FrustumVisual();
 
-      /// \brief Constructor
-      /// \param[in] _near Near plane distance. This is the distance from
-      /// the frustum's vertex to the closest plane
-      /// \param[in] _far Far plane distance. This is the distance from the
-      /// frustum's vertex to the farthest plane.
-      /// \param[in] _fov Field of view. The field of view is the
-      /// angle between the frustum's vertex and the edges of the near or far
-      /// plane. This value represents the horizontal angle.
-      /// \param[in] _aspectRatio The aspect ratio, which is the width divided
-      /// by height of the near or far planes.
-      /// \param[in] _pose Pose of the frustum, which is the vertex (top of
-      /// the pyramid).
-      /*protected: FrustumVisual(double _near,
-                      double _far,
-                      const math::Angle &_fov,
-                      double _aspectRatio,
-                      const gz::math::Pose3d &_pose = gz::math::Pose3d::Zero);*/
-
+      /// \brief Update the Visual
       public: virtual void Update() = 0;
 
       /// \brief Get the near distance. This is the distance from the
@@ -147,16 +130,6 @@ namespace gz
       /// \return Plane of the frustum.
       public: virtual gz::math::Planed Plane(const FrustumVisualPlane _plane) const = 0;
 
-      /// \brief Check if a box lies inside the pyramid frustum.
-      /// \param[in] _b Box to check.
-      /// \return True if the box is inside the pyramid frustum.
-      //public: virtual bool Contains(const gz::math::AxisAlignedBox &_b) const = 0; //TO-DO
-
-      /// \brief Check if a point lies inside the pyramid frustum.
-      /// \param[in] _p Point to check.
-      /// \return True if the point is inside the pyramid frustum.
-      //public: virtual bool Contains(const gz::math::Vector3d &_p) const = 0; //TO-DO
-
       /// \brief Get the pose of the frustum
       /// \return Pose of the frustum
       /// \sa SetPose
@@ -170,9 +143,6 @@ namespace gz
       /// \brief Compute the planes of the frustum. This is called whenever
       /// a property of the frustum is changed.
       private: void ComputePlanes();
-
-      /// \brief Private data pointer
-      //GZ_UTILS_IMPL_PTR(dataPtr)
     };
     }
   }
diff --git a/include/gz/rendering/base/BaseFrustumVisual.hh b/include/gz/rendering/base/BaseFrustumVisual.hh
index 875156ec4..fbacb04e9 100644
--- a/include/gz/rendering/base/BaseFrustumVisual.hh
+++ b/include/gz/rendering/base/BaseFrustumVisual.hh
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2020 Open Source Robotics Foundation
+ * Copyright (C) 2024 Open Source Robotics Foundation
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
diff --git a/ogre2/src/Ogre2FrustumVisual.cc b/ogre2/src/Ogre2FrustumVisual.cc
index 7669bcd8f..cae4dae21 100644
--- a/ogre2/src/Ogre2FrustumVisual.cc
+++ b/ogre2/src/Ogre2FrustumVisual.cc
@@ -25,6 +25,11 @@
 #endif
 #endif
 
+#include <array>
+#include <cmath>
+#include <memory>
+#include <string>
+
 #include <gz/common/Console.hh>
 
 #include "gz/rendering/ogre2/Ogre2Conversions.hh"
diff --git a/src/FrustumVisual.cc b/src/FrustumVisual.cc
index a1021142f..825f2851f 100644
--- a/src/FrustumVisual.cc
+++ b/src/FrustumVisual.cc
@@ -1,5 +1,5 @@
 /*
- * Copyright (C) 2020 Open Source Robotics Foundation
+ * Copyright (C) 2024 Open Source Robotics Foundation
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.