Issue warning if surface not planar in draw surfaces (#1274)

### Briefly, what does this PR introduce?
Issue warning if surface not planar in draw surfaces

### What kind of change does this PR introduce?
- [x] Bug fix (issue #1273)
- [ ] New feature (issue #__)
- [ ] Documentation update
- [ ] Other: __

### Please check if this PR fulfills the following:
- [ ] Tests for the changes have been added
- [ ] Documentation has been added / updated
- [ ] Changes have been communicated to collaborators

### Does this PR introduce breaking changes? What changes might users
need to make to their code?

### Does this PR change default behavior?

---------

Co-authored-by: Wouter Deconinck <[email protected]>

src/algorithms/tracking/ActsGeometryProvider.cc

@@ -49,7 +49,7 @@ struct fmt::formatter<
 > : fmt::ostream_formatter {};
 #endif // FMT_VERSION >= 90000
 
-void draw_surfaces(std::shared_ptr<const Acts::TrackingGeometry> trk_geo, const Acts::GeometryContext geo_ctx,
+void draw_surfaces(std::shared_ptr<const Acts::TrackingGeometry> trk_geo, const Acts::GeometryContext geo_ctx, std::shared_ptr<spdlog::logger> init_log,
                    const std::string &fname) {
     using namespace Acts;
     std::vector<const Surface *> surfaces;
@@ -68,6 +68,10 @@ void draw_surfaces(std::shared_ptr<const Acts::TrackingGeometry> trk_geo, const
     size_t nVtx = 0;
     for (const auto &srfx: surfaces) {
         const auto *srf = dynamic_cast<const PlaneSurface *>(srfx);
+        if (srf==nullptr){
+          init_log->error("Warning: Attempting cast a {} to Acts::PlaneSurface returns nullptr. This surface will not be added to the .obj output.", srfx->name());
+          continue;
+        }
         const auto *bounds = dynamic_cast<const PlanarBounds *>(&srf->bounds());
         for (const auto &vtxloc: bounds->vertices()) {
             Vector3 vtx = srf->transform(geo_ctx) * Vector3(vtxloc.x(), vtxloc.y(), 0);
@@ -187,7 +191,7 @@ void ActsGeometryProvider::initialize(const dd4hep::Detector* dd4hep_geo,
     // Visit surfaces
     m_init_log->info("Checking surfaces...");
     if (m_trackingGeo) {
-        draw_surfaces(m_trackingGeo, m_trackingGeoCtx, "tracking_geometry.obj");
+        draw_surfaces(m_trackingGeo, m_trackingGeoCtx, m_init_log, "tracking_geometry.obj");
 
         m_init_log->debug("visiting all the surfaces  ");
         m_trackingGeo->visitSurfaces([this](const Acts::Surface *surface) {

src/algorithms/tracking/ActsGeometryProvider.h

@@ -38,7 +38,7 @@ namespace dd4hep::rec {
  *  This is useful for debugging the ACTS geometry. The obj file can
  *  be loaded into various tools, such as FreeCAD, for inspection.
  */
-void draw_surfaces(std::shared_ptr<const Acts::TrackingGeometry> trk_geo, const std::string &fname);
+void draw_surfaces(std::shared_ptr<const Acts::TrackingGeometry> trk_geo, std::shared_ptr<spdlog::logger> init_log, const std::string &fname);
 
 class ActsGeometryProvider {
 public: