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 16
    • Issues 16
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 14
    • Merge requests 14
  • 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
  • !760

Switch to ducc.misc.special_add_at() if available

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Martin Reinecke requested to merge new_add_at into NIFTy_8 Apr 05, 2022
  • Overview 12
  • Commits 3
  • Pipelines 3
  • Changes 2

This should improve performance quite significantly. Since ducc0 is an optional dependency (and since the function is only available on the unreleased ducc0 branch so far), the switch is only made if the functionality is actually found.

@jroth if you have any relevant real-world benchmarks, I'd be happy about feedback!

@parras I'm not yet happy with how I import this in correlated_fields.py, but I don't have a good idea how to do it better. Any ideas?

Edited Apr 05, 2022 by Martin Reinecke
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: new_add_at