Update Slayer Best in Bank: coherent loadouts and task fixes - #14866
Open
FreeArcanes wants to merge 4 commits into
Open
Update Slayer Best in Bank: coherent loadouts and task fixes#14866FreeArcanes wants to merge 4 commits into
FreeArcanes wants to merge 4 commits into
Conversation
|
|
|
This plugin requires a review from a Plugin Hub maintainer. The reviewer will request any additional changes if needed. Internal use only: Reviewer details Maintainer details |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Updates Slayer Best in Bank to source commit
aa31d3677ca671b66b5bc5ce5c79a72af5571be3.Turael/Aya skipping and point boosting
Turael/Aya skipping / boostingcheckbox directly to the side panelPreparation and strategy corrections
Venator targeting
VenatorandVenatorsthrough the dedicated Vampyre profileCoherent loadout scoring
Review follow-up fixes
Why
The prior isolated-slot scorer could recommend an item with attractive defensive stats even when its set bonus was impossible in the selected Slayer loadout. It could also miss weapon/armour synergies and let defensive weighting override offensive gear while a protection prayer already nullified the incoming attack.
A final review found that alternate tiers could retain supporting-armour scores from their previous weapon and that non-mace Inquisitor setups omitted the complete-set bonus. The follow-up commit rebuilds the dependent package after a weapon swap and treats the Inquisitor bonus as a whole-loadout effect.
User impact
Players receive usable whole-loadout recommendations rather than incompatible mixtures. Alternate tiers no longer carry stale Crystal, atlatl, or Inquisitor scoring after a weapon swap. Complete Inquisitor crush setups are compared using their real set effect, while partial sets receive no extra bonus. Regular tasks no longer pull Wilderness-only restores, Venators receive valid Vampyre gear, and Turael/Aya users have a dedicated fast-task workflow.
Validation
plugins/slayer-best-in-bankgit diff --checkpasses