-
Notifications
You must be signed in to change notification settings - Fork 251
[fixes #1711] Refactoring Data.Nat.Divisibility
and Data.Nat.DivMod
#2182
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
Merged
Merged
Changes from all commits
Commits
Show all changes
91 commits
Select commit
Hold shift + click to select a range
e6c6116
added new definitions to `_∣_`
jamesmckinna 29c3cb7
`CHANGELOG`
jamesmckinna c4804d1
don't declare `quotient≢0` as an `instance`
jamesmckinna a37b99e
replace use of `subst` with one of `trans`
jamesmckinna 4edc8f8
what's sauce for the goose...
jamesmckinna 6d592b3
switch to a `rewrite`-based solution...
jamesmckinna 86a6058
tightened `import`s
jamesmckinna 5e636a8
simplified dependenciess
jamesmckinna 99a121c
simplified dependencies; `CHANGELOG`
jamesmckinna 21b8ca7
removed `module` abstractions
jamesmckinna fdf6b66
delegated proof of `quotient≢0` to `Data.Nat.Properties`
jamesmckinna 305d34b
removed redundant property
jamesmckinna c20e98b
cosmetic review changes; others to follow
jamesmckinna b71b014
better proof of `quotient>1`
jamesmckinna 2405ab9
`where` clause layout
jamesmckinna 60a111e
leaving in the flipped equality; moved everything else
jamesmckinna 8d6cbf6
new lemmas moved from `Core`; knock-on consequences; lots of tidying up
jamesmckinna f1f91a9
tidying up; `CHANGELOG`
jamesmckinna a081792
cosmetic tweaks
jamesmckinna 9ad6bfd
reverted to simple version
jamesmckinna 8fad669
problems with exporting `quotient`
jamesmckinna 3a0d881
added last lemma: defining equation for `_/_`
jamesmckinna f7a9cb6
improved `CHANGELOG`
jamesmckinna 8dc3901
revert: simplified imports
jamesmckinna d05b959
improved `CHANGELOG`
jamesmckinna 33ebce9
endpoint of simplifying the proof of `*-pres-∣`
jamesmckinna a03b552
simplified the proof of `n/m≡quotient`
jamesmckinna 4f438cf
simplified the proof of `∣m+n∣m⇒∣n`
jamesmckinna 2ff5f79
simplified the proof of `∣m∸n∣n⇒∣m`
jamesmckinna a3756c9
simplified `import`s
jamesmckinna efd0b20
simplified a lot of proofs, esp. wrt `divides-refl` and `NonZero` rea…
jamesmckinna 82f37d0
simplified more proofs, esp. wrt `divides-refl` reasoning
jamesmckinna a9b0827
simplified `import`s
jamesmckinna 17ac388
moved `equalityᵒ` proof out of `Core`
jamesmckinna 72c93a8
`CHANGELOG`
jamesmckinna 6d819ae
temporary fix: further `NonZero` refactoring advised!
jamesmckinna 54a4444
regularised use of instance brackets
jamesmckinna 2970fc3
further instance simplification
jamesmckinna 3bbf57f
further streamlining
jamesmckinna 8f2ce92
tidied up `CHANGELOG`
jamesmckinna 4afc16d
simplified `NonZero` pattern matching
jamesmckinna eeee16a
regularised use of instance brackets
jamesmckinna 7fc10e9
simplified proof of `/-*-interchange`
jamesmckinna b2df609
simplified proof of `/-*-interchange`
jamesmckinna b2c0406
... permitting the migration of `*-pres-∣` to `Data.Nat.Divisibility`
jamesmckinna 700aacc
tweaked proof of `/-*-interchange`
jamesmckinna ffed96a
narrowed `import`s
jamesmckinna 053cbc3
Merge branch 'agda:master' into issue1711
jamesmckinna 50bb165
simplified proof; renamed new proofs
jamesmckinna 76c5946
Capitalisation
jamesmckinna 16a14d9
streamlined `import`s; streamlined proof of decidability
jamesmckinna 5503e09
Merge branch 'master' into issue1711
jamesmckinna 7c2d70c
spurious duplication after merge
jamesmckinna 08f8f57
missing symbol import
jamesmckinna 671cb02
replaced one use of `1 < m` with `{{NonTrivial m}}`
jamesmckinna f775e43
fixed `CHANGELOG`
jamesmckinna 36771c5
removed use of identifier `k`
jamesmckinna 44a1f1b
Merge branch 'agda:master' into issue1711
jamesmckinna de4ac18
refactoring: more use of `NonTrivial` instances
jamesmckinna 449513a
knock-on consequences: simplified function
jamesmckinna abff148
two new lemmas
jamesmckinna c7c8c75
refactoring: use of `connex` in proofs
jamesmckinna e4bbcd3
new lemmas about `pred`
jamesmckinna 71dc385
new lemmas about monus
jamesmckinna 5b879bd
refactoring: use of the new properties, simplifying pattern analysis
jamesmckinna 9504548
whitespace
jamesmckinna f7a966f
questionable? revision after comments on #2221
jamesmckinna d83f222
silly argument name typo; remove parens
jamesmckinna f9b3cc0
tidied up imports of `Relation.Nullary`
jamesmckinna 117e6b4
removed spurious `instance`
jamesmckinna 41c2856
localised appeals to `Reasoning`
jamesmckinna e48835f
further use of `variable`s
jamesmckinna a98c02b
tidied up `record` name in comment
jamesmckinna 320aec8
cosmetic
jamesmckinna 6f3946f
reconciled implicit/explicit arguments in various monus lemmas
jamesmckinna feca9d7
fixed knock-on change re monus; reverted change to `m/n<m`
jamesmckinna bb08e6c
reverted change to `m/n≢0⇒n≤m`
jamesmckinna f4e1f53
reverted breaking changes involving `NonZero` inference
jamesmckinna d731717
revised `CHANGELOG`
jamesmckinna fac8e6c
Merge branch 'master' into issue1711
jamesmckinna 5062d3e
restored deleted proof
jamesmckinna ec4c543
fix whitespace
jamesmckinna 032506d
renaming: `DivMod.nonZeroDivisor`
jamesmckinna 850cb99
localised use of `≤-Reasoning`
jamesmckinna 9708ca8
reverted export; removed anonymous module
jamesmckinna 2935137
Merge branch 'master' into issue1711
jamesmckinna ca0bdc8
revert commit re `yes/no`
jamesmckinna e5006e3
renamed flipped equality
jamesmckinna 9838568
tweaked comment
jamesmckinna 70c43b4
added alias for `equality`
jamesmckinna b6f2ada
Merge branch 'master' into issue1711
jamesmckinna File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.