bkainz / fetalReconstruction

GPU accelerated source code for motion compensation of multi-stack MRI data
57 stars 28 forks source link

AppImage required '--output' yet is given #17

Open kaltu opened 5 years ago

kaltu commented 5 years ago

I am trying your new released AppImage distribution. However whether I ran 'fetalMRIReconstruction-x86_64.AppImage -h'

image

or 'fetalMRIReconstruction-x86_64.AppImage --output 050_SVR_cor.nii -i FS/050_006_CORONAL_PD_FAT_SAT.nii FS/050_003_SAGITTAL_PD_FAT_SAT.nii FS/050_002_AXIAL_PD_FAT_SAT.nii' image

It always says -o is missing but it does given.

Any ideas?

bkainz commented 5 years ago

hm. That's the first time I have tried to generate such an AppImage. I'll investigate after the MICCAI deadline next week. (perhaps @schuhschuh can help?)

schuhschuh commented 5 years ago

@bkainz The AppRun script included does not pass the command arguments on to the actual executable.

AppRun:

#!/bin/sh

export LD_LIBRARY_PATH=.
cd "$(dirname "$0")"
exec ./reconstruction_GPU2

@kaltu You could try and extract the AppImage with

mkdir fetalMRIReconstruction-x86_64 && cd fetalMRIReconstruction-x86_64
mv path/to/fetalMRIReconstruction-x86_64.AppImage .
fetalMRIReconstruction-x86_64 --appimage-extract

and edit the fetalMRIReconstruction-x86_64/squashfs-root/AppRun script:

#!/bin/sh

export LD_LIBRARY_PATH=.
cd "$(dirname "$0")"
exec ./reconstruction_GPU2 "$@"
bkainz commented 5 years ago

@schuhschuh thank you very much! @kaltu perhaps this helps temporarily. After next week I'll update the AppImage accordingly!

bkainz commented 5 years ago

"After next week I'll" ... sorry, will look into this asap.