-
Notifications
You must be signed in to change notification settings - Fork 32
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
[Merged by Bors] - Re-opening of #310 #330
Conversation
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…/DynamicPPL.jl into tor/allow-non-array-variables
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Seems like now that Bijectors.jl isn't holding ChainRules back, we're finally running on ChainRules@1 and things are not working correctly 😕 |
Do you recognize this bug @devmotion ? I haven't looked too much into CRC1 yet. |
Seems to involve generators, so maybe similar to the issues that we saw with CRC and Zygote in TuringLang/DistributionsAD.jl#198 and TuringLang/DistributionsAD.jl#197 🤷 |
bors try |
Tests will only pass when the relevant Zygote PR with the fix is merged and released. |
tryMerge conflict. |
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…cPPL.jl into tor/submodels-hotfixes
@torfjelde let's also try to merge this PR before making a new release. |
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
bors r+ |
Pull request successfully merged into master. Build succeeded: |
We forgot to merge the #310 before merging #295.