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

Support 5G on the Helium network #133

Merged
merged 7 commits into from
Feb 24, 2021
Merged

Support 5G on the Helium network #133

merged 7 commits into from
Feb 24, 2021

Conversation

zer0tweets
Copy link
Contributor

@zer0tweets zer0tweets commented Feb 23, 2021

Copy link

@mfalkvidd mfalkvidd left a comment

Choose a reason for hiding this comment

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

Thanks for introducing this. I really like the concept. Some minor comments to improve the text.


- Corner cases should be dissected by example.
- Cheap prepaid LTE data plans in US sell for $1.50/Gb (Mint Mobile 15Gb bundle)

Choose a reason for hiding this comment

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

Maybe add a reference to this price point? I am unable to find it. The 15GB/month plan costs $45 ($3/GB) when I check https://www.mintmobile.com/plans/

0000-template.md Show resolved Hide resolved
0000-template.md Show resolved Hide resolved

- How will current users be impacted?
- Continuously collect data on the unique number of IMSIs attempting to attach to the PLMN(s) broadcasted by the cell.

Choose a reason for hiding this comment

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

Might need to be clear on how to protect privacy. Depending on juridistiction, IMSIs will be considered personal identifiable information (PII).

@jamiew jamiew changed the title Update-0027-Support-5G-on-Helium-Network Support 5G on the Helium network Feb 24, 2021
@jamiew jamiew added the draft label Feb 24, 2021
Copy link
Contributor

@jamiew jamiew left a comment

Choose a reason for hiding this comment

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

LGTM, merging for discussion! Welcome HIP27

@jamiew jamiew merged commit 4c0b16c into helium:master Feb 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants