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

Added remove_edges function #414

Merged
merged 18 commits into from
Mar 21, 2024
Merged

Conversation

rbSparky
Copy link
Contributor

@rbSparky rbSparky commented Mar 8, 2024

This is the first step in successfully implementing Issue #412 for which we will need to implement DropEdge.

Tests added

src/GNNGraphs/transform.jl Outdated Show resolved Hide resolved
src/GNNGraphs/transform.jl Outdated Show resolved Hide resolved
test/GNNGraphs/transform.jl Show resolved Hide resolved
src/GNNGraphs/transform.jl Outdated Show resolved Hide resolved
src/GNNGraphs/transform.jl Outdated Show resolved Hide resolved
test/GNNGraphs/transform.jl Show resolved Hide resolved
rbSparky and others added 2 commits March 17, 2024 01:14
@rbSparky
Copy link
Contributor Author

Made all the changes

@CarloLucibello CarloLucibello changed the title Added remove_edge function Added remove_edges function Mar 17, 2024
.gitignore Outdated Show resolved Hide resolved
src/GNNGraphs/transform.jl Show resolved Hide resolved
src/GNNGraphs/transform.jl Outdated Show resolved Hide resolved
test/GNNGraphs/transform.jl Outdated Show resolved Hide resolved
test/GNNGraphs/transform.jl Outdated Show resolved Hide resolved
@rbSparky
Copy link
Contributor Author

All fixes done

@rbSparky
Copy link
Contributor Author

Does anything else need to be done in this PR?

@CarloLucibello CarloLucibello merged commit 95a90fc into JuliaGraphs:master Mar 21, 2024
4 of 5 checks passed
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