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

feat(analytical): Support extend label data for graph in eager mode. #3288

Merged
merged 6 commits into from
Dec 18, 2023

Conversation

SighingSnow
Copy link
Contributor

Signed-off-by: SighingSnow 1263750383@qq.com

What do these changes do?

Support add data to existed label.

Related issue number

FEAT #411

@welcome
Copy link

welcome bot commented Oct 12, 2023

Thanks for submitting your first pull request! You are awesome! 🤗
Please make sure you have signed the CLA, as this is a mandatory check before a PR being merged.
Welcome to the GraphScope community! 🎉 You can meet the community on DingTalk or Slack.

@CLAassistant
Copy link

CLAassistant commented Oct 12, 2023

CLA assistant check
All committers have signed the CLA.

sighingnow
sighingnow previously approved these changes Oct 13, 2023
Signed-off-by: SighingSnow <1263750383@qq.com>

Committed-by: SighingSnow from Dev container

Committed-by: SighingSnow from Dev container

Committed-by: SighingSnow from Dev container
Committed-by: SighingSnow from Dev container

.

Committed-by: SighingSnow from Dev container

Committed-by: SighingSnow from Dev container

Committed-by: SighingSnow from Dev container
siyuan0322
siyuan0322 previously approved these changes Nov 6, 2023
python/graphscope/framework/graph.py Show resolved Hide resolved
@siyuan0322 siyuan0322 changed the title [FEAT-411] Support extend label data for graph in eager mode. fix(analytical): Support extend label data for graph in eager mode. Dec 13, 2023
@codecov-commenter
Copy link

codecov-commenter commented Dec 13, 2023

Codecov Report

Merging #3288 (bd46ef5) into main (8047bf8) will increase coverage by 21.66%.
Report is 6 commits behind head on main.
The diff coverage is 94.59%.

Additional details and impacted files

Impacted file tree graph

@@             Coverage Diff             @@
##             main    #3288       +/-   ##
===========================================
+ Coverage   43.31%   64.98%   +21.66%     
===========================================
  Files         109      110        +1     
  Lines       11340    11381       +41     
===========================================
+ Hits         4912     7396     +2484     
+ Misses       6428     3985     -2443     
Files Coverage Δ
python/graphscope/framework/dag_utils.py 64.17% <ø> (+19.62%) ⬆️
...hon/graphscope/tests/unittest/test_create_graph.py 92.62% <ø> (+92.62%) ⬆️
python/graphscope/framework/graph.py 83.06% <90.00%> (+13.82%) ⬆️
python/graphscope/tests/unittest/test_graph.py 88.06% <96.29%> (+88.06%) ⬆️

... 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 cf7120b...bd46ef5. Read the comment docs.

@siyuan0322 siyuan0322 merged commit 7f27fa3 into alibaba:main Dec 18, 2023
28 of 30 checks passed
@sighingnow sighingnow changed the title fix(analytical): Support extend label data for graph in eager mode. feat(analytical): Support extend label data for graph in eager mode. Dec 18, 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.

None yet

5 participants