Skip to content
This repository has been archived by the owner on Mar 17, 2021. It is now read-only.

Build and use niftyreg_image_resampling module #473

Merged
merged 2 commits into from
Jan 14, 2020

Conversation

RafaelPalomar
Copy link
Contributor

@RafaelPalomar RafaelPalomar commented Jan 14, 2020

Status

READY

Description

Commit 1 -- According to CMakeLists.txt in niftyreg_gpu_resampler,
niftyreg_module_loader.py should be a template file named
niftyreg_module_loader.py.in; building process fails otherwise.

Commit 2 -- Trying to use niftyreg_image_resamplig with niftynet results into niftynet falling back to use a non-GPU image resampler. This change seems to fix the issue.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • My code follows the code style of this project.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

Todos

  • Tests
  • Documentation

Impacted Areas in Application

List general components of the application that this PR will affect:

  • niftyreg_image_resampling

According to CMakeLists.txt in niftyreg_gpu_resampler,
niftyreg_module_loader.py should be a template file named
niftyreg_module_loader.py.in; building process fails otherwise.
@RafaelPalomar RafaelPalomar changed the title Add template extension (.in) to niftyreg_module_loader.py Build and use niftyreg_image_resampling module Jan 14, 2020
@wyli wyli merged commit d7d6b62 into NifTK:dev Jan 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants