-
Notifications
You must be signed in to change notification settings - Fork 13
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Core; schema: Changes to use
storageCrsBbox
- inside extent.spatial instead of top-level `storageCrsExtent` - This addresses the issue raised by Tom for coverages in a native CRS other than CRS84 at the code sprint in Evora, bringing this native CRS bounding box right next to the grid definition. - Changes to rely on 'definition' rather than 'crs' for non-spatial dimension
- Loading branch information
1 parent
9f42c18
commit 43d7958
Showing
4 changed files
with
103 additions
and
14 deletions.
There are no files selected for viewing
This file contains 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
This file contains 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
This file contains 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
This file contains 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
43d7958
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tomkralidis This should mostly address the issue regarding coverages with a native CRS other than CRS84 which you raised at the code sprint in Evora.
Now the
storageCrsBbox
(bbox in native CRS) will be right next to thebbox
(in CRS84) inside thespatial
dimension ofextent
, so it will be less confusing that the grid'scoordinates
orcellResolution
are expressed in the native CRS rather than CRS84(h).The
crs
property (the CRS ofbbox
) can also be omitted and is assumed to be CRS84 if not specified. I suggest that this not be specified explicitly unless it needs to be (for CRS84h, or a non-terrestrial CRS) to help avoid the confusion.