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

feat: pickpocket except against multidrop monsters #8

Merged
merged 3 commits into from
Dec 22, 2023

Conversation

midgleyc
Copy link
Member

You can pickpocket the common drops, so try to do that. However, don't try against the mudlark or engineer: these drop 1-3 of the flotsam or machine parts normally, and if you pickpocket you always get one.

Don't condition on being able to pickpocket because if you submit a macro where you try to pickpocket but can't, you just don't. Efficient! And also I'm not certain how to check for this without enumerating all the possible cases.

Value mime army infiltration glove at 2k: it's about 5% of a common drop, and the farmable ones are 30k-50k. Might get a better valuation later but I think this is good enough for now.

@MrFizzyBubbs
Copy link
Collaborator

Would it be worth using the mall price or the cheapest/average common drop for the valuation rather than hard coding?

@midgleyc
Copy link
Member Author

Definitely, but pulling out the common drops given a location is nontrivial (unless I just hardcode them I guess).

@MrFizzyBubbs
Copy link
Collaborator

The zone monsters are hardcoded for the orb arg so that seems reasonable to me.

@midgleyc
Copy link
Member Author

righto, updated (but not tested)

@Rinn Rinn merged commit 62b7ad0 into loathers:main Dec 22, 2023
3 checks passed
github-actions bot pushed a commit that referenced this pull request Dec 22, 2023
* feat: pickpocket except against multidrop monsters

* feat: better valuation for pickpocket

* chore: lint
@midgleyc midgleyc deleted the pickpocket branch December 23, 2023 10:18
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.

3 participants