Skip to content

Commit

Permalink
Update README.md
Browse files Browse the repository at this point in the history
  • Loading branch information
badesar1 authored Oct 3, 2023
1 parent d11f910 commit 23ee61a
Showing 1 changed file with 4 additions and 49 deletions.
53 changes: 4 additions & 49 deletions README.md
Original file line number Diff line number Diff line change
@@ -1,40 +1,9 @@
# designer_v2_dev
Temporary repo to test updates to designer.
# designer-v2

Jenny, Santi, please feel free to mess with this code however you want (in fact I encourage you to do so). Make sure everything works as expected. Ask me questions if you have them.
## For complete documentation on designer & tmi installation and usage please visit the [documention](https://nyu-diffusionmri.github.io).

Jenny, you might note that I now pull some data directly from the bids .json sidecar (pe direction, pf level). I also included the fast mp here, mostly just so I didnt have to wait for the longer version to run. If you want to switch to the nonlocal mp, just replace `import mpcomplex as mp` with `import mpdenoise_sj as mp` in designer. Again, please feel free to play with this and make whatever improvements you see fit.

## Dev Installation:
After development, designer-v2 will be uploaded to pip. Until then, here are instructions for installing locally.

Note, designer still depends on FSL for topup and eddy, and mrtix3 for image-wise operations.

- I would recommend creating a python virtual environment for testing. This is helpful for ensuring that you are using a compatible version of python. You can create one with a command like the following:\
` conda create --prefix /my/path/designer-env python=3.9`\
` conda activate /my/path/designer-env`
Now any Python packages you install will be installed into this virtual environemnt.

- The current master version of mrtrix3 has a bug that stops external python modules from configuring properly. Therefore for now I am requiring users to install the dev version:\
` cd /my/path`\
` git clone https://github.com/MRtrix3/mrtrix3.git`\
` cd mrtrix3`\
` git checkout dev`\
` ./configure`\
` ./build`

- Clone and install this repository:\
` cd /my/path`\
` git clone https://github.com/badesar1/designer_v2_dev.git`\
` cd designer_v2_dev`\
` python setup.py install`
- Add the following line to your `.bashrc, .bash_profile, .zprofile` file:\
`export PYTHONPATH=/my/path/mrtrix3/lib`

To update your installation, just run `git pull` in the `designer_v2_dev` folder, and then rerun `python setup.py install`.

## Example usage for meso data
An example script can be found in the examples folder. It is copied here as well. As you can see, preprocessing and fitting are now split into two separate functions: designer for preprocessing and tmi for fitting.
## Example usage for "meso" data
An example script can be found in the examples folder. It is copied here as well. Preprocessing and fitting are now split into two separate functions: designer for preprocessing and tmi for fitting.

```
#!/bin/bash
Expand All @@ -60,17 +29,3 @@ tmi \
-nocleanup \
designer2_test.mif designer2_params_test
```

## TODO:
### Things that need to be done before this can be used in production:

- **Integration testing**. I created a directory called tests. Typically this has 2 types of scripts, integration test scripts and unit test scripts. We dont really need unit testing here, but we do need integration testing.
We need a script that will take a few example cases (preferably different resolutions, b-shells, etc) and we run designer in as many permutations as possible. We select different input datatypes, output types, and use every available option in every combination. We test what breaks designer so that we can go in and do bug fixing in an organized manner.

- **TMI**. TMI is for the most part entirely untested. It does produce sensible parameter maps, but this is the most agressive change we are making to designer and we should meet and discuss the proper way to test its functionality.

- **Documentation**. We need thorough documentation explaining all availabe options. What intput types can you use, what outputs, what different preprocessing steps are available, algorithms, etc.

- **Referencing**. The code for TMI is not yet properly referenced.

- **Comparison to Designer V1**. Now this this is 100% in python, we can expect some subtle changes in performance compared to matlab. We should run a few examples using both this version and the old, with the same processing under the hood and evaluate differences.

0 comments on commit 23ee61a

Please sign in to comment.