Skip to content

Use MPSKit's logging macros and adapt_solver - #445

Merged
leburgel merged 1 commit into
mainfrom
lb/mpskit014_logging
Oct 14, 2026
Merged

leburgel merged 1 commit into
mainfrom
lb/mpskit014_logging

Conversation

@leburgel

Copy link
Copy Markdown
Member

MPSKit 0.14 allows LoggingExtras 1.2, which deprecates @infov, @warnv and withlevel(; verbosity). MPSKit 0.14 also deprecates updatetol in favour of adapt_solver. Under Pkg.test, these deprecations print 136 and 65 warnings respectively in the Julia 1 ubuntu jobs of run 38291840013.

We now log through MPSKit's with_verbosity and its @log_* macros, and drop the LoggingExtras dependency. CTMRG verbosity levels are unchanged. The BP init message and the simple-update header and timing lines move from verbosity 1 to 2.

🤖 Generated with Claude Code

@leburgel

Copy link
Copy Markdown
Member Author

Also force merging this one, since it's approved and test failures are resolved on main.

@leburgel
leburgel merged commit e57a0bc into main Oct 14, 2026
69 of 77 checks passed
@leburgel
leburgel deleted the lb/mpskit014_logging branch October 14, 2026 13:50
@codecov

codecov Bot commented Oct 14, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 83.33333% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/algorithms/optimization/peps_optimization.jl 0.00% 3 Missing ⚠️
src/algorithms/bp/beliefpropagation.jl 80.00% 1 Missing ⚠️
Files with missing lines Coverage Δ
src/PEPSKit.jl 100.00% <ø> (ø)
src/algorithms/ctmrg/ctmrg.jl 80.68% <100.00%> (ø)
src/algorithms/time_evolution/simpleupdate.jl 96.35% <100.00%> (ø)
src/algorithms/bp/beliefpropagation.jl 93.93% <80.00%> (ø)
src/algorithms/optimization/peps_optimization.jl 35.35% <0.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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