search

spinicist / QUIT

A set of tools for processing Quantitative MR Images
Mozilla Public License 2.0
60 stars 21 forks source link

qi_lorentzian JSON file #22

Closed eugenegkim closed 5 years ago

eugenegkim commented 5 years ago

I tried running qi_lorentzian but got this error: Error Missing JSON member: Lorentz

The documentation gives an example JSON file with only "freq" defined. What else has to go in the JSON file?

Thanks

spinicist commented 5 years ago

Hi,

I expanded the functionality of qi_lorentzian significantly earlier this year but forgot to update the documentation. I have now done so, hopefully https://quit.readthedocs.io/en/latest/Docs/MT.html#qi-lorentzian should update shortly.

In short you can now choose how many Lorentzians you want to fit. If you are trying to do WASSR then you only need one.

eugenegkim commented 5 years ago

Thanks, the documentation has updated, but I still get the same error. I'm using the QUIT module on the NaN. I suppose that version expects different inputs to the most recent stable release?

On Mon, Sep 30, 2019 at 2:36 PM Tobias Wood notifications@github.com wrote:

Hi,

I expanded the functionality of qi_lorentzian significantly earlier this year but forgot to update the documentation. I have now done so, hopefully https://quit.readthedocs.io/en/latest/Docs/MT.html#qi-lorentzian should update shortly.

In short you can now choose how many Lorentzians you want to fit. If you are trying to do WASSR then you only need one.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/spinicist/QUIT/issues/22?email_source=notifications&email_token=AMVCJM3VWKOVMPYT22E3BUDQMH6G7A5CNFSM4I3ZNXAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD75VDMA#issuecomment-536564144, or mute the thread https://github.com/notifications/unsubscribe-auth/AMVCJMZI4NNBWRH7P4OD7H3QMH6G7ANCNFSM4I3ZNXAA .

spinicist commented 5 years ago

I've updated the version on the NaN - can you try again please?

eugenegkim commented 5 years ago

It's working now, thanks!

On Tue, Oct 1, 2019 at 11:02 AM Tobias Wood notifications@github.com wrote:

I've updated the version on the NaN - can you try again please?

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/spinicist/QUIT/issues/22?email_source=notifications&email_token=AMVCJM4WCV4NXVNQW6DM5YTQMMN2PA5CNFSM4I3ZNXAKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAAW54I#issuecomment-536964849, or mute the thread https://github.com/notifications/unsubscribe-auth/AMVCJMY752SBHHKJ2TPYBPTQMMN2PANCNFSM4I3ZNXAA .