Skip to content
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

Find majorities in Spark measurements #59

Closed
8 tasks done
bajtos opened this issue Jan 9, 2024 · 1 comment
Closed
8 tasks done

Find majorities in Spark measurements #59

bajtos opened this issue Jan 9, 2024 · 1 comment
Assignees

Comments

@bajtos
Copy link
Contributor

bajtos commented Jan 9, 2024

eta: 2024-08-31

When evaluating measurements submitted by Spark checker nodes, we should check whether we have enough redundant measurements from different parties to make it safe to assume there is an honest majority.

Measurements belonging to the minority with different retrieval results from the majority should be rejected.

Fields to cross-check:

  • All measurements in the majority should report the same providerId
  • All measurements in the majority should report the same indexerResult
  • All measurements in the majority should report the same provider_address and protocol
  • Response status code
  • CAR size in bytes
  • CAR checksum
  • Was it a timeout?
  • Was the CAR too large?

As part of this work, we need to change the way how we calculate retrieval success rates.

  • At the moment, we measure % of measurements that report a successful retrieval
  • After the change, we should measure % of committees (tasks/deals) where the majority reports a successful retrieval
@bajtos
Copy link
Contributor Author

bajtos commented Sep 2, 2024

I created a new roadmap item for Deal Retrievability Score (#156) so that we can prioritise it against other items in our backlog.

I created a PR to fix the RSR calculation to include minority results, i.e., to restore the algorithm we used before I implemented "committees and majorities" in filecoin-station/spark-evaluate#304:

Once that PR has landed, I'd like to close this roadmap item as done.

cc @juliangruber @patrickwoodhead

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant