Skip to content

HTML: add safe bbox function for dateline crossing#2352

Merged
tomkralidis merged 1 commit into
masterfrom
bbox-safe-dateline-js
Jun 4, 2026
Merged

HTML: add safe bbox function for dateline crossing#2352
tomkralidis merged 1 commit into
masterfrom
bbox-safe-dateline-js

Conversation

@tomkralidis
Copy link
Copy Markdown
Member

Overview

This PR adds support to HTML to render (collection) polygons supporting cases with dateline crossing. Note that data is not changed per se, as the bbox adjustment is solely for visualization purposes.

Related Issue / discussion

Downstream in World-Meteorological-Organization/wis2box#1103

Additional information

Current master:
image
This PR:

image

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 added this to the 0.24.0 milestone Jun 3, 2026
@tomkralidis tomkralidis added the ui User interface label Jun 3, 2026
@tomkralidis tomkralidis requested a review from a team June 3, 2026 14:38
@tomkralidis tomkralidis force-pushed the bbox-safe-dateline-js branch 4 times, most recently from aff92d6 to 2196c0e Compare June 3, 2026 16:48
@webb-ben
Copy link
Copy Markdown
Member

webb-ben commented Jun 3, 2026

Would this not affect a bbox of the whole world? Does the bbox function always get passed WGS84?

@tomkralidis tomkralidis force-pushed the bbox-safe-dateline-js branch from 2196c0e to c0daaf5 Compare June 3, 2026 21:18
@tomkralidis
Copy link
Copy Markdown
Member Author

Would this not affect a bbox of the whole world? Does the bbox function always get passed WGS84?

Updated. The bbox function is working off 4326 as a rule (just as existing functionality to plot is bound to 4326).

Copy link
Copy Markdown
Contributor

@pvgenuchten pvgenuchten left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Much cleaner Tom, thanx

@tomkralidis tomkralidis merged commit 0c756e1 into master Jun 4, 2026
9 checks passed
@tomkralidis tomkralidis deleted the bbox-safe-dateline-js branch June 4, 2026 01:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ui User interface

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants