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
repipes the book #794
repipes the book #794
Conversation
🎉 to this. In the actions there's something that could cause this: https://github.com/Robinlovelace/geocompr/runs/6335806113?check_suite_focus=true#step:4:4113 Also I suggest we add |
@@ -263,7 +263,7 @@ There are many reasons (linked to the advantages of the simple features model): | |||
- Enhanced plotting performance | |||
- **sf** objects can be treated as data frames in most operations | |||
- **sf** function names are relatively consistent and intuitive (all begin with `st_`) | |||
- **sf** functions can be combined using `%>%` operator and works well with the [tidyverse](http://tidyverse.org/) collection of R packages\index{tidyverse}. |
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.
Just noticed a typo. That should be:
- **sf** functions can be combined with the `|>` operator and works well with the [tidyverse](http://tidyverse.org/) collection of R packages\index{tidyverse}.
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.
Fixed
06-raster-vector.Rmd
Outdated
st_sfc(crs = crs(srtm)) |> | ||
st_sf() | ||
st_sfc(crs = crs(srtm)) %>% | ||
st_sf(geometry = .) |
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.
Why not:
st_sfc(crs = crs(srtm)) |>
st_sf(geometry = _)
?
Even if that fails on your computer it should pass on actions. I would be curious to see if it does 🤞
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.
Done
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.
Aha didn't see that, looking good 👍
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.
Looking great. Please fix merge conflicts and merge if you get a chance Jakub.
05-geometry-operations.Rmd
Outdated
@@ -464,16 +464,16 @@ plot(us_southhwest) | |||
|
|||
\index{vector!geometry casting} | |||
Geometry casting is a powerful operation that enables transformation of the geometry type. | |||
It is implemented in the `st_cast` function from the **sf** package. | |||
Importantly, `st_cast` behaves differently on single simple feature geometry (`sfg`) objects, simple feature geometry column (`sfc`) and simple features objects. | |||
It is implemented in the `st_cast()` function from the **sf** package. |
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.
👍
06-raster-vector.Rmd
Outdated
st_sfc(crs = crs(srtm)) |> | ||
st_sf() | ||
st_sfc(crs = crs(srtm)) %>% | ||
st_sf(geometry = .) |
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.
Aha didn't see that, looking good 👍
zion_transect = cbind(c(-113.2, -112.9), c(37.45, 37.2)) |> | ||
st_linestring() |> | ||
st_sfc(crs = crs(srtm)) |> | ||
st_sf(geometry = _) |
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.
Nice!
st_sf(data.frame(v = 1:n), geometry = .) | ||
zion_many = st_sample(x = zion, size = n) |> | ||
st_buffer(dist = 500) |> | ||
st_sf(data.frame(v = 1:n), geometry = _) |
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.
👍 really like this.
repipes the book 3579906
No description provided.