Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement the new Domain construct for CF-1.9 #111

Closed
davidhassell opened this issue Jan 7, 2021 · 0 comments · Fixed by #119
Closed

Implement the new Domain construct for CF-1.9 #111

davidhassell opened this issue Jan 7, 2021 · 0 comments · Fixed by #119
Labels
enhancement New feature or request

Comments

@davidhassell
Copy link
Contributor

The new CF domain variable (cf-convention/cf-conventions#301) requires a CF data model domain construct. In the existing data model, the domain is represented by an abstract Domain concept, but the new CF-netCDF domain variable "promotes" the domain to construct status, on a similar footing to the Field class.

Implementation notes:

  • Domain variables can exist independently of data variables, and so cfdm needs to be able to read them from netCDF files.
  • Climatological time bounds in a Domain do not have access to any cell methods
  • The new Domain construct will need much of the functionality that currently only exists for a Field construct, so refactoring is required to maximise code reuse.
@davidhassell davidhassell added the enhancement New feature or request label Jan 7, 2021
@davidhassell davidhassell added this to the CF-1.9 milestone Jan 7, 2021
@davidhassell davidhassell changed the title Implement the new Domain construct Implement the new Domain construct for CF-1.9 Feb 3, 2021
@davidhassell davidhassell modified the milestones: CF-1.9, 1.9.0.0 May 12, 2021
@davidhassell davidhassell modified the milestones: 1.9.0.0, CF-1.9 Aug 26, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant