Skip to content

Code Cleanup (Ready to merge)#191

Merged
annawoodard merged 15 commits intomainfrom
code_cleanup
May 24, 2025
Merged

Code Cleanup (Ready to merge)#191
annawoodard merged 15 commits intomainfrom
code_cleanup

Conversation

@collin-k
Copy link
Copy Markdown
Contributor

@collin-k collin-k commented May 1, 2025

Description
This PR addresses code readability and maintainability.

Changes

  1. The one_trial function and the signature for the train function have been revised so that variables are passed as flat parameters and not as a part of a tuple.
  2. The train_epoch function and the signature for the train_setup function have been revised so that variables are passed as flat parameters and not as a part of a tuple.
  3. The train function and the signature for the test function have been revised so that variables are passed as flat parameters and not as a part of a tuple.

@collin-k collin-k changed the title [WIP] Cleanup [WIP] Code Cleanup May 1, 2025
@collin-k collin-k requested a review from annawoodard May 4, 2025 20:48
@collin-k collin-k changed the title [WIP] Code Cleanup Code Cleanup May 4, 2025
@collin-k collin-k changed the title Code Cleanup Code Cleanup (Ready to merge) May 5, 2025
Copy link
Copy Markdown
Collaborator

@annawoodard annawoodard left a comment

Choose a reason for hiding this comment

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

Great thank you!

@annawoodard annawoodard merged commit 9015e7b into main May 24, 2025
1 check passed
@collin-k collin-k deleted the code_cleanup branch May 28, 2025 06:19
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