Skip to content

GitLab

  • Menu
Projects Groups Snippets
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
  • airGR airGR
  • Project information
    • Project information
    • Activity
    • Labels
    • Members
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 60
    • Issues 60
    • List
    • Boards
    • Service Desk
    • Milestones
  • Redmine
    • Redmine
  • Merge requests 7
    • Merge requests 7
  • CI/CD
    • CI/CD
    • Pipelines
    • Jobs
    • Schedules
  • Deployments
    • Deployments
    • Environments
    • Releases
  • Monitor
    • Monitor
    • Incidents
  • Packages & Registries
    • Packages & Registries
    • Container Registry
  • Analytics
    • Analytics
    • Value stream
    • CI/CD
    • Repository
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Activity
  • Graph
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
Collapse sidebar
  • HYCAR-Hydro
  • airGRairGR
  • Issues
  • #89

Closed
Open
Created Jan 14, 2021 by Delaigue Olivier@olivier.delaigueOwner

Fix airGRteaching 0.2.9.25 reverse dependency

​​​​From: "Uwe Ligges"
To: "airGR"
Cc: "CRAN-submissions"
Date: Jeudi 14 Janvier 2021 07:11:06
Subject: [CRAN-pretest-waiting] CRAN submission airGR 1.6.9.24

​​​​​Dear maintainer,

package airGR_1.6.9.24.tar.gz has been auto-processed. The auto-check found problems when checking the first order strong reverse dependencies.
Please reply-all and explain: Is this expected or do you need to fix anything in your package? If expected, have all maintainers of affected packages been informed well in advance? Are there false positives in our results?

*** Changes to worse in reverse dependencies ***
Debian: https://win-builder.r-project.org/incoming_pretest/airGR_1.6.9.24_20210113_193646/reverseDependencies/summary.txt

Log dir: https://win-builder.r-project.org/incoming_pretest/airGR_1.6.9.24_20210113_193646/
The files will be removed after roughly 7 days.

Pretests: Windows: https://win-builder.r-project.org/incoming_pretest/airGR_1.6.9.24_20210113_193646/Windows/00check.log
Debian: https://win-builder.r-project.org/incoming_pretest/airGR_1.6.9.24_20210113_193646/Debian/00check.log

Changes to worse in reverse depends:

Package: airGRteaching  
Check: R code for possible problems  
New result: NOTE  
  plot.CalGR: no visible global function definition for  
    "plot.OutputsModel"
  Undefined global functions or variables:  
    plot.OutputsModel

If I understand well, even when airGRteaching is installed from sources, the package has registered the space from which the functions it depends on are exported.

Summary:

Package Version CRAN plot.OutputsModel
airGR 1.4.3.65 yes exported
airGR 1.6.9.24 submitted not exported
airGRteaching 0.2.9.25 yes imported from the global environment
airGRteaching 0.2.10.106 to be submitted imported from airGR namespace
library(airGRteaching)
example("CalGR")

Using airGRteaching 0.2.9.25 (current version on the CRAN; it runs well on 0.2.10.106) with airGR 1.6.9.24:

plot(CAL)
Error in plot.OutputsModel(x$OutputsModel, Qobs = x$Qobs, ...) : 
   could not find function "plot.OutputsModel"

Nota: inside plot.CalGR(), plot.OutputsModel() is currently call using plot(OutputsModel)

Although I answer to the CRAN team that the future version of airGRteaching will solve the problem, users with the 1.6.9.24 version of airGR and the 0.2.9.25 version of airGRteaching will not be able to use airGRteaching properly and we can't warn them). In order to avoid breaking the reverse dependency of airGRteaching, I suggest :

  1. export plot.OutputsModel() in airGR 1.6
  2. call plot.OutputsModel() using airGR:::plot.OutputsModel() in airGRteaching 0.2.10
  3. not export plot.OutputsModel() in in the next version of airGR (2.0)
  4. call plot.OutputsModel() using plot() in the next version of airGRteaching (as it is used currently in the version 0.2.9.25)

I think the temporary use of ::: in airGRteaching will solve the transition smoothly for users and will help to avoid breaking airGRteaching.

@david.dorchies and @guillaume.thirel , do you agree? I think I'll ask the opinion of the CRAN team.

Edited Jan 14, 2021 by Delaigue Olivier
Assignee
Assign to
Time tracking