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.

Sign up for GitHub

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

Review rpygeo #39

Open
wants to merge 13 commits into
base: gh-pages
Choose a base branch
from
+92 −0
Open

Review rpygeo #39

Changes from 3 commits
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
ecf0213
Merge pull request #1 from r-spatial/gh-pages
be-marc Feb 23, 2019
7f15104
First draft
be-marc Mar 10, 2019
59bbfed
rename
pat-s Mar 10, 2019
06f4a2c
Second draft
be-marc Mar 14, 2019
f7c7d33
Update _rmd/2019-03-11-RPyGeo-1_0_0.Rmd
be-marc Jun 1, 2019
7e057b8
Update _rmd/2019-03-11-RPyGeo-1_0_0.Rmd
be-marc Jun 1, 2019
4e95e7d
Apply suggestions from code review
be-marc Jun 1, 2019
0339009
Change pipe operator paragraph
be-marc Jun 3, 2019
2e14017
Update date and remove old post
be-marc Jun 3, 2019
fa852c5
Add markdown file
be-marc Jun 20, 2019
8dbe82d
Change URL
be-marc Jul 20, 2019
447ca25
Change date
be-marc Jul 20, 2019
dc57df8
Remove wrong post
be-marc Jul 20, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 23 additions & 0 deletions _rmd/2019-03-10-RPyGeo-1_0_0.Rmd
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
---
layout: post
title: "RPyGeo 1.0.0"
date: "`r format(Sys.time(), '%d %B, %Y')`"
comments: true
author: Marc Becker
categories: r
---

<script src="https://cdnjs.cloudflare.com/ajax/libs/mathjax/2.7.0/MathJax.js?config=TeX-AMS-MML_HTMLorMML" type="text/javascript"></script>

[DOWNLOADHERE]

RPyGeo 1.0.0 has been released on CRAN. The RPyGeo package establishes an interface to the geoprocessing tools of ArcGIS from within R. ArGIS is a geographic information system developed by Esri.
Copy link
Member

Choose a reason for hiding this comment

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

Link to ArcGIS page. Maybe change to "ArcGIS is one of the leading [...]".

Copy link
Member

Choose a reason for hiding this comment

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

add the release date?


The new RPyGeo package accesses the ArcPy Python side-package via the [reticulate](https://cran.r-project.org/web/packages/reticulate/index.html) package. ArcGIS provides almost all of it's geoprocessing tools via ArcPy. With RPyGeo these functions are available in R. Additonally RPyGeo provides utilitiy functions to create a seamless workflow between ArcGIS and R.
Copy link
Member

Choose a reason for hiding this comment

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

  • "The new version of the package"

  • "ArcPy Python side-package" sounds a bit strange, rephrase?

  • Put "RPygeo" in backticks or in italics

  • do a spellcheck


In 2015 Esri released the R-ArcGIS Bridge, a software that also connects ArcGIS and R. Essentially, the R-ArcGIS Bridge offers read, write and conversion functions to transfer data from ArcGIS to R and vice versa. The idea is to use the large number of R packages to solve spatial problems, which cannot be solved with ArcGIS alone. R scripts can be run as geospatial scripts with an interactive user inferface from within ArcGIS. However, no functionality is added for the R user. RPyGeo is able to run the geoprocessing tools of ArcGIS directly from the R session.
Copy link
Member

Choose a reason for hiding this comment

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

  • link to R-ArcgGIS bridge

"The idea is to use the large number of R packages to solve spatial problems"

-> sth like "to use R to add additional capabilities for solving spatial problems"

"R scripts can be run as geospatial scripts"

-> put "geospatial scripts" in parantheses or explain better what is meant by this in the context of ArcGIS

"However, no functionality is added for the R user. RPyGeo is able to run the geoprocessing tools of ArcGIS directly from the R session."

-> do you want to showcase a difference here? this does not become clear. Maybe add a bit more detail.

The update added many new features to the package. It is now possible to chain ArcPy functions togheter with the pipe operator. Help files for all ArcGIS functions can be viewed directly inside RStudio. Moreover, map algebra expressions can be used to modify raster objects.
Copy link
Member

Choose a reason for hiding this comment

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

The update added many new features to the package. I

"With the new version it is now...

It is now possible to chain ArcPy functions togheter with the pipe operator

-> It is now possible to chain ArcPy functions using the pipe operator


For a detailed instruction on how to use RPyGeo we would like to refer to the [vignette](https://cran.r-project.org/web/packages/RPyGeo/RPyGeo.pdf). It includes a tutorial with all essential RPyGeo functions.
Copy link
Member

Choose a reason for hiding this comment

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

For a detailed instruction on how to use RPyGeo we would like to refer to the vignette.

-> For a detailed instruction on how to use RPyGeo please see the corresponding vignette. (Maybe a pkgdown site would help here? But maybe problematic to generate one on Travis as we need Windows)

Copy link
Contributor

Choose a reason for hiding this comment

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

Also the link is not to the vignette but to the reference PDF. The vignette is at https://cran.r-project.org/web/packages/RPyGeo/vignettes/RPyGeo.html

Copy link
Member

Choose a reason for hiding this comment

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

@be-marc


The devolepment of RPyGeo is now integrated into the r-spatial.org community. If you find any bugs please report them at our new development repository [r-spatial/RPyGeo](https://github.com/r-spatial/RPyGeo) or help us fix them via a pull request.
Copy link
Member

Choose a reason for hiding this comment

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

The devolepment of RPyGeo is now integrated into the r-spatial.org community. If you find any bugs please report them at our new development repository r-spatial/RPyGeo or help us fix them via a pull request.

The source code of RpyGeo is now hosted on Github within the r-spatial organization.