dmitrishastin
commented
1 year ago Hi Philip
Thanks for your comment - this is good to know and be useful to me and hopefully others. I am not actively working on this currently so not keeping it up to date. I will make a note later on (currently travelling) but probably won't go beyond that / ensure compatibility with the newer versions.
All the best,
Dmitri
On Tue, 28 Mar 2023, 14:23 ppruc, @.***> wrote:
Dear Dimitri,
thanks for providing the code! Unfortunately the solution provided in the documentation leads to an issue when trying to build the mrtrix3 surface seeding fork, leading to the following error message:
... failed with output
In file included from src/dwi/tractography/seeding/basic.cpp:17: src/dwi/tractography/seeding/basic.h:165:11: error: unknown type name 'MEMALIGN' { MEMALIGN(Coordinate_parser) ^ src/dwi/tractography/seeding/basic.h:165:38: error: expected ';' at end of declaration list { MEMALIGN(Coordinate_parser) ...
It seems like the provided solution no longer works because it includes a git rebase command that introduces changes from the newest mrtrix3 development branch that haven't been included in the code. Specifically MRtrix3/mrtrix3#2297 https://github.com/MRtrix3/mrtrix3/pull/2297 introduces support for qt6 which now results in issues of memory alignment ( @.*** https://github.com/MRtrix3/mrtrix3/commit/b613360f21087506e06eaaf1799b365c4ce28206) as the MEMALIGN macro-element was omitted in the updated developer branch. Users who run the provided solution after MRtrix3/mrtrix3#2297 https://github.com/MRtrix3/mrtrix3/pull/2297 was merged (20.12.2022) likely will encounter the same issue.
When not running the rebase command the mrtrix3 fork can be built successfully.
Cheers Philip
— Reply to this email directly, view it on GitHub https://github.com/dmitrishastin/SAF/issues/3, or unsubscribe https://github.com/notifications/unsubscribe-auth/ANC6TAX5CGWFBB4J4EDFXQ3W6JYWRANCNFSM6AAAAAAWKBN7GM . You are receiving this because you are subscribed to this thread.Message ID: @.***>
Dear Dimitri,
thanks for providing the code for compiling the mrtrix3 surface-seeding fork! Unfortunately the solution in the documentation leads to an issue when trying to build the fork, leading to the following error message:
It seems like the provided solution no longer works because it includes a git rebase command that introduces changes from the newest mrtrix3 development branch that haven't been included in the code. Specifically MRtrix3/mrtrix3#2297 introduces support for qt6 which now results in issues of memory alignment (https://github.com/MRtrix3/mrtrix3/pull/2297/commits/b613360f21087506e06eaaf1799b365c4ce28206) as the MEMALIGN macro-element was omitted in the updated developer branch. Users who run the provided solution after MRtrix3/mrtrix3#2297 was merged (20.12.2022) likely will encounter the same issue.
When not running the rebase command the mrtrix3 fork can be built successfully.
Cheers Philip