-
Notifications
You must be signed in to change notification settings - Fork 0
/
Copy pathissues.tex
20 lines (16 loc) · 1.85 KB
/
issues.tex
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
A section where note will be made on open issus. These aissues are ment to be discussed with Peter or Thivyian and ultimatly solved. No open issues should remain at the end of November.
\subsection{Graph matching}
% NetworkX offers graph matching algorithms as do other repos. The drawback is that they are programmed for smaller graph such as molecule graphs with undirected edges and no edge atribues, or these attributes are not taken into account and only the structure is compared.
% My implementation of the max-pooling graph matching algorithm from \cite{simonovsky_graphvae_2018} return a $n\times n\times k\times k$ matrix with integer values ${0,2}$. Most probably something went wrong here since this makes little sense regarding interpretation.
% When graphs are compared with NetworkX using the greedy-matching or minimum distance, the similarity matrix is a $2\times 2$ matrix for comparing two graphs.
% My question is which sort of graph matching makes more sense for our usecase.
% Further I struggle to interprete the similarity matrix - is that my job tho? Maybe its just meant to be used in the loss function as described!
Hungarian algorithm:
Should we assume the affinity matrix is a cost or a profit matrix. If we assume it is a cost matrix and n != k the algorithm returns an error while padding. If we assume it is a profit matrix and convert it to a profit matrix it can handle any dimension. The shortest path varies with both approaches.
Further I assume the shortest path is what we are looking for and mask these entries with 1 and the rest of the matrix with 0.
\\
Diagonal of A is zeros. As soon as I do this, code crashes. Think it is because of divided zero, Martin adds 1e-7 to the X norm
\\
Do we backpropagate through the graph matching? then all torch functions, otherwise np.
\\
Martin takes 300 iterations for hungarian. also not looping over pairs.