-
Notifications
You must be signed in to change notification settings - Fork 8
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
Training test migration #1002
Training test migration #1002
Conversation
8c4976f
to
9910b26
Compare
cfbc400
to
aad5576
Compare
@prishavall I got the build check to work for this action. Had to do some fixing with the micromamba action |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remove debug prints (nits)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
added another quick comment
What user problem are we solving?
Migrating unit tests from backend to the training module.
What solution does this PR provide?
Fixes the pathway errors and import changes with the transfer from backend to training.
Testing Methodology
How did you test your changes and verify that existing
functionality is not broken
Ran tests individually
Any other considerations
One test model_py is still not working need to look into django implementation for that.