Skip to content

Need to look into exports of toJSON #266

Open

Description

@bobular

When using plot.data in study.wrangler, we get this warning (harmless but annoying):

Warning message:
There was 1 warning in `mutate()`.
ℹ In argument: `bin_width_computed = if_else(...)`.
ℹ In row 1.
Caused by warning:
! replacing previous import ‘veupathUtils::toJSON’ by ‘jsonlite::toJSON’ when loading ‘plot.data’ 

Can probably be fixed by not exporting?

It might just be this line

exportMethods(toJSON)

But I can't find the roxygen2 directive that generated it.

Maybe we just need to run devtools::document() again?

Activity

  1. asizemore commented on Mar 11, 2025

    @asizemore
    Member

    Very possible, it's been awhile. Not sure why toJSON is being exported by plot.data. It definitely should be by veupathUtils, but off the top of my head i dont think the plots use plot.data::toJSON in practice

  2. linked a pull request that will close this issueaddress toJSON issues, hopefully #267on Dec 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions