Skip to content

Conversation

@yebai
Copy link
Member

@yebai yebai commented Jul 18, 2023

Revert #2014 which was only introduced to restrict FillArrays version.

@yebai yebai requested a review from devmotion July 18, 2023 10:22
Copy link
Member

@devmotion devmotion left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, given that tests pass.

@github-actions
Copy link
Contributor

Pull Request Test Coverage Report for Build 5586219718

  • 0 of 0 changed or added relevant lines in 0 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 0.0%

Totals Coverage Status
Change from base Build 5579283680: 0.0%
Covered Lines: 0
Relevant Lines: 1450

💛 - Coveralls

@codecov
Copy link

codecov bot commented Jul 18, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (e1d3fef) 0.00% compared to head (e4d5a7c) 0.00%.

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #2042   +/-   ##
======================================
  Coverage    0.00%   0.00%           
======================================
  Files          22      22           
  Lines        1450    1450           
======================================
  Misses       1450    1450           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@yebai yebai merged commit 6d8bb40 into master Jul 18, 2023
@yebai yebai deleted the yebai-patch-1 branch July 18, 2023 14:23
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
* 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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants