Skip to content

[ fix #1354 ] Refactoring Permutation.Propositional #1761

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

Draft
wants to merge 16 commits into
base: master
Choose a base branch
from

Conversation

gallais
Copy link
Member

@gallais gallais commented May 3, 2022

Still missing: CHANGELOG entries

@MatthewDaggitt
Copy link
Contributor

Closing as we've decided that its better to wait for typed pattern synonyms to finally make an appearance.

@jamesmckinna
Copy link
Contributor

jamesmckinna commented Mar 17, 2024

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 Permutation.*.Properties modules along the lines considered in #2317 ... incoming revised/streamlined/refactored/rebased commits shortly.

UPDATED: as with that more recent PR, will convert to DRAFT for now until ready to review.

@jamesmckinna jamesmckinna reopened this Mar 17, 2024
jamesmckinna and others added 6 commits March 17, 2024 12:25

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature.
@jamesmckinna jamesmckinna marked this pull request as draft March 17, 2024 15:45
Copy link
Contributor

@JacquesCarette JacquesCarette left a 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 odd name (which seems to actually be something pre-existing), this is looking quite good.

@MatthewDaggitt MatthewDaggitt removed the status: won't-merge Decided against merging the PR in. label Mar 18, 2024
@jamesmckinna
Copy link
Contributor

jamesmckinna commented Mar 18, 2024

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 Properties unified as well. If that succeeds, then most of the guff in Propositional disappears entirely, and with it, all/most of the angst about qualified import of refl... I hope. Fingers crossed!

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.

@jamesmckinna
Copy link
Contributor

Removing the milestone marker from this for the time being...

@JacquesCarette
Copy link
Contributor

So what needs to be done to push this to the finish line? [Other than a likely rather nasty merge!]

@JacquesCarette
Copy link
Contributor

I might have a student who could push this to the finishing line, if it was clear what needed to be done - @jamesmckinna ?

@jamesmckinna
Copy link
Contributor

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

@MatthewDaggitt
Copy link
Contributor

My understanding of the argument was that we wanted to eventually deprecate Permutation in favour of the two new definitions. Which would mean re-proving all the existing results for both variants. This probably would be a good student project.

@jamesmckinna
Copy link
Contributor

My understanding of the argument was that we wanted to eventually deprecate Permutation in favour of the two new definitions. Which would mean re-proving all the existing results for both variants. This probably would be a good student project.

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 Homogeneous properties may still be useful, but less so now than before? I suspect any end result will also end up being v3.0 because breaking...

@MatthewDaggitt
Copy link
Contributor

I suspect any end result will also end up being v3.0 because breaking...

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reimplement Permutation.Propositional in terms of Permutation.Setoid
4 participants