Hi Andreas,

Well, sorry I assumed this module was not built for a reason... It is actually an experimental module for algorithms, that may be promoted in a real category if they are judged good enough.

See https://itk.org/Doxygen/html/group__ITKReview.html.

So I have no idea if Debian wants to enable this module considering this. Otherwise, I can give a try to try to build it and propose a patch, but I admit I was a bit afraid by the importance of the package. :)

I would prefer to hear the advice of someone maintaining this package before proposing any modification, is there anyone on this list that could help?

Cheers,


*Dr. Flavien BRIDAULT*
Director of Software Development
IRCAD France & IRCAD Africa

flavien.brida...@ircad.fr <mailto:flavien.bridault-louc...@ircad.fr>
Tél. : +33 (0)3 88 119 201
                IRCAD France
http://www.ircad.fr/
http://www.ircad.africa/ <http://www.ircad.fr/>

Suivez l'IRCAD sur Facebook <http://www.facebook.com/pages/IRCAD/193785273990141>

*IRCAD France*
Hôpitaux Universitaires - 1, place de l'Hôpital - 67091 Strasbourg Cedex - FRANCE

Le 20/01/2022 à 21:00, Andreas Tille a écrit :
Hi Flavien,

Am Thu, Jan 20, 2022 at 04:33:28PM +0100 schrieb Flavien Bridault:
I have understood the problem. The missing image filter has moved in an
optional module, ITKReview.

Two possible solutions:

1. modify the ITK package in order to enable the build of the ITKReview
module
I'm not sure whether I understand this correctly.  We probably want a
feature complete package.  If not our users might be surprised about
missing features in other applications as well.
2. remove the filter from sight build - this is what we did on Windows and
why it builds with ITK5.... ^^
Finally its your decision.

This image filter is really non-essential, so I guess it is easier to take
solution 2, can you confirm please ?
Its perfectly fine whatever you decide.

Kind regards

       Andreas.


Hi Andreas,

I am a bit disappointed it doesn't build because we do use ITK5 on
Windows.

Anyway, I'll take care of this of course, thanks for trying. :)

Cheers,


*Dr. Flavien BRIDAULT*
Director of Software Development
IRCAD France & IRCAD Africa

flavien.brida...@ircad.fr  <mailto:flavien.bridault-louc...@ircad.fr>
Tél. : +33 (0)3 88 119 201
                IRCAD France
http://www.ircad.fr/
http://www.ircad.africa/  <http://www.ircad.fr/>

Suivez l'IRCAD sur Facebook
<http://www.facebook.com/pages/IRCAD/193785273990141>

*IRCAD France*
Hôpitaux Universitaires - 1, place de l'Hôpital - 67091 Strasbourg Cedex
- FRANCE

Le 12/01/2022 à 15:17, Andreas Tille a écrit :
Hi Flavien,

as you can see below ITK4 is buggy and we switched to ITK5 anyway so
ITK4 will be removed.  To verify whether sight might build with ITK5
smoothly I simply replaced libinsighttoolkit4-dev by
libinsighttoolkit5-dev in (Build-)Depends[1], but obviously:

...
[ 81%] Building CXX object 
libs/io/dicom/test/CMakeFiles/io_dicomTest.dir/tu/DicomAnonymizerTest.cpp.o
cd /build/sight-21.0.0/obj-x86_64-linux-gnu/libs/io/dicom/test && /usr/bin/c++ 
-DBOOST_ALL_DYN_LINK -DBOOST_ALL_NO_LIB -DBOOST_ATOMIC_DYN_LINK -DBOOST_CHRONO_DYN_LINK 
-DBOOST_DATE_TIME_DYN_LINK -DBOOST_FILESYSTEM_DYN_LINK -DBOOST_IOSTREAMS_DYN_LINK 
-DBOOST_LOG_DYN_LINK -DBOOST_LOG_SETUP_DYN_LINK -DBOOST_REGEX_DYN_LINK 
-DBOOST_SPIRIT_USE_PHOENIX_V3 -DBOOST_THREAD_DONT_PROVIDE_DEPRECATED_FEATURES_SINCE_V3_0_0 
-DBOOST_THREAD_DYN_LINK -DBOOST_THREAD_PROVIDES_FUTURE -DBOOST_THREAD_VERSION=2 
-DIO_DICOMTEST_EXPORTS 
-I/build/sight-21.0.0/obj-x86_64-linux-gnu/libs/core/pch/pchCore/include/pchCore 
-I/build/sight-21.0.0/obj-x86_64-linux-gnu/libs/io/dicom/include -I/build/sight-21.0.0/libs 
-I/build/sight-21.0.0/libs/core -I/build/sight-21.0.0/obj-x86_64-linux-gnu/libs/io/base/include 
-I/build/sight-21.0.0/obj-x86_64-linux-gnu/libs/core/core/include -I/usr/include/libxml2 
-I/build/sight-21.0.0/obj-x86_64-linux-gnu/libs/core/data/include 
-I/build/sight-21.0.0/obj-x86_64-linux-gnu/libs/core/service/include 
-I/build/sight-21.0.0/obj-x86_64-linux-gnu/libs/core/activity/include 
-I/build/sight-21.0.0/obj-x86_64-linux-gnu/libs/filter/dicom/include 
-I/build/sight-21.0.0/obj-x86_64-linux-gnu/libs/geometry/data/include 
-I/build/sight-21.0.0/obj-x86_64-linux-gnu/libs/io/zip/include 
-I/build/sight-21.0.0/obj-x86_64-linux-gnu/libs/core/utestData/include 
-I/build/sight-21.0.0/obj-x86_64-linux-gnu/libs/core/utest/include -isystem /usr/include/gdcm-3.0 
-isystem /usr/lib -isystem /usr/include/cppunit -g -O2 -ffile-prefix-map=/build/sight-21.0.0=. 
-fstack-protector-strong -Wformat -Werror=format-security -Wdate-time -D_FORTIFY_SOURCE=2 -O3 
-DNDEBUG -fPIE -fvisibility=hidden -fvisibility-inlines-hidden -march=sandybridge -mtune=generic 
-mfpmath=sse -fvisibility=hidden -fvisibility-inlines-hidden -Wall -Wextra -Wconversion 
-Wno-unused-parameter -Wno-ignored-qualifiers -fPIC -std=gnu++17 -include "pch.hpp"  
-Winvalid-pch -MD -MT libs/io/dicom/test/CMakeFiles/io_dicomTest.dir/tu/DicomAnonymizerTest.cpp.o 
-MF CMakeFiles/io_dicomTest.dir/tu/DicomAnonymizerTest.cpp.o.d -o 
CMakeFiles/io_dicomTest.dir/tu/DicomAnonymizerTest.cpp.o -c 
/build/sight-21.0.0/libs/io/dicom/test/tu/DicomAnonymizerTest.cpp
/build/sight-21.0.0/libs/filter/image/SpheroidExtraction.cpp:34:10: fatal 
error: itkLabelGeometryImageFilter.h: No such file or directory
     34 | #include <itkLabelGeometryImageFilter.h>
        |          ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
compilation terminated.


... the situation is more complex.  Would you mind switching the
upstream code to ITK5?

Kind regards

       Andreas.


[1]https://salsa.debian.org/med-team/sight/-/commit/4dfa098d04116ef8c0320e6786e50337d0059ce7

----- Weitergeleitete Nachricht von Debian testing autoremoval 
watch<nore...@release.debian.org>   -----

Date: Wed, 12 Jan 2022 04:39:32 +0000
From: Debian testing autoremoval watch<nore...@release.debian.org>
To:si...@packages.debian.org
Subject: sight is marked for autoremoval from testing

sight 21.0.0-3 is marked for autoremoval from testing on 2022-02-09

It (build-)depends on packages with these RC bugs:
984284: insighttoolkit4: otb: ftbfs with GCC-11
   https://bugs.debian.org/984284



This mail is generated by:
https://salsa.debian.org/release-team/release-tools/-/blob/master/mailer/mail_autoremovals.pl

Autoremoval data is generated by:
https://salsa.debian.org/qa/udd/-/blob/master/udd/testing_autoremovals_gatherer.pl

_______________________________________________
Debian-med-packaging mailing list
debian-med-packag...@alioth-lists.debian.net
https://alioth-lists.debian.net/cgi-bin/mailman/listinfo/debian-med-packaging


----- Ende weitergeleitete Nachricht -----




Attachment: OpenPGP_signature
Description: OpenPGP digital signature

Reply via email to