-
-
Notifications
You must be signed in to change notification settings - Fork 190
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
Move constraint transforms to their own folder #3039
Move constraint transforms to their own folder #3039
Conversation
@WardBrian I'm getting this error in the windows header checks ./stan/math/prim/prob/wiener5_lpdf.hpp:642:17: error: no member named 'apply' in namespace 'stan::math' As far as I can tell none of the files I touched are used here or in the includes for the wiener5_lpdf file. Is there something I'm missing here? |
You just need to add This does raise a related issue where downstream implementations might break if they're using |
@serban-nicusor-toptal can you restart the failed tests? It's something about failed to download ghostscript. |
Should we still have the old files and just have them include the new files? We can remove them from prim.hpp / rev.hpp and only include the new ones. But that would make this backwards compatible |
I'm ok with that. I just wanted to make sure that this works without those files first. I'm getting errors in runTests.py with that I don't know how to fix. make: *** [<builtin>: test/unit/math/mix/fun_4_test.o] Terminated
make: *** [<builtin>: test/unit/math/mix/fun_6_test.o] Terminated
make: *** [<builtin>: test/unit/math/mix/fun_22_test.o] Terminated
make: *** [<builtin>: test/unit/math/mix/fun_7_test.o] Terminated |
Isn't the idea that the next Math release will have a major version bump anyway? |
Yes though having the files would just make the jump a little easier. Though for 5.0 I think it's also fine not to include them |
@SteveBronder I think this will now pass all tests. Once it does please take a look |
Jenkins Console Log Machine informationNo LSB modules are available. Distributor ID: Ubuntu Description: Ubuntu 20.04.3 LTS Release: 20.04 Codename: focalCPU: G++: Clang: |
@SteveBronder, yes, absolutely! (to the 5.0 branch) |
Oh, @syclik the tests only happen if we are merging to develop. So let's only move branches over to be merged to the 5.0 branch once all tests pass and we approve the PR |
Jenkins Console Log Machine informationNo LSB modules are available. Distributor ID: Ubuntu Description: Ubuntu 20.04.3 LTS Release: 20.04 Codename: focalCPU: G++: Clang: |
…constraint-transforms-to-separate-folder
closes #3027
I moved the constraint transforms and associated "free" files to
constraint
folders.@SteveBronder not sure if I got everything here. Please double check.