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

Synchronized inventory transactions #308

Merged
merged 23 commits into from
Jun 24, 2022
Merged

Conversation

Avanatiker
Copy link
Member

@Avanatiker Avanatiker commented May 12, 2022

Fully syncs inventory transactions for several purposes:

  1. Syncs all module induced transactions
  2. Syncs player induced transactions
  3. Pauses all Baritone processes

How does it work?

Simple: Only executes inventory transactions after server side confirmation.
Effect: Impossible to create client side item stacks that do not exist.

Best way to build with Baritone on 2b?

NoGhostItems | Baritone pause > Pauses Baritone while doing the transaction
NoGhostItems | Scope ALL > Catches all user or baritone induces transactions and confirms them server side
LagNotifier | Enable > Pauses PlayerInventoryManager on lag
NoGlitchBlocks | Enable > Cancels client side block placement and decreases chance of wrong confirmation
InventoryMove | Item Movement Bypass > Bypasses 2b item transaction restriction while player is moving

- Timeout missing
- Retry function missing
- Impl for Operation.kt missing
@Avanatiker Avanatiker changed the title Syncronized inventory transactions Synchronized inventory transactions May 12, 2022
@Avanatiker Avanatiker marked this pull request as draft May 12, 2022 23:13
@Avanatiker Avanatiker marked this pull request as ready for review May 12, 2022 23:14
@Avanatiker Avanatiker marked this pull request as draft May 12, 2022 23:14
@Avanatiker Avanatiker marked this pull request as ready for review May 14, 2022 13:56
@Avanatiker Avanatiker marked this pull request as draft May 23, 2022 17:02
@Avanatiker Avanatiker force-pushed the syncedPlayerInventoryManager branch from 392ec0e to 07e8e61 Compare May 29, 2022 09:07
@Avanatiker Avanatiker marked this pull request as ready for review June 22, 2022 17:16
@Avanatiker Avanatiker merged commit c048b4f into master Jun 24, 2022
@Avanatiker Avanatiker deleted the syncedPlayerInventoryManager branch June 5, 2023 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant