Skip to content
This repository has been archived by the owner on Sep 18, 2023. It is now read-only.

[master<]Gnn link prediction #536

Merged
merged 5 commits into from
Oct 31, 2022
Merged

[master<]Gnn link prediction #536

merged 5 commits into from
Oct 31, 2022

Conversation

as51340
Copy link
Contributor

@as51340 as51340 commented Aug 29, 2022 •
edited
Loading

Description

The documentation for link prediction with graph neural networks is added.

Pull request type

Please delete options that are not relevant and check the ones that are.

  • New documentation page

Checklist:

  • I checked all content with Grammarly
  • I performed a self-review of my code
  • I made corresponding changes to the rest of the documentation
  • The build passes locally
  • My changes generate no new warnings or errors

@vpavicic vpavicic added the status: change PR reviewed - needs changes label Sep 7, 2022
@vpavicic
Copy link
Collaborator

Please rename tha PR so we know what branch you want to mergo into which branch :)

@vpavicic vpavicic changed the title Gnn link prediction [master<]Gnn link prediction Sep 15, 2022
@as51340 as51340 requested a review from vpavicic October 20, 2022 10:31
@antoniofilipovic
Copy link
Contributor

antoniofilipovic commented Oct 20, 2022 •
edited
Loading

@as51340, update following page ../machine-learning-graph-analytics/link-prediction-algorithm that we have now link prediction algorithm

vpavicic
vpavicic suggested changes Oct 26, 2022
View reviewed changes
Copy link
Collaborator

@vpavicic vpavicic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

address Antonio's comment

@as51340
Copy link
Contributor Author

as51340 commented Oct 27, 2022

address Antonio's comment

Will do it in a separate PR because of the dependency chain.

@vpavicic
Copy link
Collaborator

Due to dependencies, I will merge this so you can continue working on it, but please also contact design to replace these images
I'll create an issue and assign it to you

@vpavicic vpavicic self-requested a review October 31, 2022 14:37
vpavicic
vpavicic approved these changes Oct 31, 2022
View reviewed changes
Copy link
Collaborator

@vpavicic vpavicic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approve but with debt to images and dependancies

@vpavicic vpavicic mentioned this pull request Oct 31, 2022
Closed
@vpavicic vpavicic merged commit 32cffb7 into master Oct 31, 2022
@vpavicic vpavicic deleted the gnn_link_prediction branch February 20, 2023 08:05
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@vpavicic vpavicic vpavicic approved these changes

@kgolubic kgolubic Awaiting requested review from kgolubic kgolubic is a code owner

@katarinasupe katarinasupe Awaiting requested review from katarinasupe

@antejavor antejavor Awaiting requested review from antejavor

Assignees
No one assigned
Labels
status: change PR reviewed - needs changes
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@as51340 @vpavicic @antoniofilipovic