Skip to content

GitLab

  • Menu
Projects Groups Snippets
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
  • pygeodyn pygeodyn
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 3
    • Issues 3
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 1
    • Merge requests 1
  • Deployments
    • Deployments
    • Releases
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • Geodynamo
  • pygeodynpygeodyn
  • Merge requests
  • !51

Mem error

  • Review changes

  • Download
  • Email patches
  • Plain diff
Merged Mathieu Istas requested to merge mem_error into master Sep 23, 2020
  • Overview 7
  • Commits 1
  • Changes 1

Branch that that made pygeodyn less memory intensive when computing the PCA for the flows U, with the downside of being slower by a tiny amount (around 0.3 secs on a personal laptop). It simply delay the loading of the magnetic field and secular variation and wait for the PCA to be done. That way the space taken by magnetic data is not added on top of the PCA (which uses SVD, a linear operation that is memory costly) of the core flows.

If memory related problems arises again, more sophisticated solutions such as dask arrays could be used.

Assignee
Assign to
Reviewer
Request review from
Time tracking
Source branch: mem_error