-
Notifications
You must be signed in to change notification settings - Fork 34
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
Update docs #125
Update docs #125
Conversation
It may depend on merging #124 first. Appreciate if you can review. |
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.
Other than that one comment LGTM.
Merge once addressed?
Codecov Report
@@ Coverage Diff @@
## master #125 +/- ##
=========================================
Coverage ? 90.30%
=========================================
Files ? 12
Lines ? 712
Branches ? 0
=========================================
Hits ? 643
Misses ? 69
Partials ? 0 Continue to review full report at Codecov.
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #125 +/- ##
=========================================
Coverage ? 90.30%
=========================================
Files ? 12
Lines ? 712
Branches ? 0
=========================================
Hits ? 643
Misses ? 69
Partials ? 0 ☔ View full report in Codecov by Sentry. |
This PR updates the docs to use Project.toml and a newer version of Documenter.jl. It also adds a new subsection covering the other losses available that do not fit the distance-based nor margin-based definition.
Fix #111