murrayds / sci-mobility-emb

Embedding of scientific mobility across institutions, cities, regions, and countries
4 stars 0 forks source link

adding new math part and result with dot product #95

Closed jisungyoon closed 4 years ago

jisungyoon commented 4 years ago

I just added a new math part without symmetry condition and relaxation cosine similarity to dot product.

And, @murrayds adds the result with dot product.

murrayds commented 4 years ago

Looks good! I also committed my changes throughout the draft.

I added new figures but didn't know where you wanted to include those that are relevant to the math. These figures include,

When responding to Stasa's comments, I added text to the supporting materials so as to preserve the main structure of the current manuscript. These texts are definitely in need of editing,

I also made minor changes throughout the manuscript, fixing broken links and editing text where applicable. The only major changes were:

murrayds commented 4 years ago

Also please look over my changes