insightsoftwareconsortium / itktotalvariation Goto Github PK

External Module for Total Variation Algorithms, providing wrap for https://github.com/albarji/proxTV

License: Apache License 2.0

CMake 33.87% C++ 56.63% Python 9.50%

itk insight-toolkit itk-module denoising total-variation

itktotalvariation's Issues

Add images to Readme

resized, due to github markup not having the option to resize images in .rst. See issues:original, closed and reopened

Fix CMake in proxTV to handle compilation when OpenMP is not found

Fixed in: phcerdan/proxTV@2640519

This should fix build/test compilation on MacOS

Eigen compilation problems with in-source builds

Hi @phcerdan

I am getting an Eigen problem while trying to compile ITK with the module Total Variation. Shouldnt this dependency be downloaded and compiled automatically as part of the superbuild?

Thank you!
Bea

Remote module does not configure

-- TotalVariation_proxTV_USE_EIGEN: ON
-- ITKTotalVariation: Using internal ITK Eigen Config found in: /home/johnsonhj/Dashboard/src/ITK-clang11/ITKInternalEigen3-build
-- Configuring proxTV
--   version: 3.2.1
--   description: Toolbox for fast Total Variation proximity operators
-- Eigen Found: /home/johnsonhj/Dashboard/src/ITK/Modules/ThirdParty/Eigen3/src/itkeigen
-- OpenMP found: TRUE
CMake Error at /home/johnsonhj/Dashboard/src/ITK-clang11/_deps/proxtv_fetch-build/proxTVConfig.cmake:11 (message):
  File or directory
  /home/johnsonhj/Dashboard/src/ITK-clang11/_deps/proxtv_fetch-build/proxTVTargets.cmake
  referenced by variable proxTV_TARGETS does not exist !
Call Stack (most recent call first):
  /home/johnsonhj/Dashboard/src/ITK-clang11/_deps/proxtv_fetch-build/proxTVConfig.cmake:17 (set_and_check)
  /home/johnsonhj/Dashboard/src/ITK-clang11/lib/cmake/ITK-5.1/Modules/TotalVariation.cmake:20 (find_package)
  CMake/ITKModuleAPI.cmake:76 (include)
  CMake/ITKModuleAPI.cmake:7 (itk_module_load)
  CMake/ITKModuleAPI.cmake:153 (_itk_module_use_recurse)
  CMake/ITKModuleMacros.cmake:328 (itk_module_use)
  Modules/Remote/TotalVariation/test/CMakeLists.txt:1 (itk_module_test)


-- Configuring incomplete, errors occurred!
See also "/home/johnsonhj/Dashboard/src/ITK-clang11/CMakeFiles/CMakeOutput.log".
See also "/home/johnsonhj/Dashboard/src/ITK-clang11/CMakeFiles/CMakeError.log".

Provide more documentation for parameters

For Weights, Norm, MaximumNumberOfIterations, document in the Doxygen comment what they mean, their default value, and recommendations for good values.

wrappings for python 3.9 not in pypi

https://pypi.org/project/itk-totalvariation/#files

Please tag the repo with the commit that corresponds to ITK 5.1.0

Publish new package compatible with ITK 5.3.0

This smoothing algorithm works quite nicely. Unfortunately, it hasn't been updated recently. The current version on Pypi is not compatible with ITK 5.3.0.

Image Axes Change

The total variation can flip the image axis depending on the input orientation:

I assume it's because the temporary double output image here:

ITKTotalVariation/include/itkProxTVImageFilter.hxx

Line 70 in ada39a5

auto resultImage = DoubleImageType::New();

is not copying all the meta information from the input image.

insightsoftwareconsortium / itktotalvariation Goto Github PK

itktotalvariation's Issues

Add images to Readme

Fix CMake in proxTV to handle compilation when OpenMP is not found

Eigen compilation problems with in-source builds

Remote module does not configure

Provide more documentation for parameters

wrappings for python 3.9 not in pypi

Please tag the repo with the commit that corresponds to ITK 5.1.0

Publish new package compatible with ITK 5.3.0

Image Axes Change

Recommend Projects

React

Vue.js

Typescript

TensorFlow

Django

Laravel

D3

Recommend Topics

javascript

web

server

Machine learning

Visualization

Game

Recommend Org

Facebook

Microsoft

Google

Alibaba

D3

Tencent