-
Notifications
You must be signed in to change notification settings - Fork 248
[ add ] product structure on RawSetoid
#2720
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
base: master
Are you sure you want to change the base?
Conversation
Don't really understand why the tests are failing, but seems to be a GitHub problem, rather than at my end? Ah, I'd missed the updates to the CI... |
If anyone has any insight into why all the tests are (still!) failing on GH, please let me know! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Commented out code?
The CI failure is just for ghc-9.8.4 and seems to be related to ghcup-0.0.9. Seems related to what @andreasabel did with the CI? The exact details are:
|
This spells out as "too many requests". Seems like a github rate limit is hit here.
Not really. This workflow is autogenerated by It is also safe to ignore sporadic failures in this workflow since it does not test the integrity of the standard library, just the tools |
Last last thought: should this be parametrised on |
Ah, the perennial (un)bundling question. The biggest advantage of the unbundled version is that having a shared carrier of two setoids becomes easy. Having to bundle obscures things in that case and downstream uses might then struggle to recover that information. That is, however, a pure hypothetical. However, as you mention, |
Will refactor to unbundle...
Easy enough to make the whole thing fully unbundled, but I had a slight wobble about whether bundling the implicit parameters as |
Lifts out the relevant constructions on
RawSetoid
from #2715 which is now marked blocking on this PR.NB.
IsRelHomomorphism
Pointwise
product ofRawSetoid
s be lifted out (trivially) toRelation.Binary.Construct.Product
? (see discussion with/comments from @JacquesCarette below...)