Skip to content
This repository has been archived by the owner on Aug 29, 2023. It is now read-only.

Confusing requirements for ENSO index calculation (UC6) #496

Closed
thomas-eldridge opened this issue Jan 8, 2018 · 2 comments
Closed

Confusing requirements for ENSO index calculation (UC6) #496

thomas-eldridge opened this issue Jan 8, 2018 · 2 comments
Assignees
Milestone

Comments

@thomas-eldridge
Copy link

Expected behavior

When using enso_nino34 operation, expect to be able to provide a dataset from which the climatology values of SST are deduced.

Actual behavior

Operation requires a file as an argument - file is a netCDF Dataset of climatology values. In current happy path, this requires that a user:

  1. temporally aggregates an ingested SST dataset into months,
  2. calculates long term means for each cell for each month
  3. saves this as an external netCDF
  4. ingests this netCDF as the argument in enso_nino34

Suggestion

The steps above are not hinted at in the description of the enso_nino34 operation. More clarity is required so that users without access to these issue pages can work out what to do.

A better fix would involve having an option to calculate the climatology at the same time as calculating ENSO index: the user could provide the SST dataset and time period in the same dialogue box as for the enso_nino34 operation.

Specifications

Cate 1.0 dev 3

@forman
Copy link
Member

forman commented Jan 10, 2018

Does the same apply to Cate 1.0?
Note that version 1.0 dev 3 is outdated.

@forman forman added the ops label Jan 10, 2018
@JanisGailis JanisGailis added this to the IRM8 milestone Feb 5, 2018
@JanisGailis
Copy link
Member

JanisGailis commented Mar 5, 2018

@thomas-eldridge I agree that this is confusing. The main issue here is that the generation of a representative climatology can easily take a lot of time and memory (days and more than a laptop has), while the index calculation itself is a pretty fast operation, hence we can not really risk users running long_term_average each time they do an index calculation.

But I agree that it should at least be documented in the description that the reference dataset can be created using the long_term_average operation.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants