Ecosyste.ms: OpenCollective
An open API service for software projects hosted on Open Collective.
github.com/sourcecred/odyssey-hackathon
Repo for working on our Odyssey Hackathon project
https://github.com/sourcecred/odyssey-hackathon
The implementation is quite simple. The tests are somewhat more
comprehensive than in v2 or v1....
It turns out we forgot to add this to the API, so I added it. I also
implemented it. The tests ...
This is the change that puts the Graph into `Graph` :) We add `_inEdges`
and `_outEdges`, and u...
Summary:
A client of `Graph` is able to (e.g.) invoke `nodes()` to get a node
iterator, iterat...
Adding a conflicting edge (i.e. one with the same address, but different
`src` or `dst`) is an ...
d90cd3cf1b5d7832398e25bd0c6fa84bdf4b674f authored over 6 years ago by Dandelion Mané <[email protected]>
Summary:
We’re not mandating anything about coverage right now, but by making it
easier to tra...
Test Plan:
Run `yarn test --env=jsdom --coverage`, and note that files are
generated into `cov...
Summary:
These are superseded by the unified implementation in `address.js`.
Test Plan:
Exi...
Summary:
This is basically a textual substitution.
Test Plan:
Existing unit tests suffice. ...
Test Plan:
Unit tests added. Run `yarn travis`.
wchargin-branch: address-hasprefix
fe1dd326ed34810ef5d1e63b6609e6961ecf9b17 authored over 6 years ago by William Chargin <[email protected]>
Test Plan:
Unit tests added. Run `yarn travis`.
wchargin-branch: address-append
083bd59514815b2edff0963d8c7d334eaa34e55e authored over 6 years ago by William Chargin <[email protected]>
Test Plan:
Unit tests added. Run `yarn travis`.
wchargin-branch: address-tostring
0bf1ef8af24b96d8e8da0628ed7568dd8a60ac7e authored over 6 years ago by William Chargin <[email protected]>
Summary:
This implements the following functions for the unified addresses:
- assertions: ...
32aba15b01f8db9e3bce0918871f92ef161b03fc authored over 6 years ago by William Chargin <[email protected]>
Summary:
This commit implements all the code needed for the top-level
`makeAddressModule` func...
Summary:
We have `NodeAddress` and `EdgeAddress`, which are opaque aliases of
`string` each wi...
Summary:
In actual code, we almost always call `neighbors` with a specified
direction. Usually...
Summary:
This saves clients from having to pollute their global namespace with
`IN` and `OUT` ...
Summary:
These functions can be used for address filtering: finding all nodes
owned by a plugi...
Summary:
This fixes an organizational error on my part: the assertions were not
in `it`-blocks...
On larger repos (e.g. `ipfs/js-ipfs` and `ipfs/go-ipfs`) our GitHub
query tends to fail with an...
This commit implements the following edge related methods on graph:
- `Graph.addEdge(edge)`
...
Summary:
If you just print out an address, depending on output context the NUL
separators may ...
Summary:
Now, a client calling `assertNodeAddress` can indicate the context to
clients, like `...
Summary:
Using `throw err;` was not useful when the error’s message was an
object: `[object Ob...
Summary:
For now, this contains the logic to register an `unhandledRejection`
error. I’ve remo...
Right now, the cred explorer attempts to display every node in the
graph. As graphs easily grow...
Summary:
A `RecursiveTable` shows the tree-view from a single node; a
`RecursiveTables` shows ...
Our GitHub renderer tries to display the author name for a comments or
pull request reviews. Ho...
Consider a case where a user wants to construct many addresses sharing a
common prefix. For exa...
When the cron build fails, we want to make sure that we know about it.
This commit ensures that...
This commit implements and tests the following methods on the `Graph`:
- `addNode`
- `removeNo...
Summary:
Closes #336.
Test Plan:
Snapshots updated; changes are easily readable. Existing t...
It still passes :)
Test plan: Travis
6585700c0cc424f4ea3620dffb671a2ecad957de authored over 6 years ago by Dandelion Mané <[email protected]>
Summary:
Fixes breakage due to https://github.com/sourcecred/sourcecred/pull/327.
Test Plan:...
3e835764223c5c0c19d42112084f5bd53be25edf authored over 6 years ago by William Chargin <[email protected]>Release notes: https://prettier.io/blog/2018/05/27/1.13.0.html
3ac051b16ce84e4f4dbc37fe95a182fdbd96e13f authored over 6 years ago by Dandelion Mané <[email protected]>
This commit re-organizes our addressing code according to the following
conventions:
- All add...
This very minor commit adds the basic data structures to the `Graph`
that will act as underlyin...
This commit implements Node/Edge addresses, and helper functions for
generating and manipulatin...
The newly added `v3/src/core/graph.js` module is the heart of our `v3`
refactor, which aims at ...
We want to reset some of our basic assumptions, and make `Graph` into a
pure graph implementati...
It was not used anywhere.
Test plan: Travis
Paired with @wchargin
797a2fbf9f0ecb5e1fc9d0bc60d1d940c14b9242 authored over 6 years ago by Dandelion Mané <[email protected]>
Summary:
This fixes a bug introduced in #317, which only occurred in the cron job
variant of t...
Summary:
The storage format is similar to the V1 storage format. Nodes are sorted
by address a...
Summary:
A `Node` includes a ref and a payload. We should say that two nodes at
the same addre...
Summary:
Both the implementation and the tests here are pretty straightforward.
The only chang...
Summary:
I dropped the `edgeDecomposition` and `neighborhoodDecomposition` test
cases, because...
The implementation is adapted from our previous implementation, but has
been refactored to be m...
Summary:
Based on code originally paired with @decentralion.
Test Plan:
Unit tests added. R...
Summary:
Previously, our CI script would run `yarn backend`, which has the
side-effect of eras...
Given that we have reified the type `PluginFilter` as a consistent way
to filter nodes and edge...
As of this commit, adding the comment `//$ExpectFlowError` in flow-typed
code asserts that the ...
Summary:
It’s critical, even more so than usual, that the “base reference”
property of a `Dele...
Summary:
The use of `tee /dev/stderr` failed when running as a child process
under Node for so...
This commit adds the following methods to the `Graph`:
* `addNode`
* `removeNode`
* `ref`
...
Test Plan:
Run `yarn flow stop; yarn travis | cat` and note the absence of the
really long lin...
Currently, when filtering by type (e.g. in neighbors), we require only a
type string. This is a...
- Add the `plugins` method to graph, and have the constructor take them
- Add in demo `PluginHa...
Summary:
In fact, a constructor that only calls `super` is useful: it specifies
the API for co...
Also, rename `NodeDelegateReference` to `DelegateNodeReference` in the
design doc.
Test plan...
630a6d6532ab5a13d3431bd9160197b1ded18421 authored over 6 years ago by Dandelion Mané <[email protected]>
src/core2/graph.js has the types and Graph class scaffold as
described in address_payload_unifi...
Since this document was originally written, we've iterated on the design
a bit. @wchargin and I...
In #190, @wchargin lays out an ambitious proposal for refactoring away
the graph's address-payl...
Given that we are undergoing a major world-changing refactor (#190), all
outstanding code needs...
Summary:
Placing `STOPSHIP` or `stopship` (or any case variant) in any file
tracked by Git wil...
Summary:
This just slows down commits by a few seconds. We `check-pretty` in
Travis, so this d...
Summary:
Improves the result of the preceding commit to strip out cruft. Paired
with @decentra...
Summary:
These tests have apparently been borked since the beginning. They were
testing proper...
Summary:
We now use the “advanced” graph everywhere. Happily, all tests still
pass!
Startin...
853518fd60242624c31fb1530dbe682c54542ee1 authored over 6 years ago by William Chargin <[email protected]>
Summary:
We had three graph merging functions: `merge`, `mergeConservative`, and
`mergeManyCon...
As previously implemented, the Graph was polymorphic in its NodePayload
and EdgePayload. This w...
Summary:
See #190, and discussion on this PR, for context.
wchargin-branch: address-payload-...
e20b794edcd9d101897fa3181b3d78b0bc1709b1 authored over 6 years ago by William Chargin <[email protected]>
Summary:
This is an implementation-only, API-preserving change to the `Graph`
class. Edges’ `s...
This commit adds explicit versioning to the `Graph` and `AddressMap`
JSON representations, usin...
This commit adds the `src/util/compat` module, which is responsible for
data compatibility. It ...
Will allow us to use opaque types in #292 without breaking the build
in #293.
Test plan:
If...
`sourcecred graph` tends to die due to lack of heap during the Git
plugin. Node defaults to ~1....
`markovChain.findStationaryDistribution` is currently written such that
it blows the function c...
See #286 for context.
I also upgraded client code in src/app.
Test plan:
Unit tests are e...
See #286 for context. There are a few miscellaneous changes in
src/app/credExplorer to change c...
`NodeReference` and `NodePorcelain` act as abstractions over the two
states a Node can be in.
...
Summary:
A few changes were made to code that is correct (as far as I can tell),
but for which...
Summary:
Instead of having one function that returns a union, we present two
functions, each o...
This commit adds "Node Porcelain" for the Git plugin. Node porcelain is
a wrapper over a graph ...
Test Plan:
Unit tests added. Run `yarn test`.
wchargin-branch: extract-findStationaryDistrib...
115d7f3921c5145180b472f1678edaf804d0973f authored over 6 years ago by William Chargin <[email protected]>
Summary:
There are substantive options for `convergenceThreshold` and
`maxIterations`, as well...
Test Plan:
Unit tests added. Run `yarn test`.
wchargin-branch: extract-sparseMarkovChainAction
0a608acbfff607b8cb96627494a3d4c1fe81cbb4 authored over 6 years ago by William Chargin <[email protected]>
Test Plan:
Unit tests added. Run `yarn test`.
wchargin-branch: extract-uniformDistribution
69b9f6657dfebb56c0563d8bf4cc2969c6343dea authored over 6 years ago by William Chargin <[email protected]>
Summary:
This commit slightly reorganizes the internals of `basicPagerank` to use
the `SparseM...
Summary:
This function is mostly useful for easily describing Markov chains in
test cases.
...
e5472752acc4e5e76bf5a34b594f546573431711 authored over 6 years ago by William Chargin <[email protected]>
Summary:
For now, this module has just two types: `Distribution` and
`SparseMarkovChain`. We’l...
Summary:
See #66 for more context. This yields the following performance
improvements for me, ...
Summary:
This takes `AddressMap` access, and therefore JSON stringification, off
the critical ...
Summary:
We’re not convinced that using TFJS at this time is worth it, for two
reasons. First,...
Test plan: No change, it's just a variable rename.
Paired with @wchargin
2f0f5230658f81d62e508b7cc1a2d9d308d293fa authored over 6 years ago by Dandelion Mané <[email protected]>Test plan: Open the cred explorer, and expand some nodes.
Paired with @wchargin
2a52ff85f8ba77208863656e9702bef58a6fa2f6 authored over 6 years ago by Dandelion Mané <[email protected]>Now the node description is first.
Test plan: Observe that the behavior is unchanged, except fo...
6714d4b95ec4babe049b977ea0dae24e3c4ad818 authored over 6 years ago by Dandelion Mané <[email protected]>
We were sorting low-to-high, and then reversing. We can just sort
high-to-low.
Test plan: No be...
880b0099e960fb01f5dea199eeb87877d2ec1a97 authored over 6 years ago by Dandelion Mané <[email protected]>
Test plan: Open the cred explorer, and try clicking the + signs. They
will expand a recursive ta...
Test plan: Open the cred explorer, and use the + sign to expand the
neighbors. Observe that thos...