Skip to content
This repository has been archived by the owner on Jun 12, 2023. It is now read-only.

Update poc channel for EU #613

Closed
wants to merge 1 commit into from
Closed

Conversation

amirhaleem
Copy link
Member

Allows for 500mW PoC transmissions in EU868. Must be merged with helium/sx1302_hal#13

@Vagabond
Copy link
Contributor

Vagabond commented Feb 9, 2021

I think this will break the PoC "channel" stuff:

https://github.com/helium/blockchain-core/blob/master/src/transactions/v1/blockchain_txn_poc_receipts_v1.erl#L326-L329

as we assume PoC packets are uniformly distributed across the 8 uplink channels right now.

@amirhaleem
Copy link
Member Author

I see the issue. not sure how best to solve this without exposing region in core?

@mfalkvidd
Copy link

mfalkvidd commented Feb 24, 2021

This suggestion would force all LoRaWAN devices on the network to use a single channel. That would have a huge impact on the overall capacity.

If LoRaWAN devices do not use this single channel, PoC in EU would become very artificial, since it uses much more power than devices will.

@timcooijmans
Copy link
Contributor

Please don't do this: As @mfalkvidd mentions, it will give a unrealistic PoC indication while actual coverage will be much less (being limited by 25mW instead of 500mW).

@madninja madninja closed this Oct 18, 2021
@madninja madninja deleted the ah/update-eu-poc-channels branch October 18, 2021 14:48
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants