CaPTk icon indicating copy to clipboard operation
CaPTk copied to clipboard

Perfusion PCA model requirement

Open ashishsingh18 opened this issue 4 years ago • 4 comments

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

ashishsingh18 avatar Oct 05 '20 18:10 ashishsingh18

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.

sarthakpati avatar Oct 05 '20 18:10 sarthakpati

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.

ashishsingh18 avatar Oct 05 '20 18:10 ashishsingh18

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

sarthakpati avatar Oct 05 '20 19:10 sarthakpati

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..

ashishsingh18 avatar Oct 05 '20 19:10 ashishsingh18