Conditionally compile cvodes
@seantalts is already working on this.
Since Jul 31, 2017.
- Dominant language
- C++
- Stars
- 839
- Forks
- 220
- Avg merge
- 2d 4h
- Merged PRs (30d)
- 14
Description
Summary:
Most Stan programs don't require the cvodes library but we compile it for every model anyway right now.
Description:
I think we should figure out how much time we could save during model compilation by only compiling and linking against cvodes if its needed. I'm assuming specifying the cvodes library as something to include headers from and link against is basically free and that all of the cost would come from just compiling the library. If this is more than ~10% of model compilation time for simple models it might make sense to figure out how to conditionally compile.
I'm creating this issue to track this idea and the results of the above experiment even if we end up deciding it's not worth it.
Current Version:
v2.16.0
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.