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

Perfusion PCA model requirement #1348

Open ashishsingh18 opened 3 years ago

ashishsingh18 commented 3 years ago

Perfusion PCA GUI asks for a model directory to do PCA estimation on a batch of subjects. But there is no 'pre-trained' model provided as for other applications.

To Reproduce

  1. Click on Applications->Perfusion PCA
  2. See the dialog that opens up.

Expected behavior Either a pre-trained model should be provided or if the model is not required, the GUI should be modified.

Screenshots image

CaPTk Version: 1.8 beta

OS: Windows 10 x64

Additional Info: The documentation doesn't mention anything about the model. https://cbica.github.io/CaPTk/PCA_Extraction.html

sarthakpati commented 3 years ago

This is not present on my installation and not even in the master [ref]. This application was disabled for 1.8.0 release and is stated for release in the patch release.

ashishsingh18 commented 3 years ago

This is not present on my installation and not even in the master [ref]. This application was disabled for 1.8.0 release and is stated for release in the patch release.

Of-course it has been disabled now, but this was there up until the moment we disabled the application. Even now if you enable the application and open it, this will show up.

sarthakpati commented 3 years ago

Oh, got it, so you are opening these for you to track when you work on this application.

ashishsingh18 commented 3 years ago

Oh, got it, so you are opening these for you to track when you work on this application.

Yeah...having these on Github helps better tracking. These should have been caught during our earlier testing phases though, but better late than never..