Skip to content

ConstantTile with nodata: support correct celltype conversion - #3552

Closed
jdries wants to merge 7 commits into
locationtech:masterfrom
VitoTAP:3525
Closed

jdries wants to merge 7 commits into
locationtech:masterfrom
VitoTAP:3525

Conversation

@jdries

@jdries jdries commented Oct 10, 2024 •

Copy link
Copy Markdown
Contributor

#3525

Overview

Test and fix for the issue above.

Checklist

  • ./CHANGELOG.md updated, if necessary. Link to the issue if closed, otherwise the PR.
  • Module Hierarchy updated, if necessary
  • docs guides update, if necessary
  • New user API has useful Scaladoc strings
  • Unit tests added for bug-fix or new feature

Closes #3525

@pomadchin pomadchin added the bug label Oct 10, 2024
Comment thread raster/src/main/scala/geotrellis/raster/ConstantTile.scala Outdated
Comment thread raster/src/main/scala/geotrellis/raster/ConstantTile.scala Outdated

@moradology moradology left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A couple of small things but I think this looks good after that

Comment thread raster/src/main/scala/geotrellis/raster/ConstantTile.scala Outdated
Comment thread raster/src/main/scala/geotrellis/raster/ConstantTile.scala Outdated
Comment thread raster/src/main/scala/geotrellis/raster/ConstantTile.scala
@pomadchin

pomadchin commented Oct 28, 2024 •

Copy link
Copy Markdown
Member

Unfotunately pushes into this PR branch are forbidden, I forked off to cleanup the fix a bit: #3553

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Convert NODATA ByteConstantTile and ShortConstantCellType: unexpected result

3 participants