Skip to content
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

CompatHelper: bump compat for AbstractMCMC to 5, (keep existing compat) #551

Merged

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Oct 26, 2023

This pull request changes the compat entry for the AbstractMCMC package from 2, 3.0, 4 to 2, 3.0, 4, 5.
This keeps the compat entries for earlier versions.

Note: I have not tested your package with this new compat entry.
It is your responsibility to make sure that your package tests pass before you merge this pull request.

Depends on JuliaRegistries/General#94177 (comment)

Superced #554 Close #554

@devmotion devmotion force-pushed the compathelper/new_version/2023-10-26-00-09-06-954-01573151329 branch from 428c133 to 8d285aa Compare October 26, 2023 00:09
@codecov
Copy link

codecov bot commented Oct 26, 2023

Codecov Report

Attention: 4 lines in your changes are missing coverage. Please review.

Comparison is base (04b03cd) 80.22% compared to head (04616d1) 80.25%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #551      +/-   ##
==========================================
+ Coverage   80.22%   80.25%   +0.03%     
==========================================
  Files          25       25              
  Lines        3156     3161       +5     
==========================================
+ Hits         2532     2537       +5     
  Misses        624      624              
Files Coverage Δ
src/sampler.jl 94.11% <100.00%> (+4.11%) ⬆️
ext/DynamicPPLMCMCChainsExt.jl 81.81% <0.00%> (-18.19%) ⬇️

... and 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

…ep existing compat) (#552)

Co-authored-by: CompatHelper Julia <[email protected]>
@devmotion
Copy link
Member

devmotion commented Oct 27, 2023

This requires more changes: init_params kwarg was renamed to initial_params in AbstractMCMC 5, and we have now initial_state in AbstractMCMC which should replace or be combined with the resume_from kwarg.

ext/DynamicPPLMCMCChainsExt.jl Outdated Show resolved Hide resolved
src/sampler.jl Outdated Show resolved Hide resolved
src/sampler.jl Outdated Show resolved Hide resolved
src/sampler.jl Outdated Show resolved Hide resolved
devmotion and others added 4 commits October 27, 2023 15:17
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
…ep existing compat) (#553)

* Fix for `rand` + replace overloads of `rand` with `rand_prior_true` for testing models (#541)

* preserve context from model in `rand`

* replace rand overloads in TestUtils with definitions of
rand_prior_true so we can properly test rand

* removed NamedTuple from signature of TestUtils.rand_prior_true

* updated references to previous overloads of rand to now use rand_prior_true

* test rand for DEMO_MODELS

* formatting

* fixed tests for rand for DEMO_MODELS

* fixed linkning tests

* added missing impl of rand_prior_true for demo_static_transformation

* formatting

* fixed rand_prior_true for demo_static_transformation

* bump minor version as this will be breaking

* bump patch version

* fixed old usage of rand

* Update test/varinfo.jl

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* fixed another usage of rand

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Remove `tonamedtuple` (#547)

* Remove dependencies to `tonamedtuple`

* Remove `tonamedtuple`s

* Minor version bump

---------

Co-authored-by: Hong Ge <[email protected]>

* CompatHelper: bump compat for AbstractMCMC to 5 for package test, (keep existing compat)

---------

Co-authored-by: Tor Erlend Fjelde <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Xianda Sun <[email protected]>
Co-authored-by: Hong Ge <[email protected]>
Co-authored-by: CompatHelper Julia <[email protected]>
@sunxd3
Copy link
Collaborator

sunxd3 commented Nov 8, 2023

@yebai @devmotion It looks like that this is blocked because AbstractPPL restricts AbstractMCMC to version 4, TuringLang/AbstractPPL.jl#88

@coveralls
Copy link

coveralls commented Nov 8, 2023

Pull Request Test Coverage Report for Build 6812010138

Warning: This coverage report may be inaccurate.

We've detected an issue with your CI configuration that might affect the accuracy of this pull request's coverage report.
To ensure accuracy in future PRs, please see these guidelines.
A quick fix for this PR: rebase it; your next report should be accurate.

  • 10 of 14 (71.43%) changed or added relevant lines in 2 files are covered.
  • 4 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.03%) to 80.259%

Changes Missing Coverage Covered Lines Changed/Added Lines %
ext/DynamicPPLMCMCChainsExt.jl 0 4 0.0%
Files with Coverage Reduction New Missed Lines %
src/utils.jl 4 82.28%
Totals Coverage Status
Change from base Build 6658534141: 0.03%
Covered Lines: 2537
Relevant Lines: 3161

💛 - Coveralls

@yebai
Copy link
Member

yebai commented Nov 8, 2023

@sunxd3 can you take a look at whether we can replace resume_from with initial_state before we merge this PR?

src/sampler.jl Outdated Show resolved Hide resolved
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
@yebai yebai enabled auto-merge November 9, 2023 12:49
@yebai yebai added this pull request to the merge queue Nov 9, 2023
Merged via the queue into master with commit bda441b Nov 9, 2023
12 of 14 checks passed
@yebai yebai deleted the compathelper/new_version/2023-10-26-00-09-06-954-01573151329 branch November 9, 2023 13:27
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.

4 participants