Skip to content

Xarrayedr covjson - #2005

Closed
adanb13 wants to merge 5 commits into
geopython:masterfrom
adanb13:xarrayedr-covjson
Closed

Xarrayedr covjson#2005
adanb13 wants to merge 5 commits into
geopython:masterfrom
adanb13:xarrayedr-covjson

Conversation

@adanb13

@adanb13 adanb13 commented May 1, 2025

Copy link
Copy Markdown

Overview

  • fixes coverage json output for xarray edr
  • changing of domain axis time -> t instead of time and providing a data array of time values instead of just start/stop/num
  • replacing NaN vlaues with None so that they can properly be parsed by JSON as null

Related Issue / discussion

Coverage Json being returned from xarray edr provider is currently invalid

Additional information

Dependency policy (RFC2)

  • I have ensured that this PR meets RFC2 requirements

Updates to public demo

Contributions and licensing

(as per https://github.com/geopython/pygeoapi/blob/master/CONTRIBUTING.md#contributions-and-licensing)

  • I'd like to contribute [feature X|bugfix Y|docs|something else] to pygeoapi. I confirm that my contributions to pygeoapi will be compatible with the pygeoapi license guidelines at the time of contribution
  • I have already previously agreed to the pygeoapi Contributions and Licensing Guidelines

@tomkralidis tomkralidis mentioned this pull request May 12, 2025
5 tasks
@tomkralidis

Copy link
Copy Markdown
Member

Superceded by #2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants