FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

'estimate densities from point patterns': PROJ>=6 compliance by florisvdh · Pull Request #195 · inbo/tutorials · GitHub

/ tutorials Public

'estimate densities from point patterns': PROJ>=6 compliance - #195

Merged
florisvdh merged 10 commits into
densityfrom
density_fv
Feb 17, 2021
Merged

'estimate densities from point patterns': PROJ>=6 compliance#195
florisvdh merged 10 commits into
densityfrom
density_fv

Conversation

florisvdh commented Sep 16, 2020
edited
Loading

Copy link
Copy Markdown
Member

This amends PR #27

  • move files in the right folder
  • use (more) future-proof R code (PROJ>=6 compliant), and add a bit explanation on this.

florisvdh added 10 commits June 23, 2020 11:00
The use of 'proj4string' in R code is confusing in times where PROJ.4 strings are obsoleted.
It can be expected that a more general function name to set an object's CRS will sooner or later
be promoted instead of proj4string<-, hence preparing for that already by using the function instead of the slot.
I expect the proj4string slot will most probably remain in use for compatibility reasons, but
explicitly naming it is better avoided in times where WKT2 strings are actually used to define a CRS.
florisvdh marked this pull request as draft September 16, 2020 13:13
florisvdh changed the title PROJ>=6 compliance in 'estimate densities from point patterns' 'estimate densities from point patterns': PROJ>=6 compliance Sep 23, 2020
florisvdh requested a review from ThierryO September 23, 2020 13:13
florisvdh marked this pull request as ready for review September 23, 2020 13:15
florisvdh merged commit 3408a91 into density Feb 17, 2021
florisvdh deleted the density_fv branch February 17, 2021 10:09
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL