Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New stable release v1.0.20230411 #703

Merged
merged 95 commits into from
Apr 24, 2023
Merged

New stable release v1.0.20230411 #703

merged 95 commits into from
Apr 24, 2023

Conversation

neurolabusc
Copy link
Collaborator

Featured highlights:

casperdcl and others added 30 commits May 9, 2022 05:16
- pydcm2niix => dcm2niix
- thanks to pypi/support#1609
- closes #573
Find pigz.exe if not in current working directory for Windows [issue 650](#650)
 Python package, enables `pip install`.
SeriesInstanceUID  precedence (#655)
More permissive bval creation (#657)
neurolabusc and others added 29 commits February 4, 2023 11:21
Add codespell config, workflow (to check future additions), and fix some typos
This ensures the C & C++ compilers associated with the top-level project
are used consistently.

Note that CMAKE_CXX_COMPILER is not associated with the "openjpeg" external
project because the project only specifies C as a language.
…generator

This commit introduces the module "dcm2niixInitializeBuildType"

It allows to consistently manage the initialization and setting of build type
CMake variables.

It sets the variable EXTERNAL_PROJECT_BUILD_TYPE_CMAKE_ARGS based on the CMake generator
being used:
* If a multi-config generator (e.g Visual Studio) is used, it sets the variable with
  CMAKE_CONFIGURATION_TYPES.
* If a single-config generator (e.g Unix Makefiles) is used, it sets the variable with
  CMAKE_BUILD_TYPE.

Adapted from https://github.com/Slicer/Slicer/blob/5.2/CMake/SlicerInitializeBuildType.cmake
  1. set isForceStackSameSeries = 1 by default
  2. make adjustments for MGH bvecs output
dcm2niix_fswrapper and libdcm2niixfs.a
…r-is-used

Fix build ensuring expected compiler is used
@neurolabusc neurolabusc merged commit fe2f260 into master Apr 24, 2023
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.

8 participants