search

uhh-lt / path2vec

Learning to represent shortest paths and other graph-based measures of node similarities with graph embeddings
Apache License 2.0
33 stars 14 forks source link

Redraw the plot #25

Closed alexanderpanchenko closed 6 years ago

alexanderpanchenko commented 6 years ago
image
alexanderpanchenko commented 6 years ago

generate plots for 10-20 sentences

m-dorgham commented 6 years ago

@alexanderpanchenko What do you think about these figures?

figure_1

figure_2

figure_3

alexanderpanchenko commented 6 years ago

Thanks - it looks much better now.

  1. Please make the font of words below ca twice as larger.

  2. Set the color of the connections to the same one as the color of the blue nodes.

  3. Also enlarge font of the size of th nodes a bit.

  4. Keep only the last example (what is the sentence by the way?)

  5. Overall thickness of edges should be increased.

m-dorgham commented 6 years ago

Look at these pics for the last sentence with the changes you asked. These are 3 pics with different edge thickness.

figure_3 figure_5 figure_6

This graph is part of the whole graph generated for the sentence: More often than not, ringers think of the church as something stuck on the bottom of the belfry. Here is the whole graph:

figure_4

alexanderpanchenko commented 6 years ago

The most thick one is the best IMO.

  1. Is this one takes into account edge pruning with the threshold?

  2. Please make the original words below bigger even more.

On 3. Dec 2018, at 23:48, Mohammad Dorgham notifications@github.com wrote:

Look at these pics for the last sentence with the changes you asked. These are 3 pics with different edge thickness.

This graph is part of the whole graph generated for the sentence: More often than not, ringers think of the church as something stuck on the bottom of the belfry. Here is the whole graph:

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

m-dorgham commented 6 years ago

  1. yes I did edge pruning in this image but with a small threshold.

  2. ok I will.

alexanderpanchenko commented 6 years ago

On 4. Dec 2018, at 14:02, Mohammad Dorgham notifications@github.com wrote:

yes I did edge pruning in this image but with a small threshold.

Ok. I see.

ok I will.

Thanks!

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

alexanderpanchenko commented 6 years ago

@m-dorgham i checked the image again, it all looks good but I just need a copy of this one

image

with much larger font on the bottom (AND you can make the connections even thicker ...)

m-dorgham commented 6 years ago

@alexanderpanchenko Is this one fine?

figure_7

alexanderpanchenko commented 6 years ago

Thanks, perfect

On 5. Dec 2018, at 22:16, Mohammad Dorgham notifications@github.com wrote:

@alexanderpanchenko Is this one fine?

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.

m-dorgham commented 6 years ago

@alexanderpanchenko I enlarged the font size of the node names to be compatible with the text below it.

figure_8

alexanderpanchenko commented 6 years ago

Thanks, yes it looks nicer like this

On 5. Dec 2018, at 22:28, Mohammad Dorgham notifications@github.com wrote:

@alexanderpanchenko I enlarged the font size of the node names to be compatible with the text below it.

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub, or mute the thread.