Skip to content

Prepare new release#42

Open
mrustl wants to merge 156 commits into
masterfrom
dev
Open

Prepare new release#42
mrustl wants to merge 156 commits into
masterfrom
dev

Conversation

@mrustl
Copy link
Copy Markdown
Member

@mrustl mrustl commented Sep 20, 2022

This change is Reviewable

@mrustl mrustl requested a review from hsonne September 20, 2022 14:52
hsonne and others added 28 commits September 20, 2022 17:57
You may now read a shape file on your own,
and pass the corresponding spatial object
(or e.g. a subset of it) to the funcion
read_monthly_data_over_shape(). You may also
set both, "file" and "shape" to NULL and should
receive data for the whole area of Germany.
r@data@values is logical(0) for the whole of
Germany, however getValues() returns the values
use coordinate matrix directly within []
version 1 is deprecated
to remove Warning "LazyData DB of 5.2 MB
without LazyDataCompression set"
Remove
- filter_by_extension_asc_gz()
- filter_by_extension_tgz()

Ignore case when checking file name extension
in download_if_not_there()
hsonne and others added 30 commits May 22, 2023 15:48
using roxygen2md::roxygen2md()
The script identifies what exported functions
are related to each other and modifies the
source files by adding a seealso section to
the Roxygen header.
Use download_and_extract_radolan()
- improve names, use intermediate variables
  "first_url", "last_url" for better
  readability
- use grep(p, x, value = TRUE) instead of
  x[grep(p, x)]
- download_if_not_there()
  -> download()
- check_or_download_shapes_germany()
  -> download_shapes_germany()
- add arg. "download" to default_projection_file()
Use variable "is_no_extension" instead
and make the existing functions wrappers around
it:
- read_daily_data_over_shape()
- read_monthly_data_over_shape()
and simplify the safe selection of the function
Use more general argument "resolution" instead
of logical argument "is_daily". This allows to
extend the function to more time resolutions.
as last try didnt work out
which worked locally using remotes::install_version("rgdal", "1.6-7")
is not implemented in remotes::remotes::install_deps() (in remotes@2.5.0) despite the fact that it is documented here:
https://remotes.r-lib.org/#dependencies-on-github
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