Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • S s2dverification
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributor statistics
    • Graph
    • Compare revisions
  • Issues 65
    • Issues 65
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 7
    • Merge requests 7
  • Deployments
    • Deployments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Analytics
    • Analytics
    • Value stream
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • Earth SciencesEarth Sciences
  • s2dverification
  • Merge requests
  • !229

Bugfix for per-paired when NA existence.

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged aho requested to merge develop-Clim_perpaired_na into master Oct 01, 2020
  • Overview 6
  • Commits 2
  • Pipelines 0
  • Changes 1

Hi @nperez

This fix is for issue #245 (closed). To calculate the per-paired climatology, the NA positions along sdate of exp and obs should be consistent. You can see the old code here, it uses extra steps to find the NA positions, which causes the error mentioned in issue 245. Because some mean functions use narm = TRUE, so if that dimension has NA, the NA is removed and the average result contains no NA anymore.

I fixed the bug, but I wonder why the old code was written in that way. Do you have any idea?

Best,
An-Chi

Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: develop-Clim_perpaired_na