Repository navigation
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description of the problem being solved:
This PR rips out the PoE1 mirage and trigger code, and replaces it with a basic implementation that allows for calculating trigger DPS. Mirages are still unsupported. Invocations were not considered here.
Currently, this has configurations for:
Feedback is very welcome. This is not a very full implementation and this is still missing support for some very obvious things like Energy Retention.
This does however seem to work alright.
Though there are performance problems with cast on crit due to it rebuilding every skill in power reports. It might be a better idea to just build the previously used skill?The trigger sources are cached similar to PoB1.Some of the other triggers also seem like they might be tricky. For example, Cast on Ignite and Elemental Ailment. These two also interact with some of the fake hit spells, which PoB doesn't seem to handle in any different way to regular hits at all.
One notable unsupported thing is the
active_skill_energy_generated_+%_finalstat. I think all of the skills having this are actually using fake hits, which means coc would work with them and would be very wrong. These need to have fake hits handled somehow when the ailment triggers are added.Steps taken to verify a working solution:
Link to a build that showcases this PR:
https://poe.ninja/poe2/pob/2bf20 coc
https://poe.ninja/poe2/pob/2bf22 cod (note that EF is not properly supported here)
Before screenshot:
N/A
After screenshot: