Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[GAE/tests] Add GAE tests for FLASH algorithms of bfs and cc #2869

Merged
merged 12 commits into from
Jun 13, 2023

Conversation

longbinlai
Copy link
Collaborator

Committed-by: longbinlai from Dev container

Fix issue #2824

Committed-by: longbinlai from Dev container
Committed-by: longbinlai from Dev container
@longbinlai longbinlai requested a review from lixueclaire June 13, 2023 04:10
@github-actions
Copy link
Contributor

github-actions bot commented Jun 13, 2023

🎊 PR Preview 512f101 has been successfully built and deployed to https://alibaba-graphscope-build-pr-2869.surge.sh

🤖 By surge-preview

Copy link
Collaborator

@lixueclaire lixueclaire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You could use ./gs format python to format the code

longbinlai and others added 6 commits June 13, 2023 14:05
Committed-by: longbinlai from Dev container
 into add-flash-test

Committed-by: longbinlai from Dev container
Committed-by: longbinlai from Dev container
Committed-by: longbinlai from Dev container
Copy link
Collaborator

@lixueclaire lixueclaire left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM~

@codecov-commenter
Copy link

Codecov Report

Merging #2869 (fe3fd9a) into main (387f915) will increase coverage by 23.61%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #2869       +/-   ##
===========================================
+ Coverage   42.59%   66.20%   +23.61%     
===========================================
  Files          99       99               
  Lines       10657    10667       +10     
===========================================
+ Hits         4539     7062     +2523     
+ Misses       6118     3605     -2513     
Impacted Files Coverage Δ
python/graphscope/tests/unittest/test_flash.py 100.00% <100.00%> (+100.00%) ⬆️

... and 61 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 387f915...fe3fd9a. Read the comment docs.

@lixueclaire lixueclaire changed the title [Flash] add flash tests of bfs and cc [GAE/tests] Add FLASH tests of bfs and cc Jun 13, 2023
@lixueclaire lixueclaire changed the title [GAE/tests] Add FLASH tests of bfs and cc [GAE/tests] Add GAE tests for FLASH algorithms of bfs and cc Jun 13, 2023
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.

3 participants