Skip to content

Simple clean ups#219

Open
astricks wants to merge 5 commits intoyl4579:mainfrom
astricks:astricks/simple-clean-ups
Open

Simple clean ups#219
astricks wants to merge 5 commits intoyl4579:mainfrom
astricks:astricks/simple-clean-ups

Conversation

@astricks
Copy link
Contributor

No description provided.

@astricks
Copy link
Contributor Author

First of a few PRs to start organizing the code for maintainability and readability. I'm testing locally, but would also love for others to try to run the branch and make sure it still works.

I'm only removing unused imports and variables, so its won't affect functionality.

@astricks astricks changed the title Astricks/simple clean ups Simple clean ups Mar 20, 2024
@astricks
Copy link
Contributor Author

Testing: I ran train_first, train_second and train_finetune to make sure they still work.

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