You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@systemds.apache.org by "Baunsgaard (via GitHub)" <gi...@apache.org> on 2024/12/29 20:13:01 UTC

[PR] [MINOR] Update cost estimation in CLA [systemds]

Baunsgaard opened a new pull request, #2168:
URL: https://github.com/apache/systemds/pull/2168

   (no comment)


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@systemds.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [MINOR] Update cost estimation in CLA [systemds]

Posted by "codecov[bot] (via GitHub)" <gi...@apache.org>.
codecov[bot] commented on PR #2168:
URL: https://github.com/apache/systemds/pull/2168#issuecomment-2564835914

   ## [Codecov](https://app.codecov.io/gh/apache/systemds/pull/2168?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) Report
   Attention: Patch coverage is `48.23529%` with `44 lines` in your changes missing coverage. Please review.
   > Project coverage is 72.27%. Comparing base [(`96fd5da`)](https://app.codecov.io/gh/apache/systemds/commit/96fd5da49e4f936c378f11cc275c34ff5f8a84d1?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) to head [(`451cc2a`)](https://app.codecov.io/gh/apache/systemds/commit/451cc2a3967cfb0097317d88b110858f695e46a7?dropdown=coverage&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   > Report is 1 commits behind head on main.
   
   | [Files with missing lines](https://app.codecov.io/gh/apache/systemds/pull/2168?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) | Patch % | Lines |
   |---|---|---|
   | [...runtime/compress/estim/ComEstCompressedSample.java](https://app.codecov.io/gh/apache/systemds/pull/2168?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fapache%2Fsysds%2Fruntime%2Fcompress%2Festim%2FComEstCompressedSample.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3N5c2RzL3J1bnRpbWUvY29tcHJlc3MvZXN0aW0vQ29tRXN0Q29tcHJlc3NlZFNhbXBsZS5qYXZh) | 41.37% | [17 Missing :warning: ](https://app.codecov.io/gh/apache/systemds/pull/2168?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [...che/sysds/runtime/compress/estim/ComEstSample.java](https://app.codecov.io/gh/apache/systemds/pull/2168?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fapache%2Fsysds%2Fruntime%2Fcompress%2Festim%2FComEstSample.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3N5c2RzL3J1bnRpbWUvY29tcHJlc3MvZXN0aW0vQ29tRXN0U2FtcGxlLmphdmE=) | 56.66% | [13 Missing :warning: ](https://app.codecov.io/gh/apache/systemds/pull/2168?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [...untime/compress/cost/ComputationCostEstimator.java](https://app.codecov.io/gh/apache/systemds/pull/2168?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fapache%2Fsysds%2Fruntime%2Fcompress%2Fcost%2FComputationCostEstimator.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3N5c2RzL3J1bnRpbWUvY29tcHJlc3MvY29zdC9Db21wdXRhdGlvbkNvc3RFc3RpbWF0b3IuamF2YQ==) | 16.66% | [2 Missing and 3 partials :warning: ](https://app.codecov.io/gh/apache/systemds/pull/2168?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [...ysds/runtime/compress/estim/EstimationFactors.java](https://app.codecov.io/gh/apache/systemds/pull/2168?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fapache%2Fsysds%2Fruntime%2Fcompress%2Festim%2FEstimationFactors.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3N5c2RzL3J1bnRpbWUvY29tcHJlc3MvZXN0aW0vRXN0aW1hdGlvbkZhY3RvcnMuamF2YQ==) | 0.00% | [4 Missing :warning: ](https://app.codecov.io/gh/apache/systemds/pull/2168?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [...sysds/runtime/compress/estim/ComEstCompressed.java](https://app.codecov.io/gh/apache/systemds/pull/2168?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fapache%2Fsysds%2Fruntime%2Fcompress%2Festim%2FComEstCompressed.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3N5c2RzL3J1bnRpbWUvY29tcHJlc3MvZXN0aW0vQ29tRXN0Q29tcHJlc3NlZC5qYXZh) | 50.00% | [2 Missing :warning: ](https://app.codecov.io/gh/apache/systemds/pull/2168?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [...ime/compress/estim/CompressedSizeInfoColGroup.java](https://app.codecov.io/gh/apache/systemds/pull/2168?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fapache%2Fsysds%2Fruntime%2Fcompress%2Festim%2FCompressedSizeInfoColGroup.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3N5c2RzL3J1bnRpbWUvY29tcHJlc3MvZXN0aW0vQ29tcHJlc3NlZFNpemVJbmZvQ29sR3JvdXAuamF2YQ==) | 0.00% | [1 Missing and 1 partial :warning: ](https://app.codecov.io/gh/apache/systemds/pull/2168?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   | [...he/sysds/runtime/compress/estim/ComEstFactory.java](https://app.codecov.io/gh/apache/systemds/pull/2168?src=pr&el=tree&filepath=src%2Fmain%2Fjava%2Forg%2Fapache%2Fsysds%2Fruntime%2Fcompress%2Festim%2FComEstFactory.java&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache#diff-c3JjL21haW4vamF2YS9vcmcvYXBhY2hlL3N5c2RzL3J1bnRpbWUvY29tcHJlc3MvZXN0aW0vQ29tRXN0RmFjdG9yeS5qYXZh) | 88.88% | [0 Missing and 1 partial :warning: ](https://app.codecov.io/gh/apache/systemds/pull/2168?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache) |
   
   <details><summary>Additional details and impacted files</summary>
   
   
   ```diff
   @@             Coverage Diff              @@
   ##               main    #2168      +/-   ##
   ============================================
   - Coverage     72.29%   72.27%   -0.02%     
   - Complexity    44248    44260      +12     
   ============================================
     Files          1443     1444       +1     
     Lines        166603   166664      +61     
     Branches      32537    32546       +9     
   ============================================
   + Hits         120441   120459      +18     
   - Misses        36920    36957      +37     
   - Partials       9242     9248       +6     
   ```
   
   </details>
   
   [:umbrella: View full report in Codecov by Sentry](https://app.codecov.io/gh/apache/systemds/pull/2168?dropdown=coverage&src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).   
   :loudspeaker: Have feedback on the report? [Share it here](https://about.codecov.io/codecov-pr-comment-feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=apache).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@systemds.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] [MINOR] Update cost estimation in CLA [systemds]

Posted by "Baunsgaard (via GitHub)" <gi...@apache.org>.
Baunsgaard closed pull request #2168: [MINOR] Update cost estimation in CLA
URL: https://github.com/apache/systemds/pull/2168


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: dev-unsubscribe@systemds.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org