Skip to content

chore: RTP improvements#142

Open
Thatsmusic99 wants to merge 4 commits intov6from
chore/rtp-improvement
Open

chore: RTP improvements#142
Thatsmusic99 wants to merge 4 commits intov6from
chore/rtp-improvement

Conversation

@Thatsmusic99
Copy link
Collaborator

Information

Improves the existing RTP system, with a focus on addressing issues with pre-loaded coordinates that sit outside world borders.

  • Renames use-vanilla-border and use-plugin-borders to sync-vanilla-border and sync-plugin-borders respectively.
  • The sync options "sync" AT's RTP border limits to their respective borders, i.e. if someone changes the world border radius, then AT will generate coordinates that adhere to the full extent of the border.
  • If a border change is noticed (through /at reload or the world border events), then AT will check that each prepared location sits inside the new border. If they don't, new locations are rolled.
  • Regardless of whether syncing is enabled, AT will always shrink its coordinate range in response to other borders, i.e. if the Vanilla border is smaller than the configured range set by AT, all the coordinates generated use the Vanilla border as a reference.
  • Improved naming conventions across some portions of the code that RTP uses so it is clearer to outside users.

Prior to Merging

  • Does the PR need documenting on the wiki? If not, remove this, otherwise, leave this unchecked until a respective
    PR has been made for the wiki.
  • Has the PR been tested?
  • Do you consent to GitHub Copilot also reviewing your changes? An actual developer will review your changes first.

Signed-off-by: Thatsmusic99 <25277367+thatsmusic99@users.noreply.github.com>
Signed-off-by: Thatsmusic99 <25277367+thatsmusic99@users.noreply.github.com>
Signed-off-by: Thatsmusic99 <25277367+thatsmusic99@users.noreply.github.com>
…etched

Signed-off-by: Thatsmusic99 <25277367+thatsmusic99@users.noreply.github.com>
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.

1 participant