-
Notifications
You must be signed in to change notification settings - Fork 13
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 comparisons and concept checks #29
Conversation
Codecov ReportPatch coverage:
Additional details and impacted files@@ Coverage Diff @@
## main #29 +/- ##
==========================================
+ Coverage 94.98% 95.21% +0.22%
==========================================
Files 4 4
Lines 459 481 +22
==========================================
+ Hits 436 458 +22
Misses 23 23
☔ View full report in Codecov by Sentry. |
These changes ended up coupled as <=> needs concept checks. For CI, use AppleClang 14 and Xcode 14 to get concept support
5c01f9e
to
bc79e81
Compare
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.
Approved, but we should add the completeness tests and update the table and expected behavour for the paper
Added discussion of design and compromises. Note: Missing comparison and hash for indirect as their inclusion needs a PR (#29) to be merged.
Remove duplicated member function.
Fixes #20
Progresses #21