Support nimble2 with various features by perrydv · Pull Request #212 · nimble-dev/nCompiler

perrydv · 2026-07-16T21:53:56Z

This PR brings in features for supporting a new version of nimble with nimbleModel. These include:

renovated system for managing compile-time information on nClass inheritance in the project_env.
fixed determination of nested types, multiple levels in (A$B$C) when there were TBD types to be resolved.
fleshed out get_interface_ptr feature of the generic interface in C++.
add C++ classes RuntimeFlatView and RuntimeFlatViewGroup to support nimble's values() keyword.
other bug fixes and details

@paciorek I don't think this needs a rebase. OK to merge and then you can rebase your TBB branch (again)?

…bject.

…AbstractTypes

…n project_env$known_nClasses

…e nList's getLength and setLength to base class.

paciorek · 2026-07-17T15:18:14Z

@perrydv I think this has broken nimbleModelmaster branch. Not sure if I need to switch to the branch of nimbleModel with the nimble2 changes?

code <- quote({
tau ~ dunif(0, 100)
mu ~ dnorm(0,1)
for(i in 1:5) {
y[i] ~ dnorm(mu, var = tau)
}
})
inits <- list(tau = 25, mu = 0)
data <- list(y = rnorm(5))
## Manual workflow
mclass <- nimbleModel(code, inits = inits, data = data, returnClass = TRUE)
m <- mclass$new()
cmclass <- nCompile(mclass)
cm <- cmclass$new()

Error in NCint$process_inherit(inheritInfo = inheritInfo, symbolTable = symbolTable, : unused arguments (inheritInfo = inheritInfo, symbolTable = symbolTable, project_env = project_env)

perrydv · 2026-07-17T15:46:32Z

@paciorek have you tried rebuilding nimbleModel with the updated nCompiler installed? There is the issue that calls to nClass in nimbleModel will bake in nClass features from nCompiler, so if nCompiler updates, nimbleModel can break for that reason. Eventually we may want many or all nClasses created in a package like nimbleModel to be created in .onLoad.

paciorek · 2026-07-17T17:40:16Z

@perrydv thanks; that resolved the problem. You've probably told me about this before but I haven't absorbed it...

perrydv · 2026-07-16T21:53:56Z

This PR brings in features for supporting a new version of nimble with nimbleModel. These include:

renovated system for managing compile-time information on nClass inheritance in the project_env.
fixed determination of nested types, multiple levels in (A$B$C) when there were TBD types to be resolved.
fleshed out get_interface_ptr feature of the generic interface in C++.
add C++ classes RuntimeFlatView and RuntimeFlatViewGroup to support nimble's values() keyword.
other bug fixes and details

@paciorek I don't think this needs a rebase. OK to merge and then you can rebase your TBB branch (again)?

…bject.

…AbstractTypes

…n project_env$known_nClasses

…e nList's getLength and setLength to base class.

paciorek · 2026-07-17T15:18:14Z

@perrydv I think this has broken nimbleModelmaster branch. Not sure if I need to switch to the branch of nimbleModel with the nimble2 changes?

code <- quote({
tau ~ dunif(0, 100)
mu ~ dnorm(0,1)
for(i in 1:5) {
y[i] ~ dnorm(mu, var = tau)
}
})
inits <- list(tau = 25, mu = 0)
data <- list(y = rnorm(5))
## Manual workflow
mclass <- nimbleModel(code, inits = inits, data = data, returnClass = TRUE)
m <- mclass$new()
cmclass <- nCompile(mclass)
cm <- cmclass$new()

Error in NCint$process_inherit(inheritInfo = inheritInfo, symbolTable = symbolTable, : unused arguments (inheritInfo = inheritInfo, symbolTable = symbolTable, project_env = project_env)

perrydv · 2026-07-17T15:46:32Z

@paciorek have you tried rebuilding nimbleModel with the updated nCompiler installed? There is the issue that calls to nClass in nimbleModel will bake in nClass features from nCompiler, so if nCompiler updates, nimbleModel can break for that reason. Eventually we may want many or all nClasses created in a package like nimbleModel to be created in .onLoad.

paciorek · 2026-07-17T17:40:16Z

@perrydv thanks; that resolved the problem. You've probably told me about this before but I haven't absorbed it...

perrydv · 2026-07-16T21:53:56Z

This PR brings in features for supporting a new version of nimble with nimbleModel. These include:

renovated system for managing compile-time information on nClass inheritance in the project_env.
fixed determination of nested types, multiple levels in (A$B$C) when there were TBD types to be resolved.
fleshed out get_interface_ptr feature of the generic interface in C++.
add C++ classes RuntimeFlatView and RuntimeFlatViewGroup to support nimble's values() keyword.
other bug fixes and details

@paciorek I don't think this needs a rebase. OK to merge and then you can rebase your TBB branch (again)?

…bject.

…AbstractTypes

…n project_env$known_nClasses

…e nList's getLength and setLength to base class.

paciorek · 2026-07-17T15:18:14Z

@perrydv I think this has broken nimbleModelmaster branch. Not sure if I need to switch to the branch of nimbleModel with the nimble2 changes?

code <- quote({
tau ~ dunif(0, 100)
mu ~ dnorm(0,1)
for(i in 1:5) {
y[i] ~ dnorm(mu, var = tau)
}
})
inits <- list(tau = 25, mu = 0)
data <- list(y = rnorm(5))
## Manual workflow
mclass <- nimbleModel(code, inits = inits, data = data, returnClass = TRUE)
m <- mclass$new()
cmclass <- nCompile(mclass)
cm <- cmclass$new()

Error in NCint$process_inherit(inheritInfo = inheritInfo, symbolTable = symbolTable, : unused arguments (inheritInfo = inheritInfo, symbolTable = symbolTable, project_env = project_env)

perrydv · 2026-07-17T15:46:32Z

@paciorek have you tried rebuilding nimbleModel with the updated nCompiler installed? There is the issue that calls to nClass in nimbleModel will bake in nClass features from nCompiler, so if nCompiler updates, nimbleModel can break for that reason. Eventually we may want many or all nClasses created in a package like nimbleModel to be created in .onLoad.

paciorek · 2026-07-17T17:40:16Z

@perrydv thanks; that resolved the problem. You've probably told me about this before but I haven't absorbed it...

perrydv · 2026-07-16T21:53:56Z

This PR brings in features for supporting a new version of nimble with nimbleModel. These include:

renovated system for managing compile-time information on nClass inheritance in the project_env.
fixed determination of nested types, multiple levels in (A$B$C) when there were TBD types to be resolved.
fleshed out get_interface_ptr feature of the generic interface in C++.
add C++ classes RuntimeFlatView and RuntimeFlatViewGroup to support nimble's values() keyword.
other bug fixes and details

@paciorek I don't think this needs a rebase. OK to merge and then you can rebase your TBB branch (again)?

…bject.

…AbstractTypes

…n project_env$known_nClasses

…e nList's getLength and setLength to base class.

paciorek · 2026-07-17T15:18:14Z

@perrydv I think this has broken nimbleModelmaster branch. Not sure if I need to switch to the branch of nimbleModel with the nimble2 changes?

code <- quote({
tau ~ dunif(0, 100)
mu ~ dnorm(0,1)
for(i in 1:5) {
y[i] ~ dnorm(mu, var = tau)
}
})
inits <- list(tau = 25, mu = 0)
data <- list(y = rnorm(5))
## Manual workflow
mclass <- nimbleModel(code, inits = inits, data = data, returnClass = TRUE)
m <- mclass$new()
cmclass <- nCompile(mclass)
cm <- cmclass$new()

Error in NCint$process_inherit(inheritInfo = inheritInfo, symbolTable = symbolTable, : unused arguments (inheritInfo = inheritInfo, symbolTable = symbolTable, project_env = project_env)

perrydv · 2026-07-17T15:46:32Z

@paciorek have you tried rebuilding nimbleModel with the updated nCompiler installed? There is the issue that calls to nClass in nimbleModel will bake in nClass features from nCompiler, so if nCompiler updates, nimbleModel can break for that reason. Eventually we may want many or all nClasses created in a package like nimbleModel to be created in .onLoad.

paciorek · 2026-07-17T17:40:16Z

@perrydv thanks; that resolved the problem. You've probably told me about this before but I haven't absorbed it...

perrydv · 2026-07-16T21:53:56Z

This PR brings in features for supporting a new version of nimble with nimbleModel. These include:

renovated system for managing compile-time information on nClass inheritance in the project_env.
fixed determination of nested types, multiple levels in (A$B$C) when there were TBD types to be resolved.
fleshed out get_interface_ptr feature of the generic interface in C++.
add C++ classes RuntimeFlatView and RuntimeFlatViewGroup to support nimble's values() keyword.
other bug fixes and details

@paciorek I don't think this needs a rebase. OK to merge and then you can rebase your TBB branch (again)?

…bject.

…AbstractTypes

…n project_env$known_nClasses

…e nList's getLength and setLength to base class.

paciorek · 2026-07-17T15:18:14Z

@perrydv I think this has broken nimbleModelmaster branch. Not sure if I need to switch to the branch of nimbleModel with the nimble2 changes?

code <- quote({
tau ~ dunif(0, 100)
mu ~ dnorm(0,1)
for(i in 1:5) {
y[i] ~ dnorm(mu, var = tau)
}
})
inits <- list(tau = 25, mu = 0)
data <- list(y = rnorm(5))
## Manual workflow
mclass <- nimbleModel(code, inits = inits, data = data, returnClass = TRUE)
m <- mclass$new()
cmclass <- nCompile(mclass)
cm <- cmclass$new()

Error in NCint$process_inherit(inheritInfo = inheritInfo, symbolTable = symbolTable, : unused arguments (inheritInfo = inheritInfo, symbolTable = symbolTable, project_env = project_env)

perrydv · 2026-07-17T15:46:32Z

@paciorek have you tried rebuilding nimbleModel with the updated nCompiler installed? There is the issue that calls to nClass in nimbleModel will bake in nClass features from nCompiler, so if nCompiler updates, nimbleModel can break for that reason. Eventually we may want many or all nClasses created in a package like nimbleModel to be created in .onLoad.

paciorek · 2026-07-17T17:40:16Z

@perrydv thanks; that resolved the problem. You've probably told me about this before but I haven't absorbed it...

perrydv · 2026-07-16T21:53:56Z

This PR brings in features for supporting a new version of nimble with nimbleModel. These include:

renovated system for managing compile-time information on nClass inheritance in the project_env.
fixed determination of nested types, multiple levels in (A$B$C) when there were TBD types to be resolved.
fleshed out get_interface_ptr feature of the generic interface in C++.
add C++ classes RuntimeFlatView and RuntimeFlatViewGroup to support nimble's values() keyword.
other bug fixes and details

@paciorek I don't think this needs a rebase. OK to merge and then you can rebase your TBB branch (again)?

…bject.

…AbstractTypes

…n project_env$known_nClasses

…e nList's getLength and setLength to base class.

paciorek · 2026-07-17T15:18:14Z

@perrydv I think this has broken nimbleModelmaster branch. Not sure if I need to switch to the branch of nimbleModel with the nimble2 changes?

code <- quote({
tau ~ dunif(0, 100)
mu ~ dnorm(0,1)
for(i in 1:5) {
y[i] ~ dnorm(mu, var = tau)
}
})
inits <- list(tau = 25, mu = 0)
data <- list(y = rnorm(5))
## Manual workflow
mclass <- nimbleModel(code, inits = inits, data = data, returnClass = TRUE)
m <- mclass$new()
cmclass <- nCompile(mclass)
cm <- cmclass$new()

Error in NCint$process_inherit(inheritInfo = inheritInfo, symbolTable = symbolTable, : unused arguments (inheritInfo = inheritInfo, symbolTable = symbolTable, project_env = project_env)

perrydv · 2026-07-17T15:46:32Z

@paciorek have you tried rebuilding nimbleModel with the updated nCompiler installed? There is the issue that calls to nClass in nimbleModel will bake in nClass features from nCompiler, so if nCompiler updates, nimbleModel can break for that reason. Eventually we may want many or all nClasses created in a package like nimbleModel to be created in .onLoad.

paciorek · 2026-07-17T17:40:16Z

@perrydv thanks; that resolved the problem. You've probably told me about this before but I haven't absorbed it...

perrydv · 2026-07-16T21:53:56Z

This PR brings in features for supporting a new version of nimble with nimbleModel. These include:

renovated system for managing compile-time information on nClass inheritance in the project_env.
fixed determination of nested types, multiple levels in (A$B$C) when there were TBD types to be resolved.
fleshed out get_interface_ptr feature of the generic interface in C++.
add C++ classes RuntimeFlatView and RuntimeFlatViewGroup to support nimble's values() keyword.
other bug fixes and details

@paciorek I don't think this needs a rebase. OK to merge and then you can rebase your TBB branch (again)?

…bject.

…AbstractTypes

…n project_env$known_nClasses

…e nList's getLength and setLength to base class.

paciorek · 2026-07-17T15:18:14Z

@perrydv I think this has broken nimbleModelmaster branch. Not sure if I need to switch to the branch of nimbleModel with the nimble2 changes?

code <- quote({
tau ~ dunif(0, 100)
mu ~ dnorm(0,1)
for(i in 1:5) {
y[i] ~ dnorm(mu, var = tau)
}
})
inits <- list(tau = 25, mu = 0)
data <- list(y = rnorm(5))
## Manual workflow
mclass <- nimbleModel(code, inits = inits, data = data, returnClass = TRUE)
m <- mclass$new()
cmclass <- nCompile(mclass)
cm <- cmclass$new()

Error in NCint$process_inherit(inheritInfo = inheritInfo, symbolTable = symbolTable, : unused arguments (inheritInfo = inheritInfo, symbolTable = symbolTable, project_env = project_env)

perrydv · 2026-07-17T15:46:32Z

@paciorek have you tried rebuilding nimbleModel with the updated nCompiler installed? There is the issue that calls to nClass in nimbleModel will bake in nClass features from nCompiler, so if nCompiler updates, nimbleModel can break for that reason. Eventually we may want many or all nClasses created in a package like nimbleModel to be created in .onLoad.

paciorek · 2026-07-17T17:40:16Z

@perrydv thanks; that resolved the problem. You've probably told me about this before but I haven't absorbed it...

perrydv · 2026-07-16T21:53:56Z

This PR brings in features for supporting a new version of nimble with nimbleModel. These include:

renovated system for managing compile-time information on nClass inheritance in the project_env.
fixed determination of nested types, multiple levels in (A$B$C) when there were TBD types to be resolved.
fleshed out get_interface_ptr feature of the generic interface in C++.
add C++ classes RuntimeFlatView and RuntimeFlatViewGroup to support nimble's values() keyword.
other bug fixes and details

@paciorek I don't think this needs a rebase. OK to merge and then you can rebase your TBB branch (again)?

…bject.

…AbstractTypes

…n project_env$known_nClasses

…e nList's getLength and setLength to base class.

paciorek · 2026-07-17T15:18:14Z

@perrydv I think this has broken nimbleModelmaster branch. Not sure if I need to switch to the branch of nimbleModel with the nimble2 changes?

code <- quote({
tau ~ dunif(0, 100)
mu ~ dnorm(0,1)
for(i in 1:5) {
y[i] ~ dnorm(mu, var = tau)
}
})
inits <- list(tau = 25, mu = 0)
data <- list(y = rnorm(5))
## Manual workflow
mclass <- nimbleModel(code, inits = inits, data = data, returnClass = TRUE)
m <- mclass$new()
cmclass <- nCompile(mclass)
cm <- cmclass$new()

Error in NCint$process_inherit(inheritInfo = inheritInfo, symbolTable = symbolTable, : unused arguments (inheritInfo = inheritInfo, symbolTable = symbolTable, project_env = project_env)

perrydv · 2026-07-17T15:46:32Z

@paciorek have you tried rebuilding nimbleModel with the updated nCompiler installed? There is the issue that calls to nClass in nimbleModel will bake in nClass features from nCompiler, so if nCompiler updates, nimbleModel can break for that reason. Eventually we may want many or all nClasses created in a package like nimbleModel to be created in .onLoad.

paciorek · 2026-07-17T17:40:16Z

@perrydv thanks; that resolved the problem. You've probably told me about this before but I haven't absorbed it...

perrydv added 11 commits June 16, 2026 16:45

use namespace_env to access nCompiler from Rcpp. Fix value<- return o…
d8991db
…bject.

Add NFinternals$update_code. Do not assume obj$v has known v in label…
5a103d9
…AbstractTypes

Enhance type declarations to allow a call like "env$nctype()"
8e31494

Provide RuntimeFlatView to support values() in nimble2
06e2b06

allow rvalue as input for setting in RuntimeFlatViewGroup
cd2615e

fix bug in insertArg for mix of NULL and non-NULL names
ff5625e

disable a test of obj[["x"]] due to changed intended behavior
737bdcd

renovate how nClass inheritance is processed so inheritInfo is held i…
9e3da92
…n project_env$known_nClasses

clean up some uses of project_env and tests
4f7dba8

complete the get_interface_ptr feature of C++ generic interfaces. Mov…
5c6b152
…e nList's getLength and setLength to base class.

fix problem introduced in last commit in labelAbstractTypes$Literal
0aff688

perrydv merged commit f1416d1 into mainJul 17, 2026
5 checks passed

perrydv deleted the support-nimble2 branch July 17, 2026 13:48