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

HIP21: PoC Link Layer Upgrades #78

Closed
jamiew opened this issue Nov 16, 2020 · 3 comments · Fixed by #276
Closed

HIP21: PoC Link Layer Upgrades #78

jamiew opened this issue Nov 16, 2020 · 3 comments · Fixed by #276

Comments

@jamiew
Copy link
Contributor

jamiew commented Nov 16, 2020

Author(s): @refugeesus
Initial PR: #76
Start Date: 2020-11-09
Category: Technical

Rendered view:
https://github.com/helium/HIP/blob/master/0021-poc-link-layer.md

Summary:

The goals of these changes are to:

  1. Break up the PoC message to improve the reliability of PoC transmissions.
  2. Improve the quality of information gained from PoC transmissions in the RF layer.

The link layer is the method by which information is send from one source to another. It sits directly above the PHY layer and below the MAC layer. For a specific definition, please see the OSI model.

@refugeesus
Copy link
Contributor

Per CC, here are the ideal/robust soliton distribution implementations used in the underlying fountain coding:
https://github.com/refugeesus/statrs/tree/rfk/soliton

Here is the implementation of the fountain coding itself:
https://github.com/refugeesus/fountain/tree/rfk/ldpc

Aside from people wanting to run tests for the experience of doing it, someone could check my math on the statrs library to make sure I'm not missing some small piece. I have PR'd to that library if nobody is up for it, so maybe they will look through it a few times before we get to. The tests show the implementation working so I am inclined to believe it is correct.

@abhay
Copy link
Contributor

abhay commented Sep 6, 2021

Propose closing this as stale @jamiew?

jamiew added a commit that referenced this issue Sep 14, 2021
@jamiew jamiew closed this as completed Sep 14, 2021
jamiew added a commit that referenced this issue Sep 14, 2021
@jamiew jamiew reopened this Sep 14, 2021
@jamiew jamiew closed this as completed Sep 14, 2021
@jamiew
Copy link
Contributor Author

jamiew commented Sep 14, 2021

Sorry for close/open/close, bad automation

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

Successfully merging a pull request may close this issue.

3 participants