-
Notifications
You must be signed in to change notification settings - Fork 0
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
Add unit tests for prepare_categorical_data #5
Conversation
Looks good so far. Could you check if all unit tests pass on your machine? There seems to be an issue with the test on the CI and I just want to make sure everything runs. And please pull all changes from |
All tests are passing now. I have modified other files because of changes in |
@premsrii If you need help resolving the merge conflict. We could do it together. |
Codecov Report
@@ Coverage Diff @@
## main #5 +/- ##
==========================================
+ Coverage 94.31% 94.82% +0.50%
==========================================
Files 8 8
Lines 352 348 -4
==========================================
- Hits 332 330 -2
+ Misses 20 18 -2
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
This PR adds unit tests for the function
prepare_categoriccal_data()
. The function is also modified to handle only non-numerical data.