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

Vales Iron farm optimizations and more additions #178

Open
wants to merge 2 commits into
base: staging
Choose a base branch
from

Conversation

Dioswilson
Copy link

@Dioswilson Dioswilson commented Feb 15, 2024

Hello, this changes were not made by me but by Vales, since he doesn't want to push this changes I asked him permission to post this. I will describe what was changed below:

  • Added /carpet doorSearchOptimization
  • Added /carpet doorCheckOptimization
  • Added /carpet ironFarmAABBOptimization
  • Added /carpet ironGolemsSwim
  • Added /carpet villagerQueue
  • Added /carpet villagerQueueLength
  • Added /carpet villagerTickingRate
  • Added /carpet villagersRandomStart
  • Added /carpet doorDeregistrationTime
  • Added /log villagerPos
  • Added /log villages <dynamic|overworld|nether|end>
  • Added /log golems
  • Added /log counter all
  • Added /log portalCaching uncaching
  • Added /log pathfinding
  • Fixed NullPointerException with /log
  • Readded time and total to /counter
  • Fixed /log items not tracking items that change dimensions
  • Added /ticktimes <start|stop> [filename]
  • Changed communication with carpet client for village marker to make it much more efficient and crash resistant(If this goes through I will try to push them into carpet client)

@Dioswilson
Copy link
Author

I added his changes under a section I called "VALES" on CarpetSettings, if you want to merge this changes I should first move them into the according position. Also added "Vales" comment on CarpetCommands which I should also remove

Feel free to tell me to remove any addition in case you don't want it.

@Earthcomputer
Copy link
Collaborator

Uhh normally you would PR one feature at a time, so we can review and decide to accept them individually. But I understand this is now going to be hard to split up. I will review when I get the chance.

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