Skip to content
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

Last sprint's deliverables #237

Merged
merged 15 commits into from
Feb 7, 2024
Merged

Last sprint's deliverables #237

merged 15 commits into from
Feb 7, 2024

Conversation

ultiwinter
Copy link
Contributor

@ultiwinter ultiwinter commented Feb 4, 2024

Last sprint's deliverables

  • User documentation
  • Design documentation
  • Build documentation
  • Planning documentation
  • Added conclusion and updated classifier comparison

@ultiwinter ultiwinter linked an issue Feb 4, 2024 that may be closed by this pull request
@ultiwinter ultiwinter added the documentation Improvements or additions to documentation label Feb 4, 2024

# Conclusion

In summary, XGBoost consistently demonstrated superior performance, showcasing robust results across various splits and subsets. However, it is crucial to note that its elevated score is attributed to potential overfitting on the XS class. Given SumUp's emphasis on accurate predictions for higher classes, we recommend considering LightGBM. This model outperformed XGBoost in predicting the XL class and the other classes, offering better results in both the five-class and three-class splits.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice summary.

Copy link
Contributor

@Tims777 Tims777 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good work overall. Please just provide me with the source files of the compiled PDFs, because I would like to update the underlying Wiki.

Deliverables/sprint-13/build-documentation.pdf Outdated Show resolved Hide resolved
Deliverables/sprint-13/design-documentation.pdf Outdated Show resolved Hide resolved
Deliverables/sprint-13/planning-documents.pdf Outdated Show resolved Hide resolved
Deliverables/sprint-13/user-documentation.pdf Outdated Show resolved Hide resolved
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice overview

@ultiwinter ultiwinter merged commit 31bdcae into main Feb 7, 2024
4 checks passed
@ultiwinter ultiwinter deleted the documentation/sprint-13-ahmed branch February 7, 2024 09:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Finalize documentation
5 participants