CBICA / CaPTk

Cancer Imaging Phenomics Toolkit (CaPTk) is a software platform to perform image analysis and predictive modeling tasks. Documentation: https://cbica.github.io/CaPTk
https://www.cbica.upenn.edu/captk
Other
175 stars 63 forks source link

dependency upgrade progress #1390

Open mschachter opened 3 years ago

mschachter commented 3 years ago

Hi, just wanted to give an update on the process of upgrading dependencies. On this branch I have all the cmake files and many small issues in the source code corrected so that CaPTk starts to build for:

Things were going ok until I encountered build issues with the greedy package (in src/applications/Preprocessing). This package seems to be forked - why is that?

I started upgrading dependencies to greedy on a forked branch, but paused because some of the threading code needs to be refactored and want some input from the maintainer before diving into it.

Let me know what you think if you get the chance!

sarthakpati commented 3 years ago

This package seems to be forked - why is that?

Looking at the diff with the main repo, it seems to have an integrated CMake build system and an extra CLI wrapper that makes working with greedy a bit easier.

I started upgrading dependencies to greedy on a forked branch, but paused because some of the threading code needs to be refactored and want some input from the maintainer before diving into it.

That sounds great! I am hopeful that Paul would be able to resolve the questions you have.

sarthakpati commented 3 years ago

Paul has updated Greedy to support ITK-5 in a new branch: https://github.com/pyushkevich/greedy/tree/itk5

mschachter commented 3 years ago

Noted - I'll return to this project as soon as I can.