Skip to content
GitLab
Projects Groups Topics 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
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 22
    • Issues 22
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 17
    • Merge requests 17
  • 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

On Monday, June 12, 2023, from 9.00 to 10.00 am there will be a maintenance with a short downtime of the GitLab service.

  • ift
  • NIFTy
  • Merge requests
  • !147

re-organize PowerSpace

  • Review changes

  • Download
  • Email patches
  • Plain diff
Closed Martin Reinecke requested to merge index_games into master Jun 04, 2017
  • Overview 4
  • Commits 2
  • Pipelines 12
  • Changes 10

This branch aims at making the current PowerSpace design simpler.

The following changes have been made so far:

  • the PowerSpace members are always fully computed upon construction of the PowerSpace; caching has been removed. This still has performance drawbacks which I hope to address in the future.
  • the parameters logarithmic and nbin can still be supplied to the constructor, but they are no longer part of the PowerSpace's state. They are used to compute the binbounds, and forgotten afterwards. (In the current implementation it is possible to set these attributes to values which contradict the actual binbounds, which is inconsistent.)
  • pundex is currently not supported, but can be reintroduced if needed.
  • "natural" binning (which happens if neither logarithmic, nbin or binbounds are specified to the constructor) has been improved to avoid spurious extra bins due to numerical noise.
  • index computation has been accelerated significantly
  • the config property has been removed; binbounds is now a direct property of the class.

I'm sure that a lot will have to change before this can be merged; please consider this merge request mainly as a starting point for discussion!

As a quick demonstration of the advantages of the new implementation, run the following commands in both "master" and "index_games" branch and watch CPU time and memory consumption:

a=RGSpace((10000,10000),harmonic=True)
p=PowerSpace(a,binbounds=sqrt(2.)*np.arange(10000.))
Edited Jun 13, 2017 by Martin Reinecke
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: index_games