Merge pull request #228 from ossimlabs/op-7

Removed the cuttern and placed it at the end. The cutter was not bei…
ossimlabs · Jun 5, 2019 · ea69ce5 · ea69ce5
2 parents 880b493 + 2b2933b
commit ea69ce5
Showing 1 changed file with 27 additions and 18 deletions.
diff --git a/src/util/ossimChipperUtil.cpp b/src/util/ossimChipperUtil.cpp
@@ -1164,6 +1164,9 @@ ossimRefPtr<ossimImageSource> ossimChipperUtil::initializeChain(ossimIrect &aoi)
          source->initialize();
       }
 
+      // GP 2019: we will add cutters at the end
+      // Will remove temporarily bt leave the code hear just in
+      // case I messed something up
       if (source.valid() && !aoi.hasNans())
       {
          //---
@@ -1172,29 +1175,29 @@ ossimRefPtr<ossimImageSource> ossimChipperUtil::initializeChain(ossimIrect &aoi)
          // 2) Speed up by not propagating get tile request outside the cut or "aoi"
          //    to the left hand side(input).
          //---
-         ossimRefPtr<ossimRectangleCutFilter> cutter = new ossimRectangleCutFilter();
+         // ossimRefPtr<ossimRectangleCutFilter> cutter = new ossimRectangleCutFilter();
 
-         // Set the cut rectangle:
-         cutter->setRectangle(aoi);
+         // // Set the cut rectangle:
+         // cutter->setRectangle(aoi);
 
-         // Null outside.
-         cutter->setCutType(ossimRectangleCutFilter::OSSIM_RECTANGLE_NULL_OUTSIDE);
+         // // Null outside.
+         // cutter->setCutType(ossimRectangleCutFilter::OSSIM_RECTANGLE_NULL_OUTSIDE);
 
-         // Connect cutter input to source chain.
-         cutter->connectMyInputTo(0, source.get());
+         // // Connect cutter input to source chain.
+         // cutter->connectMyInputTo(0, source.get());
 
-         source = cutter.get();
+         // source = cutter.get();
 
-         // Dependent on correct aoi so place after the cutter.
-         if (hasAnnotations())
-         {
-            // Put annotations after scalar remapper.
-            ossimRefPtr<ossimImageSource> result = addAnnotations(source);
-            if (result.valid())
-            {
-               source = result.get();
-            }
-         }
+         // // Dependent on correct aoi so place after the cutter.
+         // if (hasAnnotations())
+         // {
+         //    // Put annotations after scalar remapper.
+         //    ossimRefPtr<ossimImageSource> result = addAnnotations(source);
+         //    if (result.valid())
+         //    {
+         //       source = result.get();
+         //    }
+         // }
       }
    }
 
@@ -1652,6 +1655,7 @@ ossimRefPtr<ossimImageData> ossimChipperUtil::getChip(const ossimKeywordlist &op
    {
       m_kwl->addList(optionsKwl, true);
    }
+
 
    // (GP)
    // Until we add more ellaborate code to check for scale changes
@@ -1668,6 +1672,7 @@ ossimRefPtr<ossimImageData> ossimChipperUtil::getChip(const ossimKeywordlist &op
 
    if (optionsKwl.getSize() > 0)
    {
+
       //---
       // Only do this if new options were passed in. This was causing an off by
       // one error when now options were passed in using thumbnail option.
@@ -1679,6 +1684,10 @@ ossimRefPtr<ossimImageData> ossimChipperUtil::getChip(const ossimKeywordlist &op
 
       m_geom->setImageSize(aoi.size());
    }
+   else
+   {
+      getAreaOfInterest(m_source.get(), aoi);
+   }
 
    if (m_source.valid())
    {