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

Auto equip #699

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

Auto equip #699

wants to merge 10 commits into from

Conversation

CarlPoppa1
Copy link

@CarlPoppa1 CarlPoppa1 commented Feb 25, 2025

This is in a finished state but it remains as draft until PR 77 is merged in D2Go

Heavily borrowed from kolbot-soloplay
This only supports softcore sorc with an act 2 merc.

Known issues:

  • After equipping a ring, it immediately tries to equip the same ring in the other ring slot
  • No inventory space checks when swapping items of different sizes in left and right arm

To do:

  • Add other breakpoints (FCR, FHR etc) and check if replacing an item with FCR or FHR for one with an item without (or an item with less FCR, FHR) will drop past a breakpoint.
  • Add handling of equippable quest items in D2Go
  • Add the +x in +x to skills. Currently it just adds a flat value if there is a relevant +x to skills. It should multiply the weighting by the +x. So if the weighting is 40, +3 to Blizzard should add 120 to the item score, it currently only adds 40 to the score.

@CarlPoppa1 CarlPoppa1 marked this pull request as draft February 25, 2025 15:18
@artosimonyan
Copy link
Collaborator

@CarlPoppa1,

This definitely needs to be an option that can be enabled, not turned on at all times.

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