[ refactor ] name of Data.Product.Relation.Binary.Pointwise.NonDependent.×-decidable#2958
[ refactor ] name of Data.Product.Relation.Binary.Pointwise.NonDependent.×-decidable#2958jamesmckinna wants to merge 8 commits intoagda:masterfrom
Data.Product.Relation.Binary.Pointwise.NonDependent.×-decidable#2958Conversation
|
I agree with your really. Feels like this intermediate step is not needed, we should just go to the 'proper' names. |
|
If we rename |
|
Should it be called |
|
@gallais that's only so far for unary relations, right? This is far more analogous to |
|
OK, thanks for piling in... @Taneb regarding the choice of names, this is (was) intended to be one more chipping away at renaming @gallais your renaming suggestion for
On the first suggestion, I guess that I appreciate @JacquesCarette support for my suggestion, but I'd probably rather fix this up downstream, after we have had a 'proper' discussion on a separate issue? UPDATED: no, let's just do this now. |
|
As to how to import these things qualified, I'd be OK with import ...Product.Base as Product using (...)
import ...Pointwise as Pointwise using (...)and then distinguishing UPDATED: or else, as on |
|
Aaaargh/A-ha: in _-×-_ : (A → B → Set p) → (A → B → Set q) → (A → B → Set _)
f -×- g = f -⟪ _×_ ⟫- gShould we also consider deprecating this? Or generalising it and using this name instead? NB this definition is (essentially) the pullback of |
What generalization do you have in mind? (Definitely should be done in a different PR). |
It may be easier to write and typecheck than to try to explain... ;-) But we do have a function in _-⟨_⟩-_ : (A → C) → (C → D → E) → (B → D) → (A → B → E)which would then be applied at |
| -- Definition | ||
|
|
||
| Pointwise : REL A B ℓ₁ → REL C D ℓ₂ → REL (A × C) (B × D) (ℓ₁ ⊔ ℓ₂) | ||
| Pointwise R S (a , c) (b , d) = (R a b) × (S c d) |
There was a problem hiding this comment.
I'm not sure about this at all. Agree with @Taneb that we probably don't want to conflate the type combinator and the relation combinator.
Yet another item from #2846 .
NB. There's maybe an argument for saying that this should really consist of:
Pointwiseto_×_(because this is 'merely' a lifting toBinaryrelations of the underlyingNullarycombinator)×-decidableto_×?_(in some sense: ditto.)UPDATED: now moved to DRAFT while we figure out the right solution to #2865