Skip to content

Conversation

@JDBetteridge
Copy link
Contributor

No description provided.

@review-notebook-app
Copy link

Check out this pull request on  ReviewNB

See visual diffs & provide feedback on Jupyter Notebooks.


Powered by ReviewNB

@JDBetteridge JDBetteridge force-pushed the JDBetteridge/ruff_manual_changes_part2 branch 6 times, most recently from 8a9153a to 29b228e Compare January 3, 2026 01:25
@codecov
Copy link

codecov bot commented Jan 3, 2026

Codecov Report

❌ Patch coverage is 66.95005% with 311 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.96%. Comparing base (74243a7) to head (540a43f).
⚠️ Report is 6 commits behind head on JDBetteridge/ruff_manual_changes.

Files with missing lines Patch % Lines
devito/mpi/routines.py 0.00% 51 Missing ⚠️
tests/test_ir.py 36.36% 21 Missing ⚠️
tests/test_data.py 50.00% 18 Missing ⚠️
benchmarks/user/benchmark.py 5.88% 16 Missing ⚠️
devito/mpi/distributed.py 34.78% 15 Missing ⚠️
devito/arch/compiler.py 23.52% 12 Missing and 1 partial ⚠️
devito/types/dense.py 66.66% 10 Missing ⚠️
devito/data/data.py 38.46% 8 Missing ⚠️
devito/tools/data_structures.py 27.27% 8 Missing ⚠️
devito/types/grid.py 70.37% 8 Missing ⚠️
... and 66 more
Additional details and impacted files
@@                          Coverage Diff                          @@
##           JDBetteridge/ruff_manual_changes    #2814       +/-   ##
=====================================================================
+ Coverage                             45.06%   78.96%   +33.89%     
=====================================================================
  Files                                   248      248               
  Lines                                 51047    50826      -221     
  Branches                               4474     4393       -81     
=====================================================================
+ Hits                                  23003    40133    +17130     
+ Misses                                27084     9889    -17195     
+ Partials                                960      804      -156     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@JDBetteridge JDBetteridge force-pushed the JDBetteridge/ruff_manual_changes_part2 branch 5 times, most recently from 09b1053 to aa9925a Compare January 3, 2026 19:34
@JDBetteridge JDBetteridge mentioned this pull request Jan 5, 2026
@JDBetteridge JDBetteridge force-pushed the JDBetteridge/ruff_manual_changes_part2 branch from 7f6c4df to 03c5e29 Compare January 5, 2026 15:19
@JDBetteridge JDBetteridge force-pushed the JDBetteridge/ruff_manual_changes_part2 branch from 03c5e29 to 380f399 Compare January 5, 2026 15:55
@JDBetteridge JDBetteridge force-pushed the JDBetteridge/ruff_manual_changes_part2 branch from 1557959 to 540a43f Compare January 6, 2026 11:26
@JDBetteridge JDBetteridge changed the base branch from main to JDBetteridge/ruff_manual_changes January 6, 2026 11:26
@JDBetteridge JDBetteridge merged commit c91949e into JDBetteridge/ruff_manual_changes Jan 6, 2026
39 of 40 checks passed
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