Skip to content

ci: update tests, JuliaFormatter v2#706

Merged
haakon-e merged 2 commits into
aj/julia_v12from
he/ci-update-tests-juliaformatter
Mar 18, 2026
Merged

ci: update tests, JuliaFormatter v2#706
haakon-e merged 2 commits into
aj/julia_v12from
he/ci-update-tests-juliaformatter

Conversation

@haakon-e
Copy link
Copy Markdown
Member

@haakon-e haakon-e commented Mar 18, 2026

This pull request introduces improvements to the performance testing suite, focusing on enhanced allocation profiling and minor test threshold adjustments. It also updates the JuliaFormatter workflow to use a newer version and adds a missing dependency. The most important changes are grouped below:

Performance Testing Enhancements:

  • Added in-depth allocation profiling using Profile.Allocs in bench_press to provide detailed diagnostics when allocation limits are exceeded, including stacktraces and summaries of the largest allocations. This will help identify and address performance regressions more effectively. [1] [2]
  • Increased the allocation threshold for P3.integrate from 3,000 to 3,300, and for CMN.conv_q_vap_to_q_lcl_icl_MM2015 from 70 to 75. [1] [2]

Dependency and Workflow Updates:

  • Added the Profile package to test/Project.toml to ensure allocation profiling tools are available during testing.
  • Updated the JuliaFormatter GitHub Actions workflow to use version 2 of the formatter

trontrytel and others added 2 commits March 18, 2026 11:46
- in performance_tests, print allocations if they
exceed the threshold (usually 0)
- update to JuliaFormatter v2
@haakon-e
Copy link
Copy Markdown
Member Author

haakon-e commented Mar 18, 2026

@haakon-e haakon-e merged commit 92e26aa into aj/julia_v12 Mar 18, 2026
8 of 10 checks passed
@haakon-e haakon-e deleted the he/ci-update-tests-juliaformatter branch March 18, 2026 20:52
@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 18, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
⚠️ Please upload report for BASE (aj/julia_v12@132a5d1). Learn more about missing BASE report.

Additional details and impacted files
@@               Coverage Diff               @@
##             aj/julia_v12     #706   +/-   ##
===============================================
  Coverage                ?   94.98%           
===============================================
  Files                   ?       53           
  Lines                   ?     2371           
  Branches                ?        0           
===============================================
  Hits                    ?     2252           
  Misses                  ?      119           
  Partials                ?        0           
Components Coverage Δ
src 96.04% <0.00%> (?)
ext 69.47% <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