Skip to content

Port many PoB2 trader features#9934

Draft
vaisest wants to merge 3 commits into
PathOfBuildingCommunity:devfrom
vaisest:trader-stuff
Draft

Port many PoB2 trader features#9934
vaisest wants to merge 3 commits into
PathOfBuildingCommunity:devfrom
vaisest:trader-stuff

Conversation

@vaisest

@vaisest vaisest commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Description of the problem being solved:

Fix #9685
This fixes:

  • some mods not having weights generated (incr block chance for example). I'm not sure if there are more errors
  • querymods generation using tags which are too wide, like "weapon", which caused weapon searches to have a ton of implicits that couldn't actually be on the category
  • adding corrupted mods if they have zero weight. I think these might be some standard-only stuff, which might be useful, but PoE1 has so many mods that I don't think filter slots should be wasted on these.

End result: searches should be more relevant. This was mostly done by porting existing PoB2 changes which I believe were made by Quickstick

This ports:

Steps taken to verify a working solution:

  • todo

Link to a build that showcases this PR:

Before screenshot:

image ### After screenshot: image

@vaisest
vaisest marked this pull request as draft July 8, 2026 16:43
@vaisest vaisest changed the title Fix unrelated mods polluting trader searches Port many PoB2 trader features Jul 21, 2026
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.

Trade URL not getting recognized

1 participant