[ 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 5 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 |
| rawMagma M N = record | ||
| { Carrier = M.Carrier × N.Carrier | ||
| ; _≈_ = Pointwise M._≈_ N._≈_ | ||
| { Carrier = M.Carrier Product.× N.Carrier |
There was a problem hiding this comment.
This, and all subsequent such definitions, could be elided as Carrier never has to be explicitly defined for RawX bundles, because inferrable from _≈_.
So removing these would also 'simplify' these modules, but perhaps at a slightly higher cognitive load wrt intelligbility/legibility?
| infix 4 _≋_ | ||
| _≋_ : Rel (A × B) _ | ||
| _≋_ = Pointwise _≈₁_ _≈₂_ | ||
| _≋_ : Rel (A Product.× B) _ |
There was a problem hiding this comment.
Similarly, here, the carrier of the Rel is inferrable from its definition...
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 have done so.