Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Node classification tutorial #198

Merged

Conversation

Dsantra92
Copy link
Collaborator

@Dsantra92 Dsantra92 commented Jul 12, 2022

  • Writeup
  • Code cleanup

@codecov
Copy link

codecov bot commented Jul 13, 2022

Codecov Report

Merging #198 (278b624) into master (c8216bc) will increase coverage by 0.85%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #198      +/-   ##
==========================================
+ Coverage   86.43%   87.28%   +0.85%     
==========================================
  Files          15       15              
  Lines        1364     1369       +5     
==========================================
+ Hits         1179     1195      +16     
+ Misses        185      174      -11     
Impacted Files Coverage Δ
src/GNNGraphs/gnngraph.jl 71.42% <0.00%> (-1.08%) ⬇️
src/GNNGraphs/transform.jl 96.55% <0.00%> (-0.42%) ⬇️
src/layers/basic.jl 81.25% <0.00%> (+32.46%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c8216bc...278b624. Read the comment docs.

@CarloLucibello CarloLucibello changed the title Graph Node Classification Node classification tutorial Jul 13, 2022
@CarloLucibello
Copy link
Member

the notebook has a mixture of 4 and 8 blank indents, can we make it 4 everywhere?

@Dsantra92
Copy link
Collaborator Author

Yes sure, after I finish the text part, I will clean it up.

@CarloLucibello
Copy link
Member

what's the status here?

@Dsantra92 Dsantra92 added the gsoc label Jul 31, 2022
@Dsantra92 Dsantra92 marked this pull request as ready for review September 3, 2022 23:15
@CarloLucibello CarloLucibello merged commit 7f16b69 into JuliaGraphs:master Sep 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants