-
Notifications
You must be signed in to change notification settings - Fork 374
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
Another go at updating dependencies #192
Changes from 3 commits
7b465f1
63016be
1b0208d
531b881
58647bd
4c1e3ee
6900fb8
1ee2e5d
dca905c
c840121
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -25,7 +25,7 @@ structures: | |
* [Bifunctor](#bifunctor) | ||
* [Profunctor](#profunctor) | ||
|
||
<img src="figures/dependencies.png" width="677" height="212" /> | ||
<img src="https://cdn.rawgit.com/fantasyland/fantasy-land/master/figures/dependencies.svg" /> | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is inadvisable:
The approach I used with Orthogonal was to add the SVG in one commit then reference its location by commit hash in the following commit. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Ah! Makes sense. |
||
|
||
## General | ||
|
||
|
@@ -210,7 +210,7 @@ method takes one argument: | |
|
||
3. `ap` must apply the function in Apply `b` to the value in | ||
Apply `a` | ||
|
||
1. No parts of return value of that function should be checked. | ||
|
||
### Applicative | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,35 @@ | ||
digraph { | ||
node [fontcolor=blue;shape=plaintext] | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Let's |
||
|
||
# Algebras | ||
Applicative [URL="#applicative"]; | ||
Apply [URL="#apply"]; | ||
Bifunctor [URL="#bifunctor"]; | ||
Chain [URL="#chain"]; | ||
ChainRec [URL="#chainrec"]; | ||
Comonad [URL="#comonad"]; | ||
Extend [URL="#extend"]; | ||
Foldable [URL="#foldable"]; | ||
Functor [URL="#functor"]; | ||
Monad [URL="#monad"]; | ||
Monoid [URL="#monoid"]; | ||
Profunctor [URL="#profunctor"]; | ||
Semigroup [URL="#semigroup"]; | ||
Setoid [URL="#setoid"]; | ||
Traversable [URL="#traversable"]; | ||
|
||
# Dependencies | ||
Applicative -> Monad; | ||
Apply -> Applicative; | ||
Apply -> Chain; | ||
Chain -> ChainRec; | ||
Chain -> Monad; | ||
Extend -> Comonad; | ||
Foldable -> Traversable; | ||
Functor -> Apply; | ||
Functor -> Bifunctor; | ||
Functor -> Extend; | ||
Functor -> Profunctor; | ||
Functor -> Traversable; | ||
Semigroup -> Monoid; | ||
} |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,4 @@ | ||
#!/bin/sh | ||
|
||
dot -Tsvg -odependencies.svg dependencies.dot | ||
dot -Tpng -odependencies.png dependencies.dot |
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.
Let's remove this list. It's duplicative now that the diagram is up to date. :)
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.
I think it's best to keep this. It's nice to have a fallback in case something happens with the image, different people process information differently, and it's still plaintext so it can be searched.
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.
Two points in favour of the ASCII version. 😜
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.
😄
I'm fine with the output graph being ASCII. I just don't want to maintain the shape of the graph in raw ASCII.
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.
I'm totally with you. A program like
dot
which generates an ASCII diagram would be neat!I very much like your change. Thanks for taking the time to work on it.
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.
It does support multiple output formats, including
plain
.plain
gives you the size of the graph, a sequence of nodes with positions, and a sequence of edges with position if you want to parse the coordinates into an ascii thing. It looks like a regular language, so probably not too hard to parse.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.
Cool! I may investigate that one day (but certainly not today).