Transfer some test utility function into DynamicPPL #2049
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
There is an error with the Tracker AD backend. PDMatDistribution AD: Error During Test at /home/runner/work/Turing.jl/Turing.jl/test/essential/ad.jl:98
Got exception outside of a @test
MethodError: no method matching Float64(::Tracker.TrackedReal{Float64})
Closest candidates are:
(::Type{T})(::Real, ::RoundingMode) where T<:AbstractFloat at /opt/hostedtoolcache/julia/1.7.3/x64/share/julia/base/rounding.jl:200
(::Type{T})(::T) where T<:Number at /opt/hostedtoolcache/julia/1.7.3/x64/share/julia/base/boot.jl:770
(::Type{T})(::AbstractChar) where T<:Union{AbstractChar, Number} at /opt/hostedtoolcache/julia/1.7.3/x64/share/julia/base/char.jl:50
...@devmotion do you know any potential cause for this error? It is worth noting this error is not related to this PR but rather relaxing the compat constraint to use the most recent Bijectors as #2018 |
yebai
merged commit c7b0a30
into
compathelper/new_version/2023-06-20-00-28-18-752-01921173878
0 of 11 checks passed
yebai
added a commit
that referenced
this pull request
Jul 27, 2023
* CompatHelper: bump compat for Bijectors to 0.13, (keep existing compat) * Update Project.toml * Replacement for #2039 (#2040) * Fix testset for external samplers * Update abstractmcmc.jl * Update test/contrib/inference/abstractmcmc.jl Co-authored-by: Tor Erlend Fjelde <tor.erlend95@gmail.com> * Update test/contrib/inference/abstractmcmc.jl Co-authored-by: Tor Erlend Fjelde <tor.erlend95@gmail.com> * Update FillArrays compat to 1.4.1 (#2035) * Update FillArrays compat to 1.4.0 * Update test compat * Try to enable ReverseDiff tests * Update Project.toml * Update Project.toml * Bump version * Revert dependencies on FillArrays (#2042) * Update Project.toml * Update Project.toml * Fix redundant definition of `getstats` (#2044) * Fix redundant definition of `getstats` * Update Inference.jl * Revert "Update Inference.jl" This reverts commit e4f51c2. * Bump version --------- Co-authored-by: Hong Ge <3279477+yebai@users.noreply.github.com> * Transfer some test utility function into DynamicPPL (#2049) * Update OptimInterface.jl * Only run optimisation tests in numerical stage. * fix function lookup after moving functions --------- Co-authored-by: Xianda Sun <sunxdt@gmail.com> * Move Optim support to extension (#2051) * Move Optim support to extension * More imports * Update Project.toml --------- Co-authored-by: Hong Ge <3279477+yebai@users.noreply.github.com> --------- Co-authored-by: CompatHelper Julia <compathelper_noreply@julialang.org> Co-authored-by: haris organtzidis <organtzh@gmail.com> Co-authored-by: Tor Erlend Fjelde <tor.erlend95@gmail.com> Co-authored-by: David Widmann <devmotion@users.noreply.github.com> Co-authored-by: Xianda Sun <sunxdt@gmail.com> Co-authored-by: Cameron Pfiffer <cpfiffer@gmail.com>
yebai
added a commit
that referenced
this pull request
Aug 16, 2023
…t) (#2018) * CompatHelper: bump compat for Bijectors to 0.13, (keep existing compat) * Update Project.toml * Replacement for #2039 (#2040) * Fix testset for external samplers * Update abstractmcmc.jl * Update test/contrib/inference/abstractmcmc.jl Co-authored-by: Tor Erlend Fjelde <tor.erlend95@gmail.com> * Update test/contrib/inference/abstractmcmc.jl Co-authored-by: Tor Erlend Fjelde <tor.erlend95@gmail.com> * Transfer some test utility function into DynamicPPL (#2049) * Update OptimInterface.jl * Only run optimisation tests in numerical stage. * fix function lookup after moving functions --------- Co-authored-by: Xianda Sun <sunxdt@gmail.com> * Remove tracker tests. * Update Project.toml * Update Project.toml * Update Project.toml --------- Co-authored-by: CompatHelper Julia <compathelper_noreply@julialang.org> Co-authored-by: Hong Ge <3279477+yebai@users.noreply.github.com> Co-authored-by: haris organtzidis <organtzh@gmail.com> Co-authored-by: Tor Erlend Fjelde <tor.erlend95@gmail.com> Co-authored-by: Xianda Sun <sunxdt@gmail.com> Co-authored-by: Cameron Pfiffer <cpfiffer@gmail.com>
yebai
added a commit
that referenced
this pull request
Aug 16, 2023
* first draft * abstractcontext + tests * bug * externalsampler() in tests * Name Tupple problems * moving stuff to DynamicPPL RP * using new DynamicPPL PR * mistakenly removed line * specific constructors * no StaticMH RWMH * Bump bijectors compat (#2052) * CompatHelper: bump compat for Bijectors to 0.13, (keep existing compat) * Update Project.toml * Replacement for #2039 (#2040) * Fix testset for external samplers * Update abstractmcmc.jl * Update test/contrib/inference/abstractmcmc.jl Co-authored-by: Tor Erlend Fjelde <tor.erlend95@gmail.com> * Update test/contrib/inference/abstractmcmc.jl Co-authored-by: Tor Erlend Fjelde <tor.erlend95@gmail.com> * Update FillArrays compat to 1.4.1 (#2035) * Update FillArrays compat to 1.4.0 * Update test compat * Try to enable ReverseDiff tests * Update Project.toml * Update Project.toml * Bump version * Revert dependencies on FillArrays (#2042) * Update Project.toml * Update Project.toml * Fix redundant definition of `getstats` (#2044) * Fix redundant definition of `getstats` * Update Inference.jl * Revert "Update Inference.jl" This reverts commit e4f51c2. * Bump version --------- Co-authored-by: Hong Ge <3279477+yebai@users.noreply.github.com> * Transfer some test utility function into DynamicPPL (#2049) * Update OptimInterface.jl * Only run optimisation tests in numerical stage. * fix function lookup after moving functions --------- Co-authored-by: Xianda Sun <sunxdt@gmail.com> * Move Optim support to extension (#2051) * Move Optim support to extension * More imports * Update Project.toml --------- Co-authored-by: Hong Ge <3279477+yebai@users.noreply.github.com> --------- Co-authored-by: CompatHelper Julia <compathelper_noreply@julialang.org> Co-authored-by: haris organtzidis <organtzh@gmail.com> Co-authored-by: Tor Erlend Fjelde <tor.erlend95@gmail.com> Co-authored-by: David Widmann <devmotion@users.noreply.github.com> Co-authored-by: Xianda Sun <sunxdt@gmail.com> Co-authored-by: Cameron Pfiffer <cpfiffer@gmail.com> * Bugfixes. * Add TODO. * Update mh.jl * Update Inference.jl * Removed obsolete exports. * removed unnecessary import of extract_priors * added missing ) in MH tests * fixed incorrect referneces to AdvancedMH in tests * improve ESLogDensityFunction * remove hardcoding of SimpleVarInfo * added fixme comment * minor style changes * fixed issues with MH with RandomWalkProposal being used as an external sampler * fixed accidental typo * move definitions of unflatten for NamedTuple * improved TODO * Update Project.toml --------- Co-authored-by: Hong Ge <3279477+yebai@users.noreply.github.com> Co-authored-by: CompatHelper Julia <compathelper_noreply@julialang.org> Co-authored-by: haris organtzidis <organtzh@gmail.com> Co-authored-by: Tor Erlend Fjelde <tor.erlend95@gmail.com> Co-authored-by: David Widmann <devmotion@users.noreply.github.com> Co-authored-by: Xianda Sun <sunxdt@gmail.com> Co-authored-by: Cameron Pfiffer <cpfiffer@gmail.com> Co-authored-by: Hong Ge <hg344@cam.ac.uk>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Companion PR of TuringLang/DynamicPPL.jl#503