-
-
Notifications
You must be signed in to change notification settings - Fork 95
Fix includes for Stan Math /prim flatten #792
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.
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
Conversation
This PR should be merged together with #792 and stan-dev/stan#2876 |
Jenkins Console Log Machine informationProductName: Mac OS X ProductVersion: 10.11.6 BuildVersion: 15G22010CPU: G++: Clang: |
Jenkins Console Log Machine informationProductName: Mac OS X ProductVersion: 10.11.6 BuildVersion: 15G22010CPU: G++: Clang: |
I hope it's fine for me to approve this in cmdstan. It'll have to wait for stan-dev/math#1606 to be merged. |
You have the permissions and understand the changes made, so I think its totally fine. Thanks by the way! |
Jenkins Console Log Machine informationProductName: Mac OS X ProductVersion: 10.11.6 BuildVersion: 15G22010CPU: G++: Clang: |
Submisison Checklist
./runCmdStanTests.py src/test
Summary:
stan-dev/math#1606 will remove mat, arr and scal from the Stan Math's /prim folder. This PR fixes one include in the Cmdstan repo.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company):
Rok Češnovar
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses: