search

SiRumCz / CSC501

CSC501 assignments
0 stars 1 forks source link

add edge-bundling data #90

Closed SiRumCz closed 4 years ago

SiRumCz commented 4 years ago

finished midnight, forgot to create PR for this.

edge bundling graph on top selected nodes API format: /edge-bundling?size= @param size: top n number of nodes. default value is 100 [ {'name': '48.pcmasterrace', 'size': 60, 'imports': ["0.personalfinance", "0.self", ...]}, {...} ... ]

name string: community name + '.' + subreddit name size: number of outgoing edges after downsizing imports: list of outgoing subreddit name strings

API format: /edge-bundling?size=

jonhealy1 commented 4 years ago

Awesome yea.

soroushysfi commented 4 years ago

I was trying to review the API, and it got merged?

jonhealy1 commented 4 years ago

Sorry I didn't know you were working.

soroushysfi commented 4 years ago

It's cool I thought PRs need two reviews to be merged. I didn't know we can merge them without the second review.

jonhealy1 commented 4 years ago

Nope just one review.

SiRumCz commented 4 years ago

I changed it to just one since second assignment since I feel that two reviewers sometimes block the workflows.

Kevin

On Tue., Nov. 5, 2019, 10:00 a.m. Jonathan Healy notifications@github.com wrote:

Nope just one review.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/SiRumCz/CSC501/pull/90?email_source=notifications&email_token=AEWWYWVWFBT3EHYOOLKEBY3QSGYC7A5CNFSM4JJF5442YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDDXDSY#issuecomment-549941707, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEWWYWTDSIVYFSZWDZAJRMTQSGYC7ANCNFSM4JJF544Q .