Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

terra support #64

Closed
Nowosad opened this issue Dec 17, 2021 · 8 comments
Closed

terra support #64

Nowosad opened this issue Dec 17, 2021 · 8 comments

Comments

@Nowosad
Copy link
Contributor

Nowosad commented Dec 17, 2021

Hi @jhollist -- quick question -- do you have any plans to add terra support for the elevatr package?

@jhollist
Copy link
Owner

I do have plans for some fairly extensive re-factoring. Removing sp entirely. I haven't been paying too much attention to the development of terra or stars, but figuring out what to do about raster was on the list too.

So short answer is more or less, yes! Just not sure of the timing at the moment.

@Nowosad
Copy link
Contributor Author

Nowosad commented Dec 17, 2021

Great -- I asked because I want to add elevatr to the chapter 7 of the Geocomputation with R book. Let me know if you need some help with the refactoring.

@jhollist
Copy link
Owner

jhollist commented Dec 17, 2021 via email

@youngdjn
Copy link

Another vote for upgrading to terra! Thanks for considering this.

@jhollist
Copy link
Owner

I am taking a pretty close look at my schedule coming up and hoping to chunk out some time just for elevatr! This is at the top of the list,

@jhollist
Copy link
Owner

jhollist commented Jun 6, 2023

All, see #83 (comment) I am slowly getting there. Hope to have something that resembles a working version in main this week.

@Nowosad Nowosad closed this as completed Oct 9, 2023
@Nowosad
Copy link
Contributor Author

Nowosad commented Nov 25, 2023

@jhollist Thanks for working on that. I will add elevatr to the data packages table in Geocompr ed2.

@jhollist
Copy link
Owner

Thanks for waiting for me @Nowosad. Took me long enough to get to it. I expect another update in the next 6 months or so to Version 1.0 that will make the final switch and return SpatRasters instead of RasterLayers. I kept that last part in to give folks time to get ready for that switch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants