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

New stuff from Ancestor To Do #6288

Merged
merged 20 commits into from
Aug 18, 2023

Conversation

Paliak
Copy link
Contributor

@Paliak Paliak commented Aug 1, 2023

Implements the following mods:

  • Enemies you kill have 5% chance to explode, dealing 500% of their maximum Life as Fire Damage
    I added another line to keep the match string simpler but this maybe could be done by modifying the existing one.

  • Skills from equipped Body Armour are supported by Level 30 Ancestral Call
    Skills from equipped Body Armour are supported by Level 20 Fist of War
    I moved the code from the socketed [%a+]* ?gems a?r?e? ?supported by level (%d+) (.+) line handling the extra support mod creation into a function and added the ability to pass a slot string. Tested with both the new mods and a few old ones and it seems to work correctly.

  • Grants Armour equal to 25% of your Reserved Mana to you and nearby Allies

  • Auras grant 5% increased Recovery Rate of Life, Mana and Energy Shield to you and allies
    Handled with piece wise parsing.

  • Elemental Damage with Hits dealt by Allies between you and Linked Targets is Lucky

  • Allies between you and Linked Targets have +5% to all maximum Elemental resistances
    Handled with piece wise parsing. Uses the "Is the enemy in the way of a Link skill?" config option (possibly misleading?).

  • Enemies between you and Linked Targets cannot apply Elemental Ailments
    The implementation for this mod could likely be generalized but i was not able to make it work. Implemented by setting player ailment avoid chance to 100%.

  • 10% of Damage from hits is taken from Sentinel of Radiance's life before you

  • Nearby Enemies have no Fire Resistance against Damage over time while you are stationary
    Implemented using generic "no" mod modForm. This allows the ModParser to piece wise parse similar mods.

  • Modifiers to Maximum Fire Resistance apply to Maximum Cold and Lightning Resistances
    Should work for any combination of resistances. The implementation should be simplified by using a hard coded constant but i wanted to make it more flexible.

  • Modifiers to Fire Resistance apply to Cold and Lightning Resistances at 50% value
    Similar to above

  • Non-unique Jewels cause Increases and Reductions to other damage types in Large radius to be Transformed to apply Fire Damage
    Due to the way it's implemented the mod only works if it comes from a node on the tree. This should be fine since it's an ascendancy node.

  • +910 to maximum Life if there are no Life Modifiers on other Equipped items
    The chieftain jewel mod required a rework of the ItemCondition code in ModStore.lua so i added support for allSlots and excludeSelf flags while there.

Adds new config options:

  • conditionHaveRadianceSentinel
    "Do you have a Sentinel of Radiance?"
  • conditionBetweenYouAndLinkedTarget
    "Is the enemy in the way of a Link skill?"
    This one could maybe use better wording.

Adds the following items to the New.lua file

Kitava's Hunger
Majestic Plate
88% increased Armour
Recover 2% of Life on Kill
37% Increased Stun and Block Recovery
+910 to maximum Life if there are no Life Modifiers on other Equipped items

Should be fully supported.

Arohongui's Tending
Coral Amulet
Regenerate 4.4 Mana per second
37% increased Life Recovery from Flasks
26% increased Mana Recovery from Flasks
Life Flasks used while on Low Life apply Recovery Instantly
Mana Flasks used while on Low Mana apply Recovery Instantly

Instant recovery mods are not supported but don't seem to be supported on flasks either. I'm not sure if there's a point in supporting those in POB

Bound Fate
Cloth Belt
+28 to Dexterity
+26 to Intelligence
+64 to Maximum Life
Every 5 seconds, gain one of the following for 5 seconds:
Your Hits are always Critical Strikes
Hits against you always are Critical Strikes
Attacks cannot Hit you
Attacks against you always Hit
Your Damage with Hits is Lucky
Damage of Hits against you is Lucky

I'm not sure what would be the correct way of supporting the temp mods here. For now i've disabled lucky damage mod by replacing spaces with nbsp.

Paliak added 6 commits July 31, 2023 02:33
Currently implemented as less damage taken as that's what the
spreadsheet said.

I think this may be possible to implement like frost shield or other
"taken before you" mods given more info on stats of the sentinel.
@QuickStick123 QuickStick123 added the enhancement New feature, calculation, or mod label Aug 2, 2023
@Paliak Paliak changed the title Ancestor mods from Ancestor To Do New stuff from Ancestor To Do Aug 7, 2023
@Wires77 Wires77 mentioned this pull request Aug 14, 2023
3 tasks
@Wires77 Wires77 linked an issue Aug 14, 2023 that may be closed by this pull request
3 tasks
@Nostrademous
Copy link
Contributor

LGTM

@Wires77
Copy link
Member

Wires77 commented Aug 17, 2023

10% of Damage from hits is taken from Sentinel of Radiance's life before you
This could maybe be possible to implement like frost shield. Currently implemented with a generic less damage taken mod.

Looking at the code this doesn't seem accurate anymore, fixed in 0784fa1 correct?

@Wires77
Copy link
Member

Wires77 commented Aug 17, 2023

For the rotating mods, I really dislike the nbsp "fix". Would much rather make each one a variant instead, because a year from now no one is going to understand why that isn't working. I'll change that, add the new unique (wording only), and update the wording for the lines that have changed.

Is the "no" ModForm still needed now that the wording changed to "is 0%" now?

@LocalIdentity LocalIdentity merged commit ceb331c into PathOfBuildingCommunity:dev Aug 18, 2023
LocalIdentity added a commit that referenced this pull request Aug 24, 2023
…ork code. (#6447)

* Fix version number mishap

* move timeless jewel stuff (#5747)

* move timeless jewel stuff

* add back all the local versions of functions for future use

* Help section improvements (#6156)

* increase size of box and text

* implement hotkey to directly open help at a given section

* add ability to "fast scroll" to next section

* add section heights to changelog

* update txt

* fix merge issue

* fix line overlap and add note about scrollbars

* Add lot of stuff for ancestor league (#6288)

* FEAT: impl explode and extra support mods

* FEAT: impl new aura recovery mod

* FEAT: impl between you and linked target mods

* FEAT: impl Sentinel of Radiance mod

Currently implemented as less damage taken as that's what the
spreadsheet said.

I think this may be possible to implement like frost shield or other
"taken before you" mods given more info on stats of the sentinel.

* FEAT: impl reserved mana as armor mod

* FIX: use arbitrary taken before you instead of generic less dmg taken

* FIX: cache result of dmg type max hit str concat

* FEAT: code for generally supporting have no mods

* FEAT: impl resistance conversion mods

* FEAT: impl extra jewel func code for new chief mod

* FEAT: impl mod on all other slots condition

* FEAT: add new uniques

* Wording fixes, new unique

* Remove debugging lines

* Adding fire mastery explosion mod

* FIX: use ailment immunity flags from #6297

* Late breaking Chieftain change

* Update mod ranges

* Fix Amulet and belt

---------

Co-authored-by: Wires77 <[email protected]>
Co-authored-by: LocalIdentity <[email protected]>

* FEAT: add immunity flags to breakdown (#6389)

* FIX: fix totem duration mods not applying (#6388)

* Fix DPS on Vaal Flicker when using 2x 1h weapons (#6380)

The block was inside the 2 pass loop so was running twice
Also fixes the issue when you used a custom `more DPS` mod

Co-authored-by: LocalIdentity <[email protected]>

* Apply RepeatCount according to their SkillTypes (#6376)

* Apply RepeatCount according to their SkillTypes

* Fix Vaal Flicker Strike

---------

Co-authored-by: LocalIdentity <[email protected]>

* Fix Immunities breakdown

* Fix Maata'a Teaching typo

* Fix allocation issue on convert while maintaining node id/name filtering (#6364)

* Fix lower channel time stat using red text (#6381)

* Fix lower channel time stat using red text

* Fix Channel time showing on calcs page for more skills than necessary

---------

Co-authored-by: LocalIdentity <[email protected]>

* Fix "Enemies maimed by you take inc damage over time` not in breakdown (#6400)

The mod was not displaying in the breakdown as it was using the dot flag instead of `DamageTakenOverTime`

Co-authored-by: LocalIdentity <[email protected]>

* Fix Pierce and Chain count config not appearing sometimes (#6401)

* Fix Pierced count config not appearing sometimes

* Fix Chain config not appearing

---------

Co-authored-by: LocalIdentity <[email protected]>

* Regen ModCache

* Fix Snipe damage going negative (#6399)

* FIX: snipe damage going negative

* Set SnipeStacks min value earlier

---------

Co-authored-by: LocalIdentity <[email protected]>

* Null-check node.finalModList when hovering trees (#6408)

Looks like node.finalModList is not initalized when hovering trees, this
simply null checks it.

Closes #6405

Signed-off-by: Tomas Slusny <[email protected]>

* Update 3.22 tree (#6411)

* update tree

* Fix rewording on Chieftain Node

---------

Co-authored-by: LocalIdentity <[email protected]>

* Fix Snipe showing DPS values when triggering support skills (#6415)

Added damage was still contributing to the damage of the skill so it was showing DPS numbers even if the skill was actually dealing no damage
Also removes the debug tooltip I forgot to remove from testing

Co-authored-by: LocalIdentity <[email protected]>

* 3.22 Additions (#6418)

* FEAT: impl more sources of self hit dmg

This code has been moved from #4599 after abandoning cwdt loop support.

* FIX: spelling

* FIX: remove file implemented in #4599

* Add support for non-Boneshatter Trauma

* Add new gems from 3.22 and other stuff

* Fix Crucible Export

* Fix Ruthless support to work with all ailments

* Update for actual support gem

* Support inc stun duration per trauma

* add devour, volatility and locus mine support (#6412)

* add devour, volatility and locus mine support

* Add full support for gems

Add mine tag to Locus Mine
Add support for Anomalous Devour
Fix Mine PvP damage multiplier
Fix flags for damage against full/low life enemies

---------

Co-authored-by: LocalIdentity <[email protected]>

* Spellblade support (#6343)

* Initial spellblade support

* Update label for spellblade damage breakdown

* Update with actual stats and qualities

* Update .txt too

* Floor all final flat values

---------

Co-authored-by: LocalIdentity <[email protected]>

* Fix export

* Add spacing and fix typos

Alter the breakdowns to no longer show the damage taken multi/taken % line if the value has not changed
Added spacing to the lines so you can more clearly read the breakdown when you have multiple taken as elements
Fix some spacing and spelling mistakes

---------

Co-authored-by: Paliak <[email protected]>
Co-authored-by: Lilylicious <[email protected]>
Co-authored-by: LocalIdentity <[email protected]>
Co-authored-by: Regisle <[email protected]>
Co-authored-by: Lily <[email protected]>

* Regen ModCache

* FIX: allow setting inspiration charges to 0 (#6421)

* Add support for Guardian's Blessing no reservation for auras (#6425)

Signed-off-by: Tomas Slusny <[email protected]>

* FEAT: impl sadism less ailment duration mod (#6431)

* FIX: incorrect increased usage mod range (#6434)

* Implement Corrupting Cry support (#6436)

* FEAT: impl Corrupting Cry support

* FIX: crash caused by incorrect capitalization

* FIX: add statMap and baseFlags to txt template

* FIX: spelling

* Revert "FEAT: impl Corrupting Cry support"

* FIX: reimplement using generic physDot

* FIX: copy paste issue

* FIX: cap corrupting blood staged.

Note that this config option also is affected by the one off bug in
config tab code.

* FIX: off by one in the config

* FIX: don't override skillData.durationSecondary

Overriding skillData.durationSecondary of exterted attacks may mess with
secondary effect of skills such as Dominating Blow

* FIX: CorruptingCry cfg stages not affecting exerts

* FIX: crash when hovering over more mult

* change dot type to corrupting blood to prevent dot from stacking

* Remove Attack dot addition + move config

* Fix removal of line

* Empty

---------

Co-authored-by: Paliak <[email protected]>
Co-authored-by: LocalIdentity <[email protected]>

* Add support for new Ancestor uniques (#6426)

* Add mod text only

Signed-off-by: Wires77 <[email protected]>

* Fix Level requirements and add Replica Dragonfang's Flight

* Support Ahuana's Bite

* Support Bound Fate

* Support Kahuturoa's Certainty

* Support Rakiata's Dance

* Fix global gem level/quality name change

---------

Signed-off-by: Wires77 <[email protected]>
Co-authored-by: LocalIdentity <[email protected]>

* Removing allocated mastery from hover list (#6374)

* Removing allocated mastery from hover list

* Updating wording on mastery

---------

Co-authored-by: LocalIdentity <[email protected]>

* Add Ruthless tree (#6367)

* Add support for Ruthless trees

* Fix export URL for Ruthless trees

* Removed data.json

* Fix PR comments

* Fix edge cases around converting and importing

* Updating support for 3rd party ruthless tree links

* Update tree and add Tattoos

* Add support for new Ruthless tree mods

* Fix comment

---------

Co-authored-by: LocalIdentity <[email protected]>

* Support ruthless for poeskilltree

* Optimize itemLib.applyRange, improve startup time (#6407)

* Add applyRange tests

* Optimize itemLib.applyRange, improve startup time

* Fix tree import crash

* Tattoo implementation (#6396)

* Initial tattoo implementation

* Tattoo spec update

* Tattoo export

* Update spec

* Fix spec

* New dat files

* More fixes

* Add artwork, restrict Makanga tattoos

Signed-off-by: Wires77 <[email protected]>

* Fix bug with saving/loading, cleanup code

* Fixed import bug and matched format for save files

* Fixing logic issues preventing proper tattoos from appearing

* Fixed bug with linked node count and supported new tattoo jewel

* Temp fix for effectSprites

* Can't tattoo changed nodes

* Fix tattoo interaction with Timeless jewels

* Update tree, fix more bugs

* Add dropdown search

* Add help text to tattoo dropdown

* Fix stat description error

* Dynamically set popup width

* Made popup width fully dynamic

* Truncate mod lines in tattoo popup

* text wrap in popup

* Dynamic word wrapping, ignoring GGG wrapping

* Revert tree merge

---------

Signed-off-by: Wires77 <[email protected]>
Co-authored-by: LocalIdentity <[email protected]>
Co-authored-by: Peechey <[email protected]>

* Fix secondary effect skills being added as supports (#6393)

* FIX:secondary effect skills being added as support

* FIX: cleanup duplicate code

* Fix Bleed/Ignite Stack potential issues (#6386)

The config was not affecting the number of bleeds/ignites affecting the enemy and was instead only affecting the average roll %
Ignite and Bleed roll % was not calculated to be at least 50% roll as a minimum
Ignite was also not taking into account Hit Chance when calculating the average roll
Breakdowns were improved to show the potential when max stacks is more than 1

Co-authored-by: LocalIdentity <[email protected]>

* Regen ModCache

* Version dropdown QoL (#6365)

* Version dropdown QoL

* ruthless update

* Add support for Fresh Meat Support (#6443)

* Add support for Fresh Meat Support

* Adding general support for increased minion critical strike chance

* Move Config

---------

Co-authored-by: LocalIdentity <[email protected]>

* Implement Sacrifice, Frigid Bond, Guardian's Blessing, Controlled Blaze

* Add initial support for Guardian Minions (#6445)

Adds support for the new Guardian minions to show in the sidebar. The level is manually set to 85 right now as we don't support the `minion_actor_level_is_user_level_up_to_maximum` stat yet

Co-authored-by: LocalIdentity <[email protected]>

* Release 2.32.0 (#6440)

* Prepare release 2.32.0

* Fix changelogs

* Spelling

---------

Co-authored-by: LocalIdentity <[email protected]>
Co-authored-by: LocalIdentity <[email protected]>

---------

Signed-off-by: Tomas Slusny <[email protected]>
Signed-off-by: Wires77 <[email protected]>
Co-authored-by: Wires77 <[email protected]>
Co-authored-by: Regisle <[email protected]>
Co-authored-by: LocalIdentity <[email protected]>
Co-authored-by: LocalIdentity <[email protected]>
Co-authored-by: Peechey <[email protected]>
Co-authored-by: Tomas Slusny <[email protected]>
Co-authored-by: Lilylicious <[email protected]>
Co-authored-by: Lily <[email protected]>
Co-authored-by: Lancej <[email protected]>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: LocalIdentity <[email protected]>
@Paliak Paliak deleted the ancestorMods branch January 20, 2024 15:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature, calculation, or mod
Projects
None yet
Development

Successfully merging this pull request may close these issues.

New Chieftain ascendancy
5 participants