Skip to content

Commit

Permalink
fix: update allListedVars to include levelVars instead of initVars (t…
Browse files Browse the repository at this point in the history
…he latter was causing duplicate items, since it includes a mix of const, aux, and level vars)
  • Loading branch information
chrispcampbell committed May 30, 2024
1 parent 69b7061 commit c0a184c
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions packages/compile/src/model/model.js
Original file line number Diff line number Diff line change
Expand Up @@ -1062,11 +1062,10 @@ function allListedVars() {
if (timeVar) {
vars.push(timeVar)
}
vars.push(...initVars())
vars.push(...auxVars())
// TODO: Also levelVars not covered by initVars?
vars.push(...levelVars())

// Filter out data/lookup variables and variables that are generated/used internally
// Filter out variables that are generated/used internally
const isInternal = v => {
return v.includeInOutput === false
}
Expand Down

0 comments on commit c0a184c

Please sign in to comment.