CRBS / cdeep3m

Please go to https://github.com/CRBS/cdeep3m2 for most recent version
Other
58 stars 10 forks source link

Merge_LargeData.m looks for de_augmentation_info.mat in fm directory #26

Closed coleslaw481 closed 6 years ago

coleslaw481 commented 6 years ago

Would it be possible for Merge_LargeData.m to look for the de_augmentation_info.mat file in the parent directory? Or maybe be given the path to that file as a command line argument.

$ /home/ubuntu/training_data/predict/run_all_predict.sh /home/ubuntu/100_it_trained_model/ /home/ubuntu/training_data/5stackaug/
Running 1fm predict (1) packages to process
  Processing Pkg001_Z01 1 of 1 
Non zero exit code from caffe for predict /home/ubuntu/training_data/predict/1fm/Pkg001_Z01 model. Exiting.
Here is last 10 lines of /home/ubuntu/training_data/predict/1fm/Pkg001_Z01/out.log:

Starting to merge large image dataset
Processing:
/home/ubuntu/training_data/predict/1fm/de_augmentation_info.mat
error: load: unable to find file /home/ubuntu/training_data/predict/1fm/de_augmentation_info.mat
error: called from
    /home/ubuntu/deep3m/Merge_LargeData.m at line 41 column 1
Command exited with non-zero status 1
real 227.88
user 120.86
sys 78.49
haberlmatt commented 6 years ago

Thought i had fixed that using following two lines if fm_dir(end)==filesep; fm_dir=fm_dir(1:end-1); end [parent_dir,~,ext] = fileparts(fm_dir); de_aug_file = fullfile(parent_dir,'de_augmentation_info.mat');

But, in case you are feeding it with the Pkg directory instead of the fm directory you could add another line, say: if pkg_dir(end)==filesep; pkg_dir=pkg_dir(1:end-1); end fm_dir = fileparts(pkg_dir); parent_dir = fileparts(fm_dir); de_aug_file = fullfile(parent_dir,'de_augmentation_info.mat');

coleslaw481 commented 6 years ago

Merge_LargeData.m expects the #fm directory so modified call in predict to pass that path and it works as of release 0.10.0