Skip to content

[MINOR] CLA Factory logging cleanup#2211

Closed
Baunsgaard wants to merge 3 commits intoapache:mainfrom
Baunsgaard:CLAFactoryLog
Closed

[MINOR] CLA Factory logging cleanup#2211
Baunsgaard wants to merge 3 commits intoapache:mainfrom
Baunsgaard:CLAFactoryLog

Conversation

@Baunsgaard
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Feb 3, 2025

Codecov Report

Attention: Patch coverage is 78.37838% with 8 lines in your changes missing coverage. Please review.

Project coverage is 72.38%. Comparing base (ece172a) to head (436feb2).
Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
...runtime/compress/CompressedMatrixBlockFactory.java 85.71% 2 Missing and 2 partials ⚠️
.../sysds/runtime/compress/CompressedMatrixBlock.java 50.00% 1 Missing ⚠️
.../sysds/runtime/compress/CompressionStatistics.java 0.00% 0 Missing and 1 partial ⚠️
...g/apache/sysds/runtime/compress/colgroup/ASDC.java 50.00% 1 Missing ⚠️
...ache/sysds/runtime/compress/colgroup/ASDCZero.java 75.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main    #2211      +/-   ##
============================================
- Coverage     72.39%   72.38%   -0.01%     
+ Complexity    45333    45320      -13     
============================================
  Files          1467     1467              
  Lines        170536   170629      +93     
  Branches      33247    33260      +13     
============================================
+ Hits         123454   123505      +51     
- Misses        37691    37726      +35     
- Partials       9391     9398       +7     

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

@Baunsgaard
Copy link
Contributor Author

merged

@Baunsgaard Baunsgaard closed this Feb 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant