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

HIP 50: Display All Potential Beacon Witnesses #331

Closed
hiptron opened this issue Jan 24, 2022 · 9 comments
Closed

HIP 50: Display All Potential Beacon Witnesses #331

hiptron opened this issue Jan 24, 2022 · 9 comments

Comments

@hiptron
Copy link
Collaborator

hiptron commented Jan 24, 2022

rendered view

https://github.com/helium/HIP/blob/master/0050-display-all-potential-beacon-witnesses.md

summary

Under the current Proof Of Coverage (POC) model, for each beacon sent, current tools / reporting mechanisms allow interested parties to monitor only those witnessing hotspots which have been randomly selected as potentially eligible to receive rewards. The details of other hotspots which could have successfully witnessed said beacon but were excluded by that random selection process remain unreported and inaccessible to hotspot owners

This can, especially during periods of sub-optimal network performance, make it difficult for hotspot owners to know if their hotspot is performing poorly in terms of witnessing beacons, or if they are just 'unlucky' with regard to the random selection process in effect. This in turn can prove an obstacle to hotspot owners wishing to optimise their installations for coverage and rewards. I propose a change to reporting, to display all potential witnesses regardless of the results of the random selection process, to provide greater clarity and confidence for hotspot owners seeking to optimise their installations.

@hiptron
Copy link
Collaborator Author

hiptron commented Jan 24, 2022

@captainhindsite apologies again for delay getting this merged. I've also updated title to clarify that this would require recording all witnesses in order to display them

@captainhindsite
Copy link
Contributor

captainhindsite commented Jan 24, 2022 via email

@captainhindsite
Copy link
Contributor

captainhindsite commented Jan 24, 2022 via email

@hiptron
Copy link
Collaborator Author

hiptron commented Jan 24, 2022 via email

@jamiew jamiew changed the title HIP 50: Record All Potential Beacon Witnesses HIP 50: Display All Potential Beacon Witnesses Jan 24, 2022
@jamiew
Copy link
Contributor

jamiew commented Jan 24, 2022

I'm renaming back to the original title, Display All Potential Beacon Witnesses

@abhay
Copy link
Contributor

abhay commented Jan 27, 2022

This is a potentially unbounded addition of data to the chain without obvious benefit to all participants.

If the purpose of this is diagnostic, I'd consider encouraging diagnostic tools being added to manufacturer firmware that allows for collection of PoC packets and "purchased" via Helium Routers. These Routers could be owned by individual Hotspot owners or by manufacturers and provided as a service (somewhat like Discovery mode). This would both encourage usage of the network and get the data you're looking for.

@jamiew
Copy link
Contributor

jamiew commented Jan 27, 2022

@abhay what do you think of the old idea of a listen-only gossip stream from the validator group? This was mentioned in discord channel again. From what I understood that would allow for general auditing as well as enable a third-party to capture data for something like this

@vincenzospaghetti
Copy link
Contributor

@captainhindsite This HIP is resolved by HIP 70. Can we close this HIP? Please confirm or if I don't hear from you, I will go ahead and close it at the end of the week. Congrats on submitting your HIP and contributing to the community!

@captainhindsite
Copy link
Contributor

captainhindsite commented Oct 18, 2022 via email

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

5 participants