Highway builder: baritone pathing #5254
Conversation
|
I have some local changes regarding highway builder that I want to clean up and commit before examining something like this. I haven't tested this yet but I skimmed the code, and I don't really like the fact that the module now requires baritone to be added in order to be used. If this were to be added I think it should be optional for it to use baritone. How does the module perform using baritone compared to without? Is it faster/slower, or more reliable? There are a couple of small issues with the module as it stands, but from my testing it's very stable overall, so I can't say with certainty if using baritone comes with more benefits that not yet. I'll test it on my own after I finish my local changes but I want to hear your perspective. |
|
It's faster now because it can break/place and move simultaneously. These changes In hwu discord, I saw people saying that they lose obsidian when mining ender chests. I have never seen this myself, but since baritone now collects obsidian, it shouldn't happen anymore. Making baritone optional requires 2 versions of the module, which can't be consistently kept in sync, so I don't think that's an option. |
Type of change
Description
Now, pathing, breaking, and placing don't depend on the player's position. Collecting shulkers and obsidian is now also done using baritone, so blockade states have been removed. I was rewriting some parts of the code, so I might have deleted something important. If you have any questions about my code, please ask.
Related issues
None.
How Has This Been Tested?
Will test properly later, after review. And I will probably change something else, so making this draft for now.
Checklist: