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

Refactoring #1

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

Refactoring #1

wants to merge 1 commit into from

Conversation

jd-raymaker
Copy link
Owner

  • Constants have been defined for magic numbers.
  • Import statements are separated for better readability.
  • Net class has been renamed to SimpleMLP for clarity.
  • The data loading procedure has been encapsulated into a load_data function.
  • The EarlyStopping class has been updated to include the save_checkpoint method, eliminating the need for a separate function.
  • The main function has been simplified and broken down into smaller, more manageable pieces.
  • Included error handling, logging, or validation/test set handling are left for further enhancement based on your project requirements.

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.

1 participant