Open
Conversation
- Create abstract types AbstractTile and AbstractTileGrid - Add CRS field to Tile and TileGrid types - Implement interface functions: crs(), x(), y(), z(), zoom(), bounds() - Maintain backward compatibility with default WebMercator CRS - Add comprehensive tests for CRS functionality - Update exports to include new interface functions This allows tiles to store and query their coordinate reference system, making it explicit which CRS a tile is using rather than implicit. 🤖 Generated with Claude Code Co-Authored-By: Claude <noreply@anthropic.com>
- Replace custom crs() function with GeoInterface.crs() - Remove crs from exports (users should use GeoInterface.crs) - Update all usages in code and tests - This follows Julia ecosystem conventions for geospatial interfaces 🤖 Generated with Claude Code Co-Authored-By: Claude <noreply@anthropic.com>
Member
Author
- Create TileGridSpec type to define custom tile grids with bbox, zoom, and size - Add CustomTile type that references a TileGridSpec for flexible tiling - Add CustomTileGrid type for managing collections of custom tiles - Implement AbstractTile/AbstractTileGrid interfaces for custom types - Add create_raster_tiles() helper for tiling raster datasets - Add comprehensive tests for all custom tile functionality This allows MapTiles.jl to work with arbitrary tilesets beyond web mercator, such as tiles generated from raster overviews or custom projections. 🤖 Generated with Claude Code Co-Authored-By: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
This PR adds explicit CRS (Coordinate Reference System) support to tiles and tile grids, making it clear which coordinate system a tile is using.
Changes
AbstractTileandAbstractTileGridfor interface definitionsTileandTileGridconcrete typescrs(tile)- Get the CRS of a tile/gridx(tile),y(tile),z(tile)- Access tile indiceszoom(tile)- Alias for z(tile)bounds(tile)- Get bounds in native CRSbounds(tile, target_crs)- Get bounds projected to target CRSTest plan
test/test_crs_interface.jl)🤖 Generated with Claude Code