Skip to content
GitLab
Projects Groups Topics 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
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 3
    • Issues 3
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 3
    • Merge requests 3
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Container Registry
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • GeodynamoGeodynamo
  • pygeodynpygeodyn
  • Merge requests
  • !51

Mem error

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Mathieu Istas requested to merge mem_error into master Sep 23, 2020
  • Overview 7
  • Commits 1
  • Pipelines 0
  • 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
Reviewers
Request review from
Time tracking
Source branch: mem_error