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

Refactor FeatureExpander Class for Improved Clarity and Functionality #9

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

singhtanishq
Copy link

This pull request refactors the FeatureExpander class in the following ways:

  • Modularizes noise addition to edges into the add_noise_to_edges method.
  • Modularizes edge removal into the remove_specified_edges method.
  • Modularizes node grouping by degree into the group_nodes_by_degree method.
  • Ensures all functionalities and the overall workflow of the original code are preserved.

These changes enhance the readability and maintainability of the code without altering its core functionality.

  • Adds support for grouped nodes by degree.
  • Cleans up feature calculation methods.

Please review the changes and provide feedback.

@yyou1996
Copy link
Collaborator

Thank you for your optimization. I will find time to take a look.

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.

None yet

2 participants