search

TheStonedTurtle / banked-experience

A RuneLite plugin used to calculate the experience you have banked
BSD 2-Clause "Simplified" License
15 stars 44 forks source link

Feat: Exclude XP from missing secondaries #124

Closed GregTheGreek closed 10 months ago

GregTheGreek commented 10 months ago

Happy to submit a PR if it's helpful.

Feature

A toggle option that allow users to separate Total Banked number amount into two new values: banked with secondaries, and banked without secondaries.

Banked without secondaries Displays the total amount of experience you would receive based on the total number of secondaries you have banked. Banked with secondaries Operates the exact same way Total Banked works today in the interface

TheStonedTurtle commented 10 months ago

Similar to #9 I'd recommend you read that to see why I haven't attempted it but if you can create a PR for it sure.

Namely just this part:

it becomes difficult for the skills such as Smithing where a single secondary resource (coal) is used for multiple activities. I could ignore cases like this but then the toggle is sorta useless for that skill, i could prioritize higher leveled activities but maybe that isn't what the player wants.

GregTheGreek commented 10 months ago

Closing in favour of #9 Thanks for the reply