-
Notifications
You must be signed in to change notification settings - Fork 629
Piglin Barter Drops are Biased #4326
Copy link
Copy link
Open
Labels
good first issueThis is an Issue that may be easy to fix if you are a beginnerThis is an Issue that may be easy to fix if you are a beginner🐞 Bug ReportA bug that needs to be fixed.A bug that needs to be fixed.🧨 Confirmed BugThis bug has been confirmed by our team.This bug has been confirmed by our team.
Metadata
Metadata
Assignees
Labels
good first issueThis is an Issue that may be easy to fix if you are a beginnerThis is an Issue that may be easy to fix if you are a beginner🐞 Bug ReportA bug that needs to be fixed.A bug that needs to be fixed.🧨 Confirmed BugThis bug has been confirmed by our team.This bug has been confirmed by our team.
❗ Checklist
📍 Description
The piglin barter drop recipe type uses a hashset and iterates over it when generating the custom drop, the hash set, after its done being modified (post finalized event) will have a deterministic order for the runtime of the server.
Because of this, the items at the beginning of the iteration order have a bias towards being dropped.
📑 Reproduction Steps
Look at the code
💡 Expected Behavior
Drops would be properly randomized always
📷 Screenshots / Videos
No response
📜 Server Log
No response
📂
/error-reports/folderNo response
💻 Server Software
Spigot
🎮 Minecraft Version
1.20.x
⭐ Slimefun version
🧭 Other plugins
No response