Skip to content

Conversation

@HenkMutsaerts
Copy link
Member

Linked issue

#257

@MauricePasternak sorry for the delay, this should fix your issue. @mathijs you can test this with NOMARED, if you call the different sequences "sites" (we may want to rename this later).

@HenkMutsaerts HenkMutsaerts linked an issue Jun 19, 2021 that may be closed by this pull request
@MichaelStritt MichaelStritt added the revamp Restructuring of ExploreASL label Jun 21, 2021
@MichaelStritt MichaelStritt changed the title #257 xASL_wrp_CreateBiasfield: Slight revamp Fixes #257 xASL_wrp_CreateBiasfield: Slight revamp Jul 29, 2021
MDijsselhof
MDijsselhof approved these changes Aug 5, 2021
@MDijsselhof MDijsselhof self-requested a review August 5, 2021 11:12
Copy link
Contributor

@MDijsselhof MDijsselhof left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Works fine in the NOMARED dataset, using two different sites!

@jan-petr jan-petr assigned jan-petr and unassigned BeatrizPadrela Aug 16, 2021
@jan-petr jan-petr requested review from jan-petr and removed request for MauricePasternak August 16, 2021 12:22
Copy link
Contributor

@jan-petr jan-petr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK

@jan-petr jan-petr force-pushed the bug-#257_xASL_wrp_CreateBiasfield branch from bb412b6 to 8fd8a16 Compare August 16, 2021 16:03
@jan-petr jan-petr merged commit 8fd8a16 into develop Aug 16, 2021
@jan-petr jan-petr deleted the bug-#257_xASL_wrp_CreateBiasfield branch August 16, 2021 16:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

revamp Restructuring of ExploreASL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CreateBiasfield.m submodule not up-to-date

6 participants