Skip to content

Restore previous behavior of model name arguments#1492

Merged
WardBrian merged 3 commits intomasterfrom
fix/1491-stancjs-model-name
Jan 21, 2025
Merged

Restore previous behavior of model name arguments#1492
WardBrian merged 3 commits intomasterfrom
fix/1491-stancjs-model-name

Conversation

@WardBrian
Copy link
Copy Markdown
Member

Submission Checklist

  • Run unit tests
  • Documentation
    • If a user-facing facing change was made, the documentation PR is here:
    • OR, no user-facing changes were made

Release notes

Fixes #1491 where repeated calls to stanc.js would re-use the same model namespace in the generated C++

Copyright and Licensing

By submitting this pull request, the copyright holder is agreeing to
license the submitted work under the BSD 3-clause license (https://opensource.org/licenses/BSD-3-Clause)

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 21, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 90.46%. Comparing base (d13e71e) to head (b8fd154).
Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1492      +/-   ##
==========================================
+ Coverage   90.45%   90.46%   +0.01%     
==========================================
  Files          66       66              
  Lines       10706    10707       +1     
==========================================
+ Hits         9684     9686       +2     
+ Misses       1022     1021       -1     
Files with missing lines Coverage Δ
src/driver/Entry.ml 93.75% <100.00%> (+1.34%) ⬆️
src/stanc/stanc.ml 86.66% <100.00%> (ø)

@WardBrian WardBrian merged commit 1b601d5 into master Jan 21, 2025
@WardBrian WardBrian deleted the fix/1491-stancjs-model-name branch January 21, 2025 17:56
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.

[BUG] stanc.js persisting model name & namespace between calls

2 participants