CarloLucibello / GraphNeuralNetworks.jl

Graph Neural Networks in Julia
https://carlolucibello.github.io/GraphNeuralNetworks.jl/dev/
MIT License
216 stars 46 forks source link

Fix gnnheterographs failed test #241

Closed aurorarossi closed 1 year ago

aurorarossi commented 1 year ago

With this PR I fixed the print order of Dicts and the gnnheterographs tests should now not fail.

codecov[bot] commented 1 year ago

Codecov Report

Merging #241 (96a7d08) into master (156cfae) will increase coverage by 0.12%. The diff coverage is 93.75%.

@@            Coverage Diff             @@
##           master     #241      +/-   ##
==========================================
+ Coverage   88.29%   88.42%   +0.12%     
==========================================
  Files          16       16              
  Lines        1589     1624      +35     
==========================================
+ Hits         1403     1436      +33     
- Misses        186      188       +2     
Impacted Files Coverage Δ
src/GNNGraphs/gnnheterograph.jl 85.91% <93.75%> (+4.09%) :arrow_up:
src/GNNGraphs/query.jl 94.02% <0.00%> (+0.27%) :arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

aurorarossi commented 1 year ago

(I will open another PR for the laplacian_lambda_max error)