Skip to content

replace utils::data with BiocGenerics::data#1

Open
Artur-man wants to merge 2 commits into
genomaths:mainfrom
Artur-man:main
Open

replace utils::data with BiocGenerics::data#1
Artur-man wants to merge 2 commits into
genomaths:mainfrom
Artur-man:main

Conversation

@Artur-man
Copy link
Copy Markdown

@Artur-man Artur-man commented May 26, 2026

Recently BiocGenerics::data has been added as a new generic:
Bioconductor/BiocGenerics@93a0f55

This PR fixes the following warning by avoid importing data from BiocGenerics:

> devtools::load_all(".")
ℹ Loading GenomAutomorphism
Warning message:
replacing previous importBiocGenerics::databyutils::datawhen loadingGenomAutomorphism

CC @hpages

@hpages
Copy link
Copy Markdown

hpages commented May 27, 2026

Thanks @Artur-man.

Just a few comments:

  1. The new warning is actually:

    Warning: replacing previous import ‘BiocGenerics::data’ by ‘utils::data’ when loading ‘GenomAutomorphism’
    

    (replace when loading ‘metaMS’ with when loading ‘GenomAutomorphism’)

  2. This new warning can be seen on the Bioconductor daily report for BioC 3.24:

  3. @genomaths The main branch of the GenomAutomorphism repo here on GitHub seems to be lagging significantly behind the devel branch of the GenomAutomorphism repo on git.bioconductor.org. I'd recommend that you resync the two repos before you merge this PR. This might introduce some conflicts with the current PR that will need to be resolved. The other way to go would be to resync the two repos after merging the PR. If there are conflicts they will appear one way or another but maybe the first approach will make it easier to deal with them. See https://contributions.bioconductor.org/git-version-control.html#sync-existing-repositories for how to sync a GitHub repo with a repo on git.bioconductor.org. Thanks for your contribution to Bioconductor.

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