Skip to content

Re-compute correct timeslot on applyModemConfig#4469

Merged
thebentern merged 2 commits into
2.5-changesfrom
compute-timeslot
Aug 15, 2024
Merged

Re-compute correct timeslot on applyModemConfig#4469
thebentern merged 2 commits into
2.5-changesfrom
compute-timeslot

Conversation

@thebentern

Copy link
Copy Markdown
Contributor

Per our conversation in contributors room in Discord
Closes #3892

@GUVWAF

GUVWAF commented Aug 15, 2024

Copy link
Copy Markdown
Member

Looks good, but we might want to decrease CWmax then also. Currently the absolute maximum delay on LongFast before rebroadcasting is 2^8 * slotTimeMsec = 10.7 seconds, which will become 19.7 seconds. Maybe we can decrease CWmax to 7 such that we get 9.86 seconds?

@thebentern

Copy link
Copy Markdown
Contributor Author

Looks good, but we might want to decrease CWmax then also. Currently the absolute maximum delay on LongFast before rebroadcasting is 2^8 * slotTimeMsec = 10.7 seconds, which will become 19.7 seconds. Maybe we can decrease CWmax to 7 such that we get 9.86 seconds?

See this is why I added you :-)

@thebentern thebentern merged commit 6f1dae1 into 2.5-changes Aug 15, 2024
@thebentern thebentern deleted the compute-timeslot branch August 15, 2024 20:05
jeek pushed a commit to jeek/Meshtastic-Exploiteers-Hacker-Pager that referenced this pull request Jun 30, 2026
* Re-compute correct timeslot on applyModemConfig

* Cap contention window max at 7
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants