Skip to content

Comments

Update deprecated array syntax#210

Merged
koenderks merged 1 commit intokoenderks:developmentfrom
andrjohns:rstan2.36-compat
Mar 16, 2025
Merged

Update deprecated array syntax#210
koenderks merged 1 commit intokoenderks:developmentfrom
andrjohns:rstan2.36-compat

Conversation

@andrjohns
Copy link
Contributor

This PR updates the deprecated array syntax in your Stan models which will break installation under future RStan versions. Let me know if you have any questions, thanks!

@codecov
Copy link

codecov bot commented Mar 16, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 64.00%. Comparing base (8385053) to head (0a77ea4).
Report is 7 commits behind head on development.

Additional details and impacted files
@@             Coverage Diff              @@
##           development     #210   +/-   ##
============================================
  Coverage        64.00%   64.00%           
============================================
  Files               21       21           
  Lines             6267     6267           
============================================
  Hits              4011     4011           
  Misses            2256     2256           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@koenderks
Copy link
Owner

Hi @andrjohns,

Thanks for the update. From when on will this cause problems with the current package on CRAN? In other words, do I need to update the CRAN version asap?

@andrjohns
Copy link
Contributor Author

No worries! It won't be an issue for several months (at least), so there's no great urgency

@koenderks koenderks merged commit 65fc223 into koenderks:development Mar 16, 2025
2 checks passed
@koenderks
Copy link
Owner

OK, thanks again for the update!

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