[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: Sight needs to be ported to insighttoolkit5 [Fwd: sight is marked for autoremoval from testing]



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.bridault@ircad.fr
Tél. : +33 (0)3 88 119 201
  IRCAD
                  France
http://www.ircad.fr/
http://www.ircad.africa/


Suivez l'IRCAD sur  Facebook
 

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.bridault@ircad.fr <mailto:flavien.bridault-louchez@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<noreply@release.debian.org>  -----

Date: Wed, 12 Jan 2022 04:39:32 +0000
From: Debian testing autoremoval watch<noreply@release.debian.org>
To:sight@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-packaging@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 to: