search

shimming-toolbox / tissue-to-MRproperty

Assign magnetic resonance property values to segmented anatomy
GNU General Public License v3.0
3 stars 0 forks source link

Create CLI and add instructions how to use the code #4

Closed jcohenadad closed 6 days ago

jcohenadad commented 2 weeks ago

I suggest creating a command line script interface (CLI) that can be called from the Terminal, and instructions on how to use the script in the README.

Similar to what was done for https://github.com/shimming-toolbox/susceptibility-to-fieldmap-fft?tab=readme-ov-file#usage

sriosq commented 1 week ago

Currently working on adding this to the code and will also add instructions to the README file. Thanks

sriosq commented 1 week ago

Initial working CLI recently pushed. See https://github.com/shimming-toolbox/tissue-to-MRproperty/commit/1aad13159453e992d1a62a158811ef776d7582c2. Working on adding a description to the README file later tonight.

sriosq commented 1 week ago

Here is the link for the commit: https://github.com/shimming-toolbox/tissue-to-MRproperty/commit/6c2cfb8f041cea93c2fc2df0500d6d940f2bfa1b Thanks for the recommendation, it is a great idea to implement this. @jcohenadad

jcohenadad commented 1 week ago

Someone else should try your CLI and instructions, eg @CharlesPageot

CharlesPageot commented 1 week ago

I cloned the repo, followed the instructions and it works

sriosq commented 6 days ago

Recently updated to work with pip install here 96803c64a03616a6b8335203237f14615aeaea57 . Works correctly. Closing issue. Thanks for the help and testing!