SteenMoeller / NORDIC_Raw

Matlab code for performing image reconstruction in MRI and performing the NORDIC denoising
Other
52 stars 24 forks source link

Unable to run DEMO: Undefined function or variable 'subfunction_loop_for_NVR_avg_update' #3

Closed CesarCaballeroGaudes closed 2 years ago

CesarCaballeroGaudes commented 3 years ago

Hello,

I'm trying to run the DEMO as indicated in the README.md (also at the end of script_for_creating_simulation_data.m). However, when running NORDIC, I get the following error. Do you know what is happening?

Thanks very much Cesar

NORDIC('demo_data_for_NORDIC.mat') Using the NVR estimation method

PP =

   name: 'demo_data_for_NORDIC.mat'
   date: '10-Sep-2021 15:54:45'
  bytes: 204721995
  isdir: 0
datenum: 7.3841e+05

kernel_size =

 8     8     8

CHECK if the right input to NORDIC is used change the scaling to the recon program

ans =

0.4117    0.4020

I am master, I have control 1 Undefined function or variable 'subfunction_loop_for_NVR_avg_update'.

Error in NORDIC>sub_NVR_for_one_slice (line 535) subfunction_loop_for_NVR_avg_update(KSP2a,ARG.kernel_size(3),ARG.kernel_size(2),ARG.kernel_size(1),lambda,1,ARG.soft_thrs,KSP2_weight_tmp,ARG,NOISE_tmp,Component_threshold_tmp,energy_removed_tmp);

Error in NORDIC>NVR_LLR_3D_MP_2 (line 253) [KSP_recon,~,KSP_weight,NOISE,Component_threshold,energy_removed]=sub_NVR_for_one_slice(KSP_recon,KSP2,ARG,n1,QQ,master_fast,KSP_weight,NOISE,Component_threshold,energy_removed) ; % save all files

Error in NORDIC (line 23) NVR_LLR_3D_MP_2(DIR,file_filter,file_idx,NORDIC_threshold,DUAL,kernel_size,patchavg,['kernel' num2str(ker)],restart_list, master_fast,soft_thrs, patch_avg_sub);

SteenMoeller commented 3 years ago

I will look at it right away, and sorry about that

Steen

On Fri, Sep 10, 2021 at 9:11 AM Cesar Caballero Gaudes < @.***> wrote:

Hello,

I'm trying to run the DEMO as indicated in the README.md (also at the end of script_for_creating_simulation_data.m). However, when running NORDIC, I get the following error. Do you know what is happening?

Thanks very much Cesar

NORDIC('demo_data_for_NORDIC.mat') Using the NVR estimation method

PP =

name: 'demo_data_for_NORDIC.mat' date: '10-Sep-2021 15:54:45' bytes: 204721995 isdir: 0 datenum: 7.3841e+05

kernel_size =

8 8 8

CHECK if the right input to NORDIC is used change the scaling to the recon program

ans =

0.4117 0.4020

I am master, I have control 1 Undefined function or variable 'subfunction_loop_for_NVR_avg_update'.

Error in NORDIC>sub_NVR_for_one_slice (line 535)

subfunction_loop_for_NVR_avg_update(KSP2a,ARG.kernel_size(3),ARG.kernel_size(2),ARG.kernel_size(1),lambda,1,ARG.soft_thrs,KSP2_weight_tmp,ARG,NOISE_tmp,Component_threshold_tmp,energy_removed_tmp);

Error in NORDIC>NVR_LLR_3D_MP_2 (line 253)

[KSP_recon,~,KSP_weight,NOISE,Component_threshold,energy_removed]=sub_NVR_for_one_slice(KSP_recon,KSP2,ARG,n1,QQ,master_fast,KSP_weight,NOISE,Component_threshold,energy_removed) ; % save all files

Error in NORDIC (line 23) NVR_LLR_3D_MP_2(DIR,file_filter,file_idx,NORDIC_threshold,DUAL,kernel_size,patchavg,['kernel' num2str(ker)],restart_list, master_fast,soft_thrs, patch_avg_sub);

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/SteenMoeller/NORDIC_Raw/issues/3, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASZJXPHWDUQB6FOCFHTOTRDUBIG2BANCNFSM5DZNPQVA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

SteenMoeller commented 3 years ago

I am very sorry, but the function was missing. I have added it into the current NORDIC.m file, so please download it again.

Not that it matters, but when I had tested it for deployment, my plain matlab version had an unintended link to a directory such that I missed the dependency.

Steen

On Fri, Sep 10, 2021 at 9:16 AM Steen Moeller @.***> wrote:

I will look at it right away, and sorry about that

Steen

On Fri, Sep 10, 2021 at 9:11 AM Cesar Caballero Gaudes < @.***> wrote:

Hello,

I'm trying to run the DEMO as indicated in the README.md (also at the end of script_for_creating_simulation_data.m). However, when running NORDIC, I get the following error. Do you know what is happening?

Thanks very much Cesar

NORDIC('demo_data_for_NORDIC.mat') Using the NVR estimation method

PP =

name: 'demo_data_for_NORDIC.mat' date: '10-Sep-2021 15:54:45' bytes: 204721995 isdir: 0 datenum: 7.3841e+05

kernel_size =

8 8 8

CHECK if the right input to NORDIC is used change the scaling to the recon program

ans =

0.4117 0.4020

I am master, I have control 1 Undefined function or variable 'subfunction_loop_for_NVR_avg_update'.

Error in NORDIC>sub_NVR_for_one_slice (line 535)

subfunction_loop_for_NVR_avg_update(KSP2a,ARG.kernel_size(3),ARG.kernel_size(2),ARG.kernel_size(1),lambda,1,ARG.soft_thrs,KSP2_weight_tmp,ARG,NOISE_tmp,Component_threshold_tmp,energy_removed_tmp);

Error in NORDIC>NVR_LLR_3D_MP_2 (line 253)

[KSP_recon,~,KSP_weight,NOISE,Component_threshold,energy_removed]=sub_NVR_for_one_slice(KSP_recon,KSP2,ARG,n1,QQ,master_fast,KSP_weight,NOISE,Component_threshold,energy_removed) ; % save all files

Error in NORDIC (line 23) NVR_LLR_3D_MP_2(DIR,file_filter,file_idx,NORDIC_threshold,DUAL,kernel_size,patchavg,['kernel' num2str(ker)],restart_list, master_fast,soft_thrs, patch_avg_sub);

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/SteenMoeller/NORDIC_Raw/issues/3, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASZJXPHWDUQB6FOCFHTOTRDUBIG2BANCNFSM5DZNPQVA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

CesarCaballeroGaudes commented 3 years ago

Thanks I will download it and let you know.