Modeling issue 31 naive bayes

dominikmn / one-million-posts

Assisting newspaper moderators with machine learning.

MIT License

2 stars 1 forks source link

Closed lima-tango closed 3 years ago

lima-tango commented 3 years ago

Done:

Add logging of confusion matrix
Fix utils.feature_engineering.add_column_text
Add cycle tag
Fix error with NaN due to new train-test-val-split
- Move iteration for labels from run_training() into main
- Move get_data() call from run_training() into main__

Needs improvement:

Closes #31

lima-tango commented 3 years ago

This MR will not be merged. The suggested improvements are discussed/included in #62. A new MR will be written to solve Issue #31.