ITK Release 5/Wish List: Difference between revisions

From KitwarePublic
Jump to navigationJump to search
No edit summary
No edit summary
Line 21: Line 21:
* This would avoid having to compare `sizeof(mytype)` to know what type to use on a specific platform.
* This would avoid having to compare `sizeof(mytype)` to know what type to use on a specific platform.
* This issue was raised in [http://review.source.kitware.com/#/c/21513/ this patch suggestion]
* This issue was raised in [http://review.source.kitware.com/#/c/21513/ this patch suggestion]
== Requiring CMake >= 3.0 ==
* Currently supports CMake >=2.8.9 mostly to support Ubuntu 14.04 LTS (5 year support)


== C++11 throughout ==
== C++11 throughout ==

Revision as of 17:10, 10 February 2017

The wish list is provided by members of the ITK development community. These requests will not necessarily be included in ITKv5.

Outstanding wishes

A detailed wish list was put together by the community when preparing for ITK v4. Many of the wishes expressed there could not be fulfilled at the time but are still valid:

Image IO

  • Replacing current integer pixel IO types defined in itk::ImageIOBase by fixed width integer types
    • unsigned short -> uint16_t
    • short -> int16_t
    • unsigned int -> uint32_t
    • int -> int32_t
    • unsigned long -> uint64_t
    • long -> int64_t
    • Portability issue of "char", "signed char" and "unsigned char" need to be discussed and addressed before using uint8_t and int8_t
      • char and unsigned char would keep being defined as char and unsigned char as these types is wildly use image formats.
  • Currently, the size of each type may vary from one platform to another.
  • This is especially true for 'long' that is 32 bits on Windows/Visual Studio C++ 2013 (64 bits) but 64 bits on 64 bits Unix systems.
  • Files saved on one platform should be read the same on a different platform.
  • This would avoid having to compare `sizeof(mytype)` to know what type to use on a specific platform.
  • This issue was raised in this patch suggestion

Requiring CMake >= 3.0

  • Currently supports CMake >=2.8.9 mostly to support Ubuntu 14.04 LTS (5 year support)

C++11 throughout

As ITK uses templates heavily, it would greatly benefit from usage of few C++11 goodies, especially auto and decltype.

To take a step further, requiring the latest c++ standard (C++14? C++17?) available when starting a v5 endeavour could also be discussed.

Note: Some c++ >= 11 features may not be supported by CastXML (and by pygccxml): https://github.com/CastXML/CastXML/issues/72 Before implementing fancy stuff it should be double-checked if it does not break the Wrappings.

VNL->Eigen

Replace VNL by Eigen, which is much more modern.

Wrapping

Filter Refactoring

  • The current Functor based filters create code bloat because each class instance requires two instantiated classes. An alternative Functor approach should be used where the same class can be used any functor. One approach is to use a "SefFunctor" method which generates the GenerateData method.