Skip to content

drasil-code: Auto-register variables defined in Mods.#4659

Draft
balacij wants to merge 1 commit intomainfrom
autoRegModsVars
Draft

drasil-code: Auto-register variables defined in Mods.#4659
balacij wants to merge 1 commit intomainfrom
autoRegModsVars

Conversation

@balacij
Copy link
Collaborator

@balacij balacij commented Jan 22, 2026

This work only affects GlassBR.

This won't pass the CI yet because two code-only variables (interpY and interpZ) appear in the SRS, but this code removes them from the scope of the SRS-generator's ChunkDB! Work will be necessary before this can be merged that: (a) removes them from the SRS and (b) hooks them through the Choices about how the ICO problem definition should be changed. This way, the two code-only variables only appear in the ChunkDB associated with the code-generator.

This work only affects GlassBR.

This won't pass the CI yet because two _code-only_ variables (interpY
and interpZ) appear in the SRS! Work will be necessary before this
can be merged that: (a) removes them from the SRS and (b) hooks them
through the `Choices` about how the ICO problem definition should be
changed.
@balacij
Copy link
Collaborator Author

balacij commented Jan 23, 2026

This PR could be unblocked if #4664 is accepted.

Copy link
Owner

@JacquesCarette JacquesCarette left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems fine to me.

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