Skip to content

Runic formatter#243

Merged
Yue-Zhengyuan merged 7 commits intoQuantumKitHub:masterfrom
Yue-Zhengyuan:runic
Aug 14, 2025
Merged

Runic formatter#243
Yue-Zhengyuan merged 7 commits intoQuantumKitHub:masterfrom
Yue-Zhengyuan:runic

Conversation

@Yue-Zhengyuan
Copy link
Member

@Yue-Zhengyuan Yue-Zhengyuan commented Aug 13, 2025

@codecov
Copy link

codecov bot commented Aug 13, 2025

Codecov Report

❌ Patch coverage is 78.89447% with 42 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/operators/infinitepepo.jl 41.66% 7 Missing ⚠️
src/environments/ctmrg_environments.jl 68.75% 5 Missing ⚠️
src/algorithms/contractions/ctmrg_contractions.jl 42.85% 4 Missing ⚠️
src/states/infinitepartitionfunction.jl 33.33% 4 Missing ⚠️
src/states/infinitepeps.jl 42.85% 4 Missing ⚠️
src/algorithms/contractions/localoperator.jl 40.00% 3 Missing ⚠️
...rithms/optimization/fixed_point_differentiation.jl 70.00% 3 Missing ⚠️
src/algorithms/ctmrg/projectors.jl 60.00% 2 Missing ⚠️
src/states/infiniteweightpeps.jl 75.00% 2 Missing ⚠️
src/utility/svd.jl 90.00% 2 Missing ⚠️
... and 6 more
Files with missing lines Coverage Δ
src/Defaults.jl 85.71% <100.00%> (ø)
src/algorithms/contractions/bondenv/als_solve.jl 100.00% <ø> (ø)
src/algorithms/contractions/bondenv/benv_ctm.jl 100.00% <100.00%> (ø)
src/algorithms/contractions/bondenv/gaugefix.jl 100.00% <ø> (ø)
src/algorithms/contractions/vumps_contractions.jl 84.84% <100.00%> (ø)
src/algorithms/correlators.jl 100.00% <100.00%> (ø)
src/algorithms/ctmrg/ctmrg.jl 90.47% <100.00%> (ø)
src/algorithms/ctmrg/gaugefix.jl 97.19% <100.00%> (ø)
src/algorithms/ctmrg/sequential.jl 98.41% <100.00%> (ø)
src/algorithms/ctmrg/simultaneous.jl 100.00% <100.00%> (ø)
... and 32 more
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Yue-Zhengyuan Yue-Zhengyuan marked this pull request as ready for review August 13, 2025 07:07
@Yue-Zhengyuan Yue-Zhengyuan requested a review from lkdvos August 13, 2025 08:00
Copy link
Member

@lkdvos lkdvos left a comment

Choose a reason for hiding this comment

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

I manually altered some long argument lists, given that this is a bit more subjective, feel free to disagree and we can remove this again. Otherwise ok for me!

@Yue-Zhengyuan
Copy link
Member Author

I'm OK with the manual adjustments. IMO it's not always the best to put each single argument on a new line, and I'm happy that Runic gives us this freedom.

@Yue-Zhengyuan Yue-Zhengyuan merged commit 0ff55b6 into QuantumKitHub:master Aug 14, 2025
43 of 45 checks passed
@Yue-Zhengyuan Yue-Zhengyuan deleted the runic branch August 26, 2025 01:02
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