-
Notifications
You must be signed in to change notification settings - Fork 91
Cleanups in real numbers #1675
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
Cleanups in real numbers #1675
Conversation
src/real-numbers/inequalities-addition-and-subtraction-real-numbers.lagda.md
Show resolved
Hide resolved
|
Have you made any measurements wrt. the performance impact of these changes? One thing that worries me is that, even though Agda may be able to infer certain arguments, doing so may take nontrivial amounts of typechecking time and memory. This for instance led to the recent memory issue #1577. Though, in that case nothing was marked |
src/real-numbers/saturation-inequality-nonnegative-real-numbers.lagda.md
Outdated
Show resolved
Hide resolved
src/real-numbers/strict-inequalities-addition-and-subtraction-real-numbers.lagda.md
Show resolved
Hide resolved
src/real-numbers/strict-inequalities-addition-real-numbers.lagda.md
Outdated
Show resolved
Hide resolved
src/real-numbers/strict-inequalities-addition-real-numbers.lagda.md
Outdated
Show resolved
Hide resolved
src/real-numbers/strict-inequalities-addition-real-numbers.lagda.md
Outdated
Show resolved
Hide resolved
fredrik-bakke
left a comment
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.
Conditional approval, depending on that the performance is not drastically worse than before.
Co-authored-by: Fredrik Bakke <fredrbak@gmail.com>
I have not measured it, but the overwhelming majority of users of these functions were already passing |
Co-authored-by: Fredrik Bakke <fredrbak@gmail.com>
…da.md Co-authored-by: Fredrik Bakke <fredrbak@gmail.com>
…da.md Co-authored-by: Fredrik Bakke <fredrbak@gmail.com>
…s.lagda.md Co-authored-by: Fredrik Bakke <fredrbak@gmail.com>
|
A clean build on my machine took 30 seconds less after this PR for some unclear reason, so I'm pretty confident this doesn't do anything disastrous. |
|
FYI we have the |
That's amazing, great work! |
I started out wanting to make some parameters implicit, now that with
opaquedefinitions they can reliably be inferred from the context.Then things snowballed from there to include a bunch of #1545, use of
abstract opaque, and elimination of redundant imports.