Skip to content

This implication should work for any setoid-based permutation #2708

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

Open
onestruggler opened this issue Apr 22, 2025 · 2 comments · May be fixed by #2709
Open

This implication should work for any setoid-based permutation #2708

onestruggler opened this issue Apr 22, 2025 · 2 comments · May be fixed by #2709
Labels

Comments

@onestruggler
Copy link

This implication should work for any setoid-based permutation, not just the equality setoid:

↭⇒↭ₛ : xs ↭ ys xs ↭ₛ ys

@MatthewDaggitt
Copy link
Contributor

Agreed. Not easy to do in a backwards compatible way unfortunately... Might have to be an additional lemma.

@jamesmckinna
Copy link
Contributor

jamesmckinna commented Apr 23, 2025

Hmmm... I think I was originally responsible for these (the one direction requiring the Propositional setoid, the other, correctly as here, being more general, and I didn't have my eye on the ball). As for how we think about this:

  • refactoring rather than, or as well as, addition?
  • breaking; I'd be tempted simply to break the API for v3.0, or even v2.3 as this was an (unintentional) bug, moreover only introduced recently as part of v2.2?

Alternatively, make the simple addition (modulo map id ≐ id, and the trivial proof that id Preserves _≡_ ⟶ _≈_) to Propositional by composing the above lemma with Setoid.Properties.map⁺ instantiated at {f = id}. Hmmm.

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