Skip to content

Conversation

@hexadec1mal
Copy link

@hexadec1mal hexadec1mal commented Mar 3, 2025

  1. Railing setting fix, it works now
  2. Slight block break/place delay decrease
  3. Common method to exit with or without disconnect
  4. Baritone pathing
  5. Ender chest mining and obsidian collection with baritone
  6. Paving position don't depend on player position
  7. Some code restructure and cleanup
  8. Removed center and ender chest blockade states
  9. Github workflows fix
  10. Gradle and fabric-loom update

movement with baritone
obsidian collection with baritone
paving position don't depend on player position

some code restructure and cleanup
removed center and ender chest blockade states
@hexadec1mal hexadec1mal changed the title Fixes and optimizations Fixes, optimizations and baritone pathing Mar 5, 2025
@RedCarlos26 RedCarlos26 self-assigned this Mar 10, 2025
@RedCarlos26 RedCarlos26 added the question Further information is requested label Mar 10, 2025
@RedCarlos26 RedCarlos26 marked this pull request as draft March 10, 2025 01:41
@RedCarlos26
Copy link
Owner

Please apply the reviewed changes, made as a draft for now, I will test it more before releasing.

@hexadec1mal
Copy link
Author

On task change now tick() is called immediately after start(). Because start() contains only initial calculations it shouldn't depend on ticks.
What "apply the reviewed changes" means? You requested none

@RedCarlos26
Copy link
Owner

On task change now tick() is called immediately after start(). Because start() contains only initial calculations it shouldn't depend on ticks. What "apply the reviewed changes" means? You requested none

Sorry, feedback wasn't yet submitted

@RedCarlos26 RedCarlos26 added planned and removed question Further information is requested labels Mar 10, 2025
@RedCarlos26 RedCarlos26 marked this pull request as ready for review March 10, 2025 19:05
@RedCarlos26 RedCarlos26 marked this pull request as draft March 10, 2025 19:05
@RedCarlos26 RedCarlos26 linked an issue Mar 11, 2025 that may be closed by this pull request
@hexadec1mal
Copy link
Author

53 merge conflicts gg

@RedCarlos26
Copy link
Owner

53 merge conflicts gg

Check discord

@hexadec1mal
Copy link
Author

What should i see there?

@RedCarlos26 RedCarlos26 marked this pull request as ready for review March 12, 2025 05:56
@RedCarlos26 RedCarlos26 self-requested a review March 12, 2025 05:57
@RedCarlos26
Copy link
Owner

What should i see there?

Dms

@hexadec1mal
Copy link
Author

I received none. My discord name is the same: hexadec1mal

@hexadec1mal
Copy link
Author

I just diffed higtools and meteor's highway tools. Differences are 1. Resume tps 2. Toggleable print statistics.
Because i need to heavily modify my patch for new version anyway i think it would be better to make pr to meteor instead

@RedCarlos26
Copy link
Owner

Probably, yes! If they don't want to make it work with baritone, then Pr it here! Thanks!

Repository owner locked and limited conversation to collaborators Mar 12, 2025
Repository owner unlocked this conversation Mar 12, 2025
@RedCarlos26 RedCarlos26 marked this pull request as draft March 28, 2025 02:20
@RedCarlos26 RedCarlos26 removed their assignment Mar 28, 2025
@RedCarlos26 RedCarlos26 linked an issue Mar 28, 2025 that may be closed by this pull request
@RedCarlos26 RedCarlos26 marked this pull request as ready for review March 28, 2025 03:25
Copy link
Owner

@RedCarlos26 RedCarlos26 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just thoroughly tested the module with the merged changes! Works really well, but there's an issue that should be fixed before merging.

It would be better to calculate the number of ender chests to mine depending on the number of available slots. (formula should be : # of slots available * 64 / 8 = # of ender chests to mine)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Compatibility measures for 2b2t.xin servers

2 participants