Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Use of size() causes compilation errors #193

Closed
seabbs opened this issue Feb 5, 2025 · 2 comments · Fixed by #194
Closed

Use of size() causes compilation errors #193

seabbs opened this issue Feb 5, 2025 · 2 comments · Fixed by #194
Labels
bug Something isn't working

Comments

@seabbs
Copy link
Contributor

seabbs commented Feb 5, 2025

As reported by @sbfnk over in epinowcast/epidist#504 the use of size() causes compilation errors when both rstan and cmdstanr are present on a system (with an unknown additional dependency on versions) for mac. We can resolve this issue by switching out to num_elements() which doesn't have this issue as has been to shown to work elsewhere (i.e. in EpiNow2).
@seabbs seabbs added the bug Something isn't working label Feb 5, 2025
@sbfnk
Copy link
Contributor

sbfnk commented Feb 5, 2025 •
edited
Loading

I don't think in EpiNow2 but there was some previous discussion (and the same issue) at #181

@seabbs seabbs mentioned this issue Feb 5, 2025
Merged
9 tasks
@sbfnk
Copy link
Contributor

sbfnk commented Feb 5, 2025

Re-reading this I'm no longer convinced my proposed solution will work.

seabbs added a commit that referenced this issue Feb 5, 2025
@seabbs
Issue #193: Swap to num_elements from size (#194)
cb9017d 
* replace size with num_elements as reported by @sbfnk

* add a news file update
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Issue #193: Swap to num_elements from size epinowcast/primarycensored
2 participants
@sbfnk @seabbs