-
Notifications
You must be signed in to change notification settings - Fork 376
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
Profunctor should require Contravariant #331
Comments
The diagram in the readme indicates the dependencies that exist between algebra in the specification. Your diagram suggests we should add new dependencies to the specification. Please explain why this would be beneficial. |
No, it is not.
|
You realize that this is open source, right? You're free to fork it and change your fork as you will, or to create your own. You are also free to create your own specification from scratch. If you feel strongly about it, you can then try to convince all those implementing these specs that your own are better and that they should switch. While you're at it, you should probably edit the Wikipedia page on Functors to match your definition and convince the maintainers of languages such as Haskell, Scala, and Clojure to change their specifications to match. Alternatively, you can simply stop trolling. |
The Fantasy Land Functor is short for "Covariant Functor" Profunctor is super useful and I don't think it's specified, that's the only difference to the diagram that I can see. |
Actually, Profunctor is specified. 🎉
Profunctor currently requires Functor but not Contravariant. @Ne4to777, please explain why Profunctor should require Contravariant in addition to Functor. We will certainly consider changing the specification if presented with a compelling reason to do so. :) |
@CrossEye ,
|
This is the question why the derivatives of a covariant functor cannot be derivatives of a contravariant functor? For example contravariant monads.
and it implements the contravariant functor with the second argument |
because:
|
I have updated this issue's title to reflect the change @Ne4to777 is proposing. @fantasyland/core, what do you think of the proposal? |
@davidchambers , aliasing Covariant to Functor is a bad idea too. Especially since we already have an abstract entity - a monad |
I think it should be something like this:
The text was updated successfully, but these errors were encountered: