Skip to content

Updated ISMRMRD to the latest versions#6111

Merged
grdowns merged 1 commit intomicrosoft:masterfrom
dchansen:ismrmrd
May 23, 2019
Merged

Updated ISMRMRD to the latest versions#6111
grdowns merged 1 commit intomicrosoft:masterfrom
dchansen:ismrmrd

Conversation

@dchansen
Copy link
Copy Markdown
Contributor

Also removed unwanted executalbes produced when FFTW is found

@grdowns grdowns self-assigned this Apr 16, 2019
@NancyLi1013
Copy link
Copy Markdown
Contributor

Hi @dchansen, here are the test results from the current CI system:

x86-windows master test notes
ismrmrd Pass Fail Regression
arm-uwp master test notes
ismrmrd Pass Fail Regression

failureLogs.zip

…uld not be optional on platforms with HDF5 available
@dchansen
Copy link
Copy Markdown
Contributor Author

Is this ok to merge now? It would make the life our our users immensely easier if they could simply install through VCPKG, but the current port someone created is quite broken.

@grdowns
Copy link
Copy Markdown
Contributor

grdowns commented May 23, 2019

Hey @dchansen! Sorry for the long wait, and thanks for fixing the merge conflicts that arose during that time. I wanted to make some small changes to disable the test and example building as opposed to allowing them to build and deleting them, but it's fine for now. Thanks for the update and fixes!

@grdowns grdowns merged commit 9c54cc9 into microsoft:master May 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants