How to disable print messages from ITK functions

pradeep · January 7, 2020, 7:36pm

I am using GPUAnisotropicSmoothingFilter for running some benchmarks and the printout similar to the below appears quite a lot disrupting the outptu from the benchmark reporting tool I use. Is there a way to disable such print outs to console ?

Defines: #define DIM_2
#define BUFPIXELTYPE float
#define OUTPIXELTYPE float
#define PIXELDIM 1

Defines: #define DIM_2
#define PIXELTYPE float

Defines: #define DIM_2
#define BLOCK_SIZE 16
#define PIXELTYPE float
#define ARGTYPE float

Defines: #define DIM_2
#define BUFPIXELTYPE float
#define OUTPIXELTYPE float
#define PIXELDIM 1

Defines: #define DIM_2
#define PIXELTYPE float

Defines: #define DIM_2
#define BLOCK_SIZE 16
#define PIXELTYPE float
#define ARGTYPE float

Defines: #define DIM_2
#define BUFPIXELTYPE float
#define OUTPIXELTYPE float
#define PIXELDIM 1

Defines: #define DIM_2
#define PIXELTYPE float

Defines: #define DIM_2
#define BLOCK_SIZE 16
#define PIXELTYPE float
#define ARGTYPE float

dzenanz · January 7, 2020, 7:59pm

Those defines are mentioned e.g. here:

github.com

InsightSoftwareConsortium/ITK/blob/c95b99852a9109a8fc176a06f88b2aa6d442154b/Modules/Filtering/GPUAnisotropicSmoothing/src/GPUScalarAnisotropicDiffusionFunction.cl#L20-L26


#define INPIXELTYPE PIXELTYPE
#define BUFPIXELTYPE float
//
// Dim 1
//
#ifdef DIM_1
#define ImageDimension 1

but I don’t know what could print them to the console except OpenCL compiler. Which OpenCL implementation are you using? Did you set any debugging options to it? Are you sure this does not come from your script?

pradeep · January 7, 2020, 8:02pm

nvidia opencl impl. I use other opencl programs and another library, arrayfire, they don’t get printed over there unless programmer explicitly does it. This happens in ITK examples too.

pradeep · January 7, 2020, 8:23pm

Actually I found some std::couts in the code.

github.com

InsightSoftwareConsortium/ITK/blob/master/Modules/Filtering/GPUAnisotropicSmoothing/include/itkGPUGradientNDAnisotropicDiffusionFunction.hxx#L109


  {
    defines << "#define ARGTYPE unsigned short\n";
  }
  else
  {
    defines << "#define ARGTYPE " << pixeltypename << "\n";
  }
#else
  defines << "#define ARGTYPE " << pixeltypename << "\n";
#endif
  std::cout << "Defines: " << defines.str() << std::endl;


  const char * GPUSource = GPUGradientNDAnisotropicDiffusionFunction::GetOpenCLSource();


  // load and build program
  this->m_GPUKernelManager->LoadProgramFromString(GPUSource, defines.str().c_str());


  // create kernel
  this->m_ComputeUpdateGPUKernelHandle = this->m_GPUKernelManager->CreateKernel("ComputeUpdate");
}

It is probably coming from those. These are not guarded with any condition, so I don’t have an option to disable them.

dzenanz · January 7, 2020, 9:30pm

I guess you will need to modify the source code then. It would be good if you also submitted your changes as a pull request.

pradeep · January 7, 2020, 10:08pm

Sure, will send a PR soon on github.

pradeep · January 8, 2020, 2:17pm

@dzenanz I hit a roadblock with git hooks.

Utilities/KWStyle/ITKOverwrite.txt
./Utilities/Hooks/pre-commit-style.bash: line 127: /home/pradeep/gitroot/ITK/413/KWStyle-build/KWStyle: No such file or directory
pre-commit hook failure
-----------------------

KWStyle check failed.

Line numbers in the errors shown refer to the file:
Modules/Core/GPUCommon/include/itkGPUReduction.hxx.kws

I am not sure how to resolve this. Any help is appreciated.

The change I made to the relevant file is just deletion of std::cout line. You can look at the changes here

I temporarily made the commit (skipping verification) in my fork to hyperlink them here.

dzenanz · January 8, 2020, 2:23pm

It looks like KWStyle was not built properly, or you made multiple binary trees from one source tree so things got confused. I made a PR from your branch.

pradeep · January 8, 2020, 2:27pm

I was switching tags and using the same binary tree, perhaps the I messed up the KWStyle build somehow - I am not fluent with its usage.

Thank you for raising the PR.