Skip to content

Mapscript catchup with changes in base class and crs#2380

Merged
tomkralidis merged 8 commits into
geopython:masterfrom
doublebyte1:mapscript
Jun 25, 2026
Merged

Mapscript catchup with changes in base class and crs#2380
tomkralidis merged 8 commits into
geopython:masterfrom
doublebyte1:mapscript

Conversation

@doublebyte1

@doublebyte1 doublebyte1 commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Overview

Related Issue / discussion

Catchup of the mapscript provider with the changes introduced in this PR: #2358

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

@doublebyte1 doublebyte1 requested review from a team, tomkralidis and webb-ben June 25, 2026 16:53
@doublebyte1 doublebyte1 added bug Something isn't working OGC API - Maps OGC API - Maps labels Jun 25, 2026
Comment thread pygeoapi/provider/mapscript_.py Outdated
"""
try:
epsg_code = int(epsg_code)
except (ValueError, TypeError) as e:

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

except (ValueError, TypeError) as err:

(err is our usual convention var for exceptions)

@tomkralidis tomkralidis added this to the 0.24.0 milestone Jun 25, 2026
@tomkralidis tomkralidis merged commit fde6266 into geopython:master Jun 25, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working OGC API - Maps OGC API - Maps

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants