Skip to content

changes supporting or noticed due to work on nimbleModel#190

Merged
perrydv merged 1 commit into
mainfrom
support-nimbleModel
Jun 1, 2026
Merged

changes supporting or noticed due to work on nimbleModel#190
perrydv merged 1 commit into
mainfrom
support-nimbleModel

Conversation

@perrydv
Copy link
Copy Markdown
Contributor

@perrydv perrydv commented Jun 1, 2026

This has some mostly minor changes arising from work on nimbleModel.

  • Error trap whether a user writing their own nClass initialize method has included super$initialize(...), which they need to.
  • Fix a bug in return of full interface objects when a user-written initialize takes arguments before the CppObj argument (that latter of which should be captured in ...)
  • Fix bug in recent change to generateCpp handler for Assign that accidentally avoided use of flex_() on the LHS.
  • Support set_all_values from a list for nList objects with flexibility for whether the input list is compiled or uncompiled and whether each element is compiled or uncompiled.
  • Tests for much of the above, although not as exhaustive as could be.

I am entering this PR before running tests, which I will go do next.

@perrydv
Copy link
Copy Markdown
Contributor Author

perrydv commented Jun 1, 2026

There is a test failure that appears unrelated and was noted in #192 . I will proceed with merging this PR.

@perrydv perrydv merged commit a3dfb7c into main Jun 1, 2026
4 of 5 checks passed
@perrydv perrydv deleted the support-nimbleModel branch June 1, 2026 21:32
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.

1 participant