GraphScope icon indicating copy to clipboard operation
GraphScope copied to clipboard

fix(interactive): fix bug in degree fusion in Insight Runtime

Open BingqingLyu opened this issue 9 months ago • 3 comments

Committed-by: BingqingLyu from Dev container

What do these changes do?

As titled.

Related issue number

Fixes #4528

BingqingLyu avatar Feb 25 '25 12:02 BingqingLyu

Please check the preview of the documentation changes at https://32b19123.graphscope-docs-preview.pages.dev

github-actions[bot] avatar Feb 25 '25 12:02 github-actions[bot]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 33.02%. Comparing base (b0b8099) to head (129248b). Report is 9 commits behind head on main.

:exclamation: There is a different number of reports uploaded between BASE (b0b8099) and HEAD (129248b). Click for more details.

HEAD has 4 uploads less than BASE
Flag BASE (b0b8099) HEAD (129248b)
5 1
Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #4529       +/-   ##
===========================================
- Coverage   63.29%   33.02%   -30.27%     
===========================================
  Files         127      127               
  Lines       13319    13299       -20     
===========================================
- Hits         8430     4392     -4038     
- Misses       4889     8907     +4018     

see 80 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update b0b8099...129248b. Read the comment docs.

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

codecov-commenter avatar Feb 25 '25 13:02 codecov-commenter

The bug occurs when using the ir-core based compiler. Skip these ir-core based tests as ir-core will be removed soon. Maybe it is better to set the default configuration in compiler as calcite-based? @shirly121

BingqingLyu avatar Mar 06 '25 07:03 BingqingLyu