Skip to content

Adds support for Lord of the Wilds non-minion reservation calculations#1943

Closed
Eucelia wants to merge 1 commit into
PathOfBuildingCommunity:devfrom
Eucelia:eucelia/lord-of-the-wilds-minion-efficiency-mod
Closed

Adds support for Lord of the Wilds non-minion reservation calculations#1943
Eucelia wants to merge 1 commit into
PathOfBuildingCommunity:devfrom
Eucelia:eucelia/lord-of-the-wilds-minion-efficiency-mod

Conversation

@Eucelia
Copy link
Copy Markdown
Contributor

@Eucelia Eucelia commented May 23, 2026

Description of the problem being solved:

Lord of the wilds didn't have support for 50% increased spirit cost for non-minion skills. Now it does.

Steps taken to verify a working solution:

image

Link to a build that showcases this PR:

eNq1G1134rbyefsrfHhmA7Yhhj2kPUlINuwhGzZkd9v70iNsAWpkm1oyCf31HUn-BoOc9OYhseX5ntFoNFJGv7361NjiiJEwuGiZZ92WgQM39Eiwumh9f7r9OGj99usvoxni64flVUyo-GL9-suHkXwx3DWKkMtxNMVbTC9jHt6HHr5o8SjGLcOliLGvyIeBcRQTr2X4iATz0H3G_HMUxhtg2TKoQIUnu2VwFK0w_5HK0_0TPm8JflFEJ_ezh8enloGYiwPvOif-ADJQ3AKxPoxmFO1wNOeIG1tEY_gKOjF4vWhdh_6CBNgbz-atTg3wwEyh5zyqhXJSoDF-rSd1nkI94r-PAZoZz0nA66H6ToHeMchuLt025GDJ05BPEVnEHNcCWo59Zlv2wOk6fcvKBHkKOaI3d7NaPNuxUtjZeseIi-g9eiV-7N8R_oSecb1sVm-Qok7Jas0DiLw34N6SCL8B7Tqkni5aQcnrNQqZLl7uJxQ9B5ix05CPmOFoCyF8CqMg05Qs6x3bz8J9g91PAnQSuLpUvwdRIo8uxiN2Q0g3aEHrRTK73cNMZjiCia8R94rTCgcJu90RXsMU6R4F6DRpaSYBesxMFaoaZiroXMY4pXPvrFdE1NO6X1CHRKSe-sfesAypocrH4aAOR9uBNwGOVrv5mmDqNYNOdb9GG01nFrGPObWGnY698-mLVpqBmaXYmy1iWilcKaOg9fTYIg9DwgpcfBr4HlOMG2HMovAv7HJCm6GBGpQ2x3gjtzFeUoEWBo-IwwKjjaHL4DLywzjS9J4C1nJeupqOkS9jyotdvZX-ZrkUCm_xFYU6TFeNDEsauxHqJefIfR6H3qqZRxthlOWbx5sN5BwxE04Q-JinQlEowPJKmEZ-ymEfYBYfyzUFBqKk0GWQwzZgkJVJulwqCKdYdUtVjrYqOfApBuaeO-8hS_qwZIBbsQe7gPp1ME87sNiAyx-RR2J2jzm812Kd22cDc9Ab2qblnOdl-G0M4Xdkk9A969rdYYLVHdi9EmLIc9yE0vyZUGowmNwuoAO0eJ-hSMT6M2RWqKKNy4iFARjpkyGiy7iCzUqrIKczcPJarYpj3JMgn_0HmdbiFIVpndTwCgPmZBUQjo2jJsr8Eb6A09diY3ekWj0EDWW0xkyMcPDPTpt-CVyLwU3gxZHIIdo8qhg5m6t4uWSGC55FfAouEAZfwFj67MYRw8mLwngiPriNsTHiyGAYRe46Bc7fbhGlC8ixYtRLtnw_UERQwE01qeSO-5ZQ2KiPAUEIJ1WsQFsCWjAedeT2XjxNOPaZwdbhi1B2TGBmRmJ-sXSTj-REFWBzzCW7mOE51DeB9xOjTRjI4SWiLNmhJ6AG8TSgP4zmNARYwJl4UuZAbvkVsGEa8xdIKPL7bPH9cZparg7vliL2bJgNMGBRNqwG8MLrzTCmeNUIPtHdaqq74NNIcz-GlNEA4Q5TvxFCqklj89oNMD5TWERYA4SrMOSN4DFtovQjLLiNVJYIdiMFvJ2RVJ5NLWs2FcxsPPushrPPbB5RNSgyD048GDSC0MMC0zo3h-dl6ITUF_yCqaG-q2zcSVKXSs0R7DPmMgn_xKKQYipzyoQpnq4RdZnKeMEm5kYQ-wscyZynGMiF9081nHBQkIxHsgF7c3t7c_00-XGTYviEuX-KFUM0RBOUudq_wLJCKdow7KXpUyZYWQTMsdiywDoQL5iCvmj9IPhFfhxDVUAo06J2R7jacJRpSTr5Nx1KP8FuERHTskjo5hVHYhuWfdUhBQXXAXGULGKTLdZQPVuJIrNMSm1erhHjsD_Uo3ItGiolImJET5WJv0G0YtxkTNvZT7sNFks1O2AUDt8Y7DTJkrgKRtPryTaqbBsX6hXk7rRoXNE926ohHeRZSJisVAvYyZiWVWW5WrGqGtOKL-yiXRlbDekgTzF2158RqUifDYfBXdZ_O07phuJLQsVmqOLZr2Eggx3mTQagQ_AekonaaFUIPvA1jtIvOpRglqjzA7Y3e9JxLVuJRnXZTHnr-oQuonlbQi20c0_NerI3Y0tt0RNuKbYqy8lsv4l5nFSyoy4RUWOabkhW_JILiv2nE5okncayEqX244mJIlPu5TYk3n62qH58N0HZhPoP5BJtpv-ATNZI1KIlC6HKbEnGatHVdqsO-zsnlPDd-4iI-fY-CmLavY-CLEO1sB_3CpHHo-VHjpnu5fcYrzB7M362738zBdWdeDO6bJ5op36IWLmPP5D7s0_vozXncTAGkxyfFblSJ6TSyB36tNRC2UhRNf_Vke6B1FY8630PIVjG73TrxVpKWQP1DiPK17MwpO8jePhM-6gLammJw694gwIvpfbQpEqvtx7s2oGo7FyNxSEbe5eUsHz7uwKdUSfd0I2-hlDSCNJiNH0ZXYfBkqySfph6STpiUopsxOCEUyyXDBTLzoGXAsm-oovXIfVwlG0Y-9nuFguZqgcVh5GG5_0SljhRKO7PDmMNrBKSvNVzDL7CIyKlg7HDOPn-l6wIfVjKtAW7kSz3alnh4GHHYUx7z3yzNIK1WFUPbmpM1z0vMyqWXodRnG6ZkdqAHkGoyFU9htGLhPLRob7hhBlOGM4pC5gcTsK0e0RBE1aZc0_wq6qW4Z3WLYvC-RrqyLnY5h8NP9N0yrxKlXGNUuIKWNVhp0WzunuzSvjrJ0byzo-GrBkBD_thcBtG_mmkCtf0nKCJ-cXU0oiqffU03JVx8iFnkg0lOPpJxFKOPanbdRgHfI7pUsvpOZH0RINf-n4oFPDeSuAz_EIe1qcxd8PIXWu4M-_QimaePnwupCjFAixux4CgdNcQW7bjdt9ZlQAsiemiphZI-SYenyKMRYcxOaWacyCsbgGpatD4Jwz9aXLxs69e_4AY7Jv2mdMznXPHGQ7FzbH6cyeB8vtFq2cOrDOza3cdp285qlM7mvibMOIGfhV_xNHmLi0hOql0yTot7l1kS7R4UZdWJwHHUYCo6BCLc2HRLWbi0bb7bdkSbkNC6Jlt2xp2-23bObeG7b5lm8N2zxnaZrvfd0x4NIfd9rnZt_ow0O0N2rbtWFa7Nxzag3a_2x302tag27WAjDnstXv9oWW3e-LMFUZ6Tg9whz34OjSBZG9gA6ue2esCsaFtly7CTtRhGhd2z6_P9sSNWwaq7JJ2jjxlTOHFoa48eUPR7rJMKiA0JQ9JYFc0h7zJm5Yr3x-n8uHDmvMN-9TpvLy8nG0QX4dL_EooPnNDv7MBomDqj7Lf_VGI2LmEn6vVt6tb8nr3P_qN239cf3ne2ve_O8y5dR96P7g7WTnR-IY9PC9-fhm-RF__WvLodbj5a7Gd3nybft0MgcSFFKKTSpE0-FkS3aKsjIinSjV4z9pR6QfDw69flWehCgt49sx4lDwn7im6TXlZuipxm_S4ioE0KoTzatyWSNcpiTfqiOCTs0hEp3iQcWvA8vRy6W1FKn6CKp5ltTAgchKgpM1Dhb82G_BXKdJl1zc5vo03YjZ8xr7oEAOdy-lUUJFFKIwmszGAxSK7pCuOjiUKu9qNZ_Nb0coq36MuA6RzM6f6LUaiLSFLl2TGqROJvDBOB0p1sLq8AAl1QfN6Xdwev8yJyIgngUtjD0-C5NZDZp8KsCze1X1ztBCKwtKBqMdaSXCArFV24jgf5JoEECYJupwXW3VaLibDnSTysJyIftBKnUXdY448xJE6CeoI43SkCPBUgk8uvouF1xWrl5ROyfCZhgtEzUOSJEIU4awU7u-CtUUSV9lNMTXCpSHZymlYEV5dZ0hC863GgOwBa_kuCbTj9siCMUHK_w3g_2yMhJ8xKRgikSb5VLLFSLnu3UF5EHYvJt8XjOkVn_SGT6KhuudTYagRq_vkGgRshXF2t6up4wqERHuPQA2i5AqDXK7cu3u3nEpehgovqGrF6tydPKa1jcqkv_4y6uz9M8y_qYGc5A==

Before screenshot:

red text

After screenshot:

image image

@LocalIdentity
Copy link
Copy Markdown
Contributor

Already fixed by #1761

@Eucelia
Copy link
Copy Markdown
Contributor Author

Eucelia commented May 23, 2026

heh, I missed a git pull for updates ._.

@LocalIdentity
Copy link
Copy Markdown
Contributor

@Eucelia you should always be using the dev branch when testing something to see if we've already made a fix

@Eucelia
Copy link
Copy Markdown
Contributor Author

Eucelia commented May 23, 2026

Yeah was my mistake, I had my local branch tracking /master instead of /dev. old habits die hard

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.

2 participants