[ fix #1354 ] Refactoring Permutation.Propositional#1761
[ fix #1354 ] Refactoring Permutation.Propositional#1761gallais wants to merge 16 commits intoagda:masterfrom
Conversation
|
Closing as we've decided that its better to wait for typed pattern synonyms to finally make an appearance. |
|
Suggest re-opening this, as I seem to have got it to work with the pattern synonyms... using some ideas from #2317 and #2321 towards further unification, if someone can help me unify the two UPDATED: as with that more recent PR, will convert to DRAFT for now until ready to review. |
JacquesCarette
left a comment
There was a problem hiding this comment.
Other than that one odd name (which seems to actually be something pre-existing), this is looking quite good.
|
So far all the heavy lifting was done years ago by @gallais so props to him. But I think that my recent work on #2317 hopefully points the way to getting the UPDATED Lots of refactoring work on #2317 ahead of bringing it to bear on this PR. But first see #2333 as a more manageable first part of a roadmap detailed there. |
|
Removing the milestone marker from this for the time being... |
|
So what needs to be done to push this to the finish line? [Other than a likely rather nasty merge!] |
|
I might have a student who could push this to the finishing line, if it was clear what needed to be done - @jamesmckinna ? |
See my comment on #2317 |
|
My understanding of the argument was that we wanted to eventually deprecate |
Happy to consult/supervise/review if there is a willing candidate. There may well be more we can usefully do, esp. eg. wrt names of lemmas, as well as the overall structure. The factorisation out of |
Yes, the big switch over will be, but we extend the API for the two new modules in a non-breaking way in 2.4. |
|
@gabriellisboaconegero this might be a more straightforward project than some you are on right now, i.e. reproving many properties of permutations given using different definitions, than some of the things you're currently working on. |
Still missing: CHANGELOG entries