search

THUKEG / saedb

the SAE platform
http://thukeg.github.com/saedb/
11 stars 19 forks source link

replace underlying graph api #23

Closed wweic closed 11 years ago

wweic commented 11 years ago

with @thinxer . I'll understand you code first.

wweic commented 11 years ago

You API is now enough for my use, local_graph, g_storage is ok to drop totally. As for graph.hpp, the use of graph now just getVerticesNum, iterateVertices, name are not the same, but with this idea. You can search in synchronousengine.hpp for `graph`.

@thinxer

thinxer commented 11 years ago

Great! Let's merge the code tomorrow!

On Thu, Mar 28, 2013 at 11:28 PM, Wei Chen notifications@github.com wrote:

You API is now enough for my use, local_graph, g_storage is ok to drop totally. As for graph.hpp, the use of graph now just getVerticesNum, iterateVertices, name are not the same, but with this idea. You can search in synchronousengine.hpp for graph.

— Reply to this email directly or view it on GitHubhttps://github.com/THUKEG/saedb/issues/23#issuecomment-15594766 .

wweic commented 11 years ago

Tomorrow I have to go back to MS to visit my old mentor, do you think it's ok Tomorrow evening + GTalk?

Wei Chen ipondering.me

On 2013年3月28日Thursday at 下午11:42, Jianfei Wang wrote:

Great! Let's merge the code tomorrow!

On Thu, Mar 28, 2013 at 11:28 PM, Wei Chen <notifications@github.com (mailto:notifications@github.com)> wrote:

You API is now enough for my use, local_graph, g_storage is ok to drop
totally.
As for graph.hpp, the use of graph now just getVerticesNum,
iterateVertices, name are not the same, but with this idea. You can
search in synchronousengine.hpp for graph.

—
Reply to this email directly or view it on GitHubhttps://github.com/THUKEG/saedb/issues/23#issuecomment-15594766
.

— Reply to this email directly or view it on GitHub (https://github.com/THUKEG/saedb/issues/23#issuecomment-15595629).

thinxer commented 11 years ago

I may not be available tomorrow evening. I'll see if I can finish it alone. Thx! On Mar 29, 2013 12:13 AM, "Wei Chen" notifications@github.com wrote:

Tomorrow I have to go back to MS to visit my old mentor, do you think it's ok Tomorrow evening + GTalk?

Wei Chen ipondering.me

On 2013年3月28日Thursday at 下午11:42, Jianfei Wang wrote:

Great! Let's merge the code tomorrow!

On Thu, Mar 28, 2013 at 11:28 PM, Wei Chen <notifications@github.com(mailto: notifications@github.com)> wrote:

You API is now enough for my use, local_graph, g_storage is ok to drop totally. As for graph.hpp, the use of graph now just getVerticesNum, iterateVertices, name are not the same, but with this idea. You can search in synchronousengine.hpp for graph.

— Reply to this email directly or view it on GitHub< https://github.com/THUKEG/saedb/issues/23#issuecomment-15594766> .

— Reply to this email directly or view it on GitHub ( https://github.com/THUKEG/saedb/issues/23#issuecomment-15595629).

— Reply to this email directly or view it on GitHubhttps://github.com/THUKEG/saedb/issues/23#issuecomment-15597507 .

wweic commented 11 years ago

Done.