Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in
owc-software-python owc-software-python
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 0
    • Issues 0
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge requests 0
    • Merge requests 0
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Operations
    • Operations
    • Incidents
    • Environments
  • Analytics
    • Analytics
    • CI/CD
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Members
    • Members
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • bodc
  • owc-software-pythonowc-software-python
  • Merge requests
  • !18

Merged
Created Feb 05, 2020 by edsmall@edsmallContributor

ARGODEV-169: refactor region data

  • Overview 0
  • Commits 9
  • Pipelines 8
  • Changes 8

Jira Issue

ARGODEV-169 (https://jira.ceh.ac.uk/browse/ARGODEV-169

Python Implementation

Move the similar code into their own functions and files so that they can be called by both retr_region and get_region

Testing

Two functions were created here, so we have two new test cases:

get_data

  • Test that we retrieve no data if we don't specify where we want the data from
  • Test that we retrieve the correct amount of data
  • Test that if the argo float currently being analysed appears in the data we are retrieving then it is removed
  • Test that the shape of each data array returned is comparatively the same

wrap_longitude

  • Test that we don't wrap values when it is not needed
  • Test that wrapped values give the expected output
  • Test that we don't lost any longitude values

Old Matlab Implementation

Had some code that had clearly been copied and pasted between get_region and retr_region. One part of the code retrieved data from some bottle/ctd/argo files, the other manipulated longitude values to ensure maps looked correct.

Assignee
Assign to
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Source branch: edsmall/refactor-region-data