Skip to content

Implement Raster.astype#423

Merged
b-j-karl merged 3 commits intodevelopfrom
copilot/implement-raster-astype
Mar 3, 2026
Merged

Implement Raster.astype#423
b-j-karl merged 3 commits intodevelopfrom
copilot/implement-raster-astype

Conversation

Copy link
Contributor

Copilot AI commented Feb 23, 2026 •
edited
Loading

  • Add DTypeLike to TYPE_CHECKING imports in raster.py
  • Implement Raster.astype(dtype) method in raster.py
  • Add TestAstype test class in test_raster.py
  • Document Raster.astype in README under Cell-wise Operations
Original prompt

This section details on the original issue you should resolve

<issue_title>Implement Raster.astype</issue_title>
<issue_description>So we can do

slope_raster = slope_raster.astype("float32")

Instead of

slope_raster.arr = slope_raster.arr.astype("float32")

Useful for method chaining. Similar to to_crs and other cleanup methods</issue_description>

Comments on the Issue (you are @copilot in this section)

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

@nathanjmcdougall
Implement Raster.astype method
e953387 
Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com>
Copilot AI changed the title [WIP] Implement Raster.astype for method chaining Implement Raster.astype Feb 23, 2026
@nathanjmcdougall nathanjmcdougall marked this pull request as ready for review March 2, 2026 22:28
@nathanjmcdougall nathanjmcdougall marked this pull request as draft March 2, 2026 23:10
@nathanjmcdougall
Copy link
Contributor

nathanjmcdougall commented Mar 2, 2026

@copilot please document this new function in the README

@nathanjmcdougall
Document Raster.astype in README
4c3cb01 
Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Mar 2, 2026

@copilot please document this new function in the README

Added Raster.astype(dtype) to the Cell-wise Operations section of the README in commit 4c3cb01.

Copilot AI requested a review from nathanjmcdougall March 2, 2026 23:11
@nathanjmcdougall nathanjmcdougall marked this pull request as ready for review March 2, 2026 23:52
@t-t-sonarqube
Copy link

t-t-sonarqube bot commented Mar 2, 2026

Passed Quality Gate passed

Issues

Measures

Project ID: tonkintaylor_rastr_c6820163-843b-45ef-9d15-db4bc95045e9

View in SonarQube

b-j-karl
b-j-karl reviewed Mar 3, 2026
View reviewed changes
@b-j-karl b-j-karl merged commit 6f85358 into develop Mar 3, 2026
15 checks passed
@b-j-karl b-j-karl deleted the copilot/implement-raster-astype branch March 3, 2026 02:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@b-j-karl b-j-karl b-j-karl approved these changes

+1 more reviewer

@nathanjmcdougall nathanjmcdougall nathanjmcdougall approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@nathanjmcdougall nathanjmcdougall

Copilot code review Copilot

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Implement Raster.astype

3 participants

@nathanjmcdougall @b-j-karl