Skip to content

Commit

Permalink
Merge pull request #186 from ha-pu/cran_update
Browse files Browse the repository at this point in the history
Cran update to 0.0.14
  • Loading branch information
ha-pu authored Mar 6, 2023
2 parents 90cf63d + 894c164 commit 726f24e
Show file tree
Hide file tree
Showing 8 changed files with 108 additions and 7 deletions.
6 changes: 3 additions & 3 deletions CRAN-SUBMISSION
Original file line number Diff line number Diff line change
@@ -1,3 +1,3 @@
Version: 0.0.13
Date: 2023-02-23 16:31:02 UTC
SHA: 6842156ba6819d072c8abb0d3bae2d3ecd2381a4
Version: 0.0.14
Date: 2023-03-06 18:32:07 UTC
SHA: ba8e0635c56b8f96a3a67a8253f66eae441d3723
2 changes: 1 addition & 1 deletion DESCRIPTION
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
Type: Package
Package: globaltrends
Title: Download and Measure Global Trends Through Google Search Volumes
Version: 0.0.13.9000
Version: 0.0.14
Authors@R: c(
person("Harald", "Puhr", , "harald.puhr@gmail.com", role = c("aut", "cre")),
person("Jakob", "Muellner", role = "ccp")
Expand Down
2 changes: 1 addition & 1 deletion NEWS.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
# globaltrends 0.0.13.9000
# globaltrends 0.0.14

* Remove dependency on `glue` package, calls of `glue::glue` are replaced with `paste` and `paste0`
* Remove dependency on `ineq` package, calls of `ineq::ineq` are replaced with internal functions
Expand Down
91 changes: 91 additions & 0 deletions cran-comments.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,94 @@
# Resubmission 0.0.14

## Test environments
* local Windows 10 Enterprise, 64 bit (R 4.2.2)
* Windows Server 2022, R-devel, 64 bit on R-Hub
* Ubuntu Linux 20.04.1 LTS, R-release, GCC on R-Hub
* Fedora Linux, R-devel, clang, gfortran on R-Hub
* macOS-latest (release) on GitHub
* windows-latest (release) on GitHub
* ubuntu-latest (devel) on GitHub
* ubuntu-latest (release) on GitHub
* ubuntu-latest (oldrel-1) on GitHub
* Windows Server 2022, R 2023-03-05 r83940 ucrt (devel) on winbuilder
* Windows Server 2022, R 4.2.2 (release) on winbuilder
* Windows Server 2008, R 4.1.3 (oldrelease) on winbuilder

## Test coverage
The test coverage [devtools::test_coverage()] of the package is 96%.

## Reverse dependencies
Results from [revdepcheck::revdep_check()]:

```
We checked 0 reverse dependencies, comparing R CMD check results across CRAN and
dev versions of this package.
* We saw 0 new problems
* We failed to check 0 packages
```

## R CMD check results
There were no ERRORs or WARNINGs.

### Note 1
There is one NOTE that is only found on Windows (Server 2022, R-devel 64-bit):

```
* checking for detritus in the temp directory ... NOTE
Found the following files/directories:
'lastMiKTeXException'
```

As noted in [R-hub issue #503](https://github.com/r-hub/rhub/issues/503), this
could be due to a bug/crash in MiKTeX and can likely be ignored. The same note
was reported in the initial submission.

### Note 2
There is one NOTE that is only found on Fedora Linux (R-devel, clang, gfortran):

```
* checking HTML version of manual ... NOTE
Skipping checking HTML validation: no command 'tidy' found
```

As noted in [R-hub issue #548](https://github.com/r-hub/rhub/issues/548), this
issue appears to be independent of the configuration on the local machine.

## winbuilder
Winbuilder might show a note regarding (possible) invalid URLs:

```
URL: https://trends.google.com/trends/explore?q=%2Fm%2F03phgz&geo=AT
From: inst/doc/globaltrends.html
Status: 429
Message: Too Many Requests
```

This does not signify an invalid URL but indicates that Google Trends blocks
downloads due to too many requests. The same note was reported in the initial
submission.

---

In this resubmission of the globaltrends package I have made the following updates:

* Remove dependency on `glue` package
* Remove dependency on `ineq` package
* Use functions `dbCreateTable` and `dbAppendTable` from `DBI 1.1.0`
* Remove the function `plot_map` from the package and point users to more flexibile alternatives
* Remove dependency on the `maps` package
* Remove reference to non-existent table `data_global` in function `remove_data`
* Adapt test *remove_data6* in `test-computations.R` to avoid check failures on R-oldrease

The adaption of test *remove_data6* in `test-computations.R` will also resolve the
failed CRAN checks for the current version of the package (0.0.13). This is because
the test failure does not signify unintended behavior of the package. Instead it
points to unintended behavior in the test. This issue is now resolved.

Thanks!
Harald Puhr

# Resubmission 0.0.13

## Test environments
Expand Down
10 changes: 10 additions & 0 deletions dev/submission_checks.rmd
Original file line number Diff line number Diff line change
Expand Up @@ -65,6 +65,16 @@ revdepcheck::revdep_check(pkg = "..")
devtools::check_win_devel()
```

* devtools::check_win_release()
```{r}
devtools::check_win_release()
```

* devtools::check_win_oldrelease()
```{r}
devtools::check_win_oldrelease()
```

* devtools::check_rhub()
```{r}
devtools::check_rhub(pkg = "..", email = "harald.puhr@gmail.com")
Expand Down
Binary file modified globaltrends_Vignette.pdf
Binary file not shown.
2 changes: 1 addition & 1 deletion vignettes/globaltrends.Rmd
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ library(globaltrends)
# package version --------------------------------------------------------------
packageVersion("globaltrends")
#> [1] '0.0.13'
#> [1] '0.0.14'
```

# Google Trends
Expand Down
2 changes: 1 addition & 1 deletion vignettes/globaltrends.html
Original file line number Diff line number Diff line change
Expand Up @@ -358,7 +358,7 @@ <h4 class="author">Harald Puhr</h4>
<span id="cb1-9"><a href="#cb1-9" aria-hidden="true" tabindex="-1"></a></span>
<span id="cb1-10"><a href="#cb1-10" aria-hidden="true" tabindex="-1"></a><span class="co"># package version --------------------------------------------------------------</span></span>
<span id="cb1-11"><a href="#cb1-11" aria-hidden="true" tabindex="-1"></a><span class="fu">packageVersion</span>(<span class="st">&quot;globaltrends&quot;</span>)</span>
<span id="cb1-12"><a href="#cb1-12" aria-hidden="true" tabindex="-1"></a><span class="co">#&gt; [1] &#39;0.0.13&#39;</span></span></code></pre></div>
<span id="cb1-12"><a href="#cb1-12" aria-hidden="true" tabindex="-1"></a><span class="co">#&gt; [1] &#39;0.0.14&#39;</span></span></code></pre></div>
<div id="google-trends" class="section level1">
<h1>Google Trends</h1>
<p>Google offers public access to global search volumes from its search
Expand Down

0 comments on commit 726f24e

Please sign in to comment.