Simon Rit

Results 49 comments of Simon Rit

Thanks. `itkAnalyticSignalImageFilter.h`seems to be doing the same thing as the current `rtkHilbertImageFilter.h` (which was badly named because we followed Matlab). Is this going to be moved to an ITK module...

Sounds good. For CUDA, it might be easy but we should fix #296. For FFT extensions, RTK would certainly benefit from the analytic signal. We also have filters which work...

Do you think that the [failing builds](https://my.cdash.org/index.php?project=RTK&date=2020-07-06) are related? I'm lost!

Thanks @AlexyPellegrini! I think that the CI must be updated to account for the ITKCudaCommon dependency. See this comment to know how to do it: https://github.com/InsightSoftwareConsortium/ITKModuleTemplate/pull/123#issuecomment-1022172448

FYI, my plan is to finalize SimonRit/ITKCudaCommon#15. I'll update then #495. I will ask you then to rebase your PR on it... It's taking longer than expected because I have...

> @SimonRit The windows cuda python packages seems to work well. However there is an issue with docker for the linux cuda python packages (see [here](https://github.com/SimonRit/RTK/runs/7087060844?check_suite_focus=true#step:4:1183)). Do you have any...

The compilation is still struggling due to missing `CudaCommonExport.h`, see [here](https://github.com/SimonRit/RTK/runs/7190266069?check_suite_focus=true#step:4:5112). This file should be generated when compiling CudaCommon, any clue why it can't find it? I'm not sure I...

I can't find the file... I can find `./_work/RTK/RTK/_skbuild/linux-x86_64-3.7/cmake-build/include/RTKExport.h` but `_work/RTK/RTK/ITKCudaCommon/_skbuild` is empty. I'm lost...

As far as I can see, the remote modules BSplineGradient, HigherOrderAccurateGradient, SplitComponents, Strain, from which Ultrasound depends, do not need this generated file (grep Export in their source repo was...