locationtech / locationtech/geotrellis

CRS should not be defaulted to LatLng

Open
#3,287 1 comment 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

enhancement
Dominant language
Scala
Stars
1.4k
Forks
360
Avg merge
6h 57m
Merged PRs (30d)
12

Description

Describe the bug

For an image without a Spatial Reference system, the default proj4string value of LatLng is being set.

To Reproduce

Provide as able:

  • Steps to reproduce the behavior:
    Ingest a GeoTiff that doesn't have a Spatial Reference System.

  • Code example:
    val geoTiffRasterSource: GeoTiffRasterSource = GeoTiffRasterSource(geotiffPath)
    val crs: CRS = geoTiffRasterSource.crs

  • Inputs

https://drive.google.com/file/d/1P9HPB9uOP3-b48DwBcZ1CIT5UtdYdj-9/view?usp=sharing

  • Actual output
    crs = LatLng

  • Expected output
    crs = None

Expected behavior

Instead of a default value LatLng, None value should be assigned to CRS.

Environment

  • Java version: 1.8.0_265
  • Scala version: 2.12.10
  • GeoTrellis version: 3.5.0

Additional context

If this is not possible, is there any way I could get the actual Proj4String of the image before assigning the CRS value(LatLng) to the image?

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start at the GeoTiffRasterSource.crs entry point and reproduce the behavior with the linked GeoTiff, checking how a missing Spatial Reference System becomes LatLng. Done means an image without a Spatial Reference System reports CRS as None rather than LatLng; verify the result with the Scala code example.

Written by the indexing model from the issue text.

Assessment

Tech stack
scala
Domain
data
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.