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.

Sign up for GitHub

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

Jump to bottom

optimize get-feat function of graph engine #32261

Merged
merged 143 commits into from
Apr 21, 2021
Merged

optimize get-feat function of graph engine #32261

merged 143 commits into from
Apr 21, 2021

Conversation

seemingwang
Copy link
Contributor

@seemingwang seemingwang commented Apr 14, 2021 •
edited
Loading

PR types

Function optimization

PR changes

Others

Describe

enable the get-feat function to avoid local brpc

seemingwang and others added 30 commits February 25, 2021 12:30
@seemingwang
graph engine demo
9d24726
@seemingwang
upload unsaved changes
7753f38
@seemingwang
fix dependency error
5fa66ab
@seemingwang
fix shard_num problem
3e12386
@seemingwang
py client
acacce0
@seemingwang
remove lock and graph-type
3f32bf1
@Yelrose
Merge pull request #2 from seemingwang/develop
dda5291 
Merge Graph Engine
@Yelrose
add load direct graph
44df9ac
@Yelrose
add load direct graph
c8e67b8
@Yelrose
add load direct graph
7320994
@WeiyueSu
batch random_sample
f4ea973
@WeiyueSu
batch_sample_k
492e551
@WeiyueSu
fix num_nodes size
193203f
@WeiyueSu
batch brpc
b55bf42
@WeiyueSu
batch brpc
102ae95
@WeiyueSu
add test
3106b2a
@WeiyueSu
add test
51ed7d8
@seemingwang
Merge pull request #1 from Yelrose/develop
94025b9 
Add Simple Loading for Direct Graph
@Yelrose
add load_nodes; change add_node function
e087485
@Yelrose
Merge pull request #3 from seemingwang/develop
455e2a0 
Merge pull request #1 from Yelrose/develop
@seemingwang
change sample return type to pair
c588c96
@Yelrose
merge
391b8e5
@Yelrose
resolve conflict
2dc7ebf
@Yelrose
Merge branch 'develop' of https://github.com/Yelrose/Paddle into develop
f1ea3ac
@Yelrose
resolved conflict
e359135
@Yelrose
resolved conflict
a4e8939
@Yelrose
resolved conflict
290840f
@seemingwang
separate server and client
d98be69
@WeiyueSu
merge pair type
db6ee86
@Yelrose
resolved conflict
550a23f
seemingwang and others added 22 commits March 28, 2021 03:28
@seemingwang
Merge remote-tracking branch 'origin/graph_engine1.0' into develop
88cd27a
@seemingwang
resolve conflict
6535661
@seemingwang
resolve linking problem
63d7fdc
@seemingwang
resolve conflict
0280122
@seemingwang
code style
fe5eb72
@seemingwang
change test_server port
6f0b2e1
@seemingwang
Merge remote-tracking branch 'upstream/develop' into develop
ed9a69f
@seemingwang
Merge remote-tracking branch 'upstream/develop' into develop
c705c84
@seemingwang
fix code problems
771be00
@seemingwang
remove shard_num config
1a3013a
@seemingwang
remove redundent threads
bd6b545
@seemingwang
optimize start server
a768899
@seemingwang
remove logs
085bca6
@seemingwang
fix code problems
3b44e5a
@seemingwang
fix code problems by reviewers' suggestions
0f85b95
@seemingwang
move graph files into a folder
2afea69
@seemingwang
resolve conflicts
ceebbf9
@seiriosPlus
Merge branch 'develop' into develop
a537778
@seemingwang
code style change
06c6c9b
@seemingwang
remove graph operations from base table
f291ea0
@seemingwang
Merge remote-tracking branch 'upstream/develop' into develop
d4d4e1a
@seemingwang
optimize get_feat function of graph engine
59fa614
@paddle-bot-old
Copy link

Thanks for your contribution!
Please wait for the result of CI firstly. See Paddle CI Manual for details.

@paddle-bot-old
Copy link

paddle-bot-old bot commented Apr 14, 2021 •
edited
Loading

✅ This PR's description meets the template requirements!
Please wait for other CI results.

@seiriosPlus
Copy link
Collaborator

LGTM

@PaddlePaddle PaddlePaddle locked and limited conversation to collaborators Apr 21, 2021
@PaddlePaddle PaddlePaddle unlocked this conversation Apr 21, 2021
@seiriosPlus seiriosPlus merged commit 2b68d20 into PaddlePaddle:develop Apr 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@seiriosPlus seiriosPlus seiriosPlus approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@seemingwang @seiriosPlus @Yelrose @WeiyueSu @Liwb5