Skip to content

Update 03_model_evaluation #2

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 3 commits into from
Nov 1, 2022
Merged

Update 03_model_evaluation #2

merged 3 commits into from
Nov 1, 2022

Conversation

marcelcastrobr
Copy link
Collaborator

Changes made are:

  • remove unnecessary folders: docker and code
  • clean up text
  • added suggestion from @corbuno to get s3 paths automatically
  • Extended the evaluation.py to also create the confusion matrix as a.png file.

@marcelcastrobr marcelcastrobr added documentation Improvements or additions to documentation enhancement New feature or request labels Nov 1, 2022
@marcelcastrobr marcelcastrobr self-assigned this Nov 1, 2022
@niklas-palm niklas-palm merged commit f8a92b7 into main Nov 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@niklas-palm niklas-palm niklas-palm approved these changes

@absynthe absynthe Awaiting requested review from absynthe

Assignees

@marcelcastrobr marcelcastrobr

Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@marcelcastrobr @niklas-palm