Skip to content
GitLab
Projects Groups Topics Snippets
  • /
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
  • Sign in
  • CSIndicators CSIndicators
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Graph
    • Compare revisions
  • Issues 15
    • Issues 15
    • List
    • Boards
    • Service Desk
    • Milestones
  • Merge requests 3
    • Merge requests 3
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Releases
  • Packages and registries
    • Packages and registries
    • Package Registry
    • Terraform modules
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • Earth SciencesEarth Sciences
  • CSIndicatorsCSIndicators
  • Merge requests
  • !55

Change default time_dim to 'time' and correct unit tests

  • Review changes

  • Download
  • Patches
  • Plain diff
Merged Eva Rifà requested to merge develop-default_time_dim into master Oct 13, 2023
  • Overview 1
  • Commits 1
  • Pipelines 1
  • Changes 46

As mentioned in issue #35 (closed) and previously discussed in an R user meeting, the time_dim parameter default value, will be changed to 'time' instead of 'ftime'. This change will be a benefit if users use startR::Start to load data and not s2dv::Load because the latter has time_dim fixed as 'ftime' whereas Start is fleixble.

Another change that will take place within this branch is to correct the pipeline. The corrections are also related with the default value of time_dim in unit tests which had some inconsistencies previously.

Edited Oct 13, 2023 by Eva Rifà
Assignee
Assign to
Reviewers
Request review from
Time tracking
Source branch: develop-default_time_dim