Item filter suggestion: allow OR and AND operators for true/false conditions

Hi there and thx for introducing this, but it should be extended.
🌞 Designer of SimpleFilter see My Item Filters 🌞
🌞 I treat PoE as an art 🌞
Last edited by koszmarnica#7777 on Dec 18, 2019, 4:19:39 PM
Limited implementation of regexp.lib to search field in stash?
That could be bit hard drop for casual people to get around it, but shouldn't be that bad. Most of time might be going to make safeguards that any given filter would not trip the code, and cause game to crash or have other less beneficial features.
Since player is at stash, nothing time critical is happening. If the filtering should have unexpected drain on CPU resources to perform requested operation, minor stalls won't cause problems. Player entering complex regexp should be aware it's nature and requirement of processing power.
might i add that something like "itemlevel < 60" would also come in handy?
age and treachery will triumph over youth and skill!
Because of code branching (in case of conditional jumps) that affects CPU caches (branch misprediction) it's actually better to code like it is now, with every rule applying to one case and setting one (or multiple) flag(s), than to run dozens/hundreds of nested "if-then-else" within every frame displayed in the game client, dozens or hundreds times per second.

Yeah, it creates an overhead when it comes to the amount of the code. But so does your compiler for you when it (for instance) "unrolls" the loops. There are no free lunches, just compromises between the "clean, readable code", performance and memory consumption.

Right now all the game has to do is just to run every rule in sequence, set the flags (if the rules apply) and that's it. No need to jump forth and back around the filter.

Relevant info: https://stackoverflow.com/questions/11227809/why-is-processing-a-sorted-array-faster-than-processing-an-unsorted-array
Last edited by jewelsy2#6622 on Jan 2, 2020, 12:30:55 PM

Report Forum Post

Report Account:

Report Type

Additional Info