Skip to content

enhance compiler with ability to handle nCompiler namespace resolution for more flexibility for dependent packages #111

@paciorek

Description

@paciorek

As discussed in issue #110, the use of importFrom(nCompiler, nClass, nFunction, nCompile) in NAMESPACE or the use of nCompiler::nCompile (etc.) without any import in NAMESPACE does not work. This fails if a user were to use nCompiler operators without use of nCompiler::. This would happen if a user used rep, e.g., not realizing that this is changed to nRep or if they use nCompiler operators in models (e.g., dinvgamma).

This is documented extensively via dependent_package_tests/test-packaging.R, with expect_error showing how uncompiled execution fails to find nRep (the operator used in the nClass method in the tests).

@perrydv and I discussed enhancing nCompile to inject nCompiler:: as needed for R execution and stripping out nCompiler:: during compilation. This seems like it would be a good idea and reasonably straightforward.

I think there is a good argument for leaving this for a future release.

Metadata

Metadata

Assignees

No one assigned

    Labels

    enhancementNew feature or request

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions