Skip to content
This repository has been archived by the owner on Aug 2, 2021. It is now read-only.

pss: Use distance metric for test success #1672

Merged
merged 1 commit into from
Aug 20, 2019

Conversation

nolash
Copy link
Contributor

@nolash nolash commented Aug 15, 2019

This PR builds on the introduction of the xor distance metric in the pot. It changes the success criteria of the TestProxNetwork test in pss to use this distance metric.

Is also removes the superfluous Handshake-related code in the test service constructor.

Copy link
Member

@janos janos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look good to me. This PR is labeled still "in progress". Will there be more changes? Could you rebase on master to fix the linting error?

Copy link
Member

@zelig zelig left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

although prox send is promising more I think, namely to be sent to all NNs of all PO-closest node(s)

@nolash
Copy link
Contributor Author

nolash commented Aug 16, 2019

although prox send is promising more I think

yeah this is just the guarantee not the possible collateral effects :)

@nolash nolash merged commit bec590a into ethersphere:master Aug 20, 2019
@nolash nolash deleted the pss-prox-distance-test branch August 20, 2019 15:10
@skylenet skylenet added this to the 0.5.0 milestone Aug 23, 2019
chadsr added a commit to chadsr/swarm that referenced this pull request Sep 23, 2019
* 'master' of github.com:ethersphere/swarm:
  chunk, storage: chunk.Store multiple chunk Put (ethersphere#1681)
  storage: fix pyramid chunker and hasherstore possible deadlocks (ethersphere#1679)
  pss: Use distance to determine single guaranteed recipient (ethersphere#1672)
  storage: fix possible hasherstore deadlock on waitC channel (ethersphere#1674)
  network: Add adaptive capabilities message (ethersphere#1619)
  p2p/protocols, p2p/testing; conditional propagation of context (ethersphere#1648)
  api/http: remove unnecessary conversion (ethersphere#1673)
  storage: fix LazyChunkReader.join potential deadlock (ethersphere#1670)
  HTTP API support for pinning contents (ethersphere#1658)
  pot: Add Distance methods with tests (ethersphere#1621)
  README.md: Update Vendored Dependencies section (ethersphere#1667)
  network, p2p, vendor: move vendored p2p/testing under swarm (ethersphere#1647)
  build, vendor: use go modules for vendoring (ethersphere#1532)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants