Skip to content
GitLab
Projects Groups Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • N NIFTy
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 20
    • Issues 20
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 13
    • Merge requests 13
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • ift
  • NIFTy
  • Merge requests
  • !156

Add Python3 compatibility

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Martin Reinecke requested to merge python3 into master Jun 30, 2017
  • Overview 5
  • Commits 32
  • Pipelines 28
  • Changes 121

This branch adds Python3 compatibility to NIFTy, while also staying compatible to Python2.

At the current state, the code compiles in both language versions and passes almost all tests.

Exception: some HDF5 tests, (1 fail with Python2, 5 fails with Python3). I need help with debugging those. [now fixed]

NOTE: An update to the "mpi_dummy" package is necessary to make it Python3 compatible as well. A merge request has been opened.

Problems encountered:

  • the "hack" in the new method of SmoothingOperator, which returns a DirectSmoothingOperator or an FFTSmoothingOperator depending on the arguments, stopped working with Python3. I have put together a crude workaround, but this could still be improved.

Things to do:

  • enhance continuous integration to test Nifty with Python3 [done now]
Edited Aug 01, 2017 by Martin Reinecke
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: python3