Skip to content

Conversation

@Peechey
Copy link
Contributor

@Peechey Peechey commented Jan 20, 2025

Description of the problem being solved:

There are Time-Lost jewel mods that can already be on the nodes in the radius of the jewel, and these are being added as a separate line. This PR combines them and fixes the issue of unallocated nodes in the radius not updating immediately.

Link to a build that showcases this PR:

eNqlW1tv27gSfq5_BWHgALs4TqKrL0GyC8d2LgdJ6tpJu7svBSPTNje05EpUEnex__0MSUmWL5KppA-tLM43MxzODIcj9uz3twVDLySMaOCf181jo46I7wUT6s_O648Pl0ft-u-_1c6GmM8_Ty9iysSI9Vvt05n8gRh5Iey8brXqyGM4iu7xgpzXR9ifkbCOcOQRf9JbD9wHPqkjjsMZ4V9TqcZ3E9BzHGKPk_BWcOzGPLgLJoDgYQyIBab-OPCeCb8Kg3gJmtbRCyWviubmbvh59FAHrT6dDRlekXDMMUcR_HVe78Ls8IxcUw4QzGKgN-onpbQXcRjxPl7A42HMeEnI5DBZL6T8LmacLhkVpknorSJ60Lc3x7631sA0Cpk_BByz_nB8WA1FGWjY4hvl8wsGc9PiK6hvZj7lRJt8GNAo8CtprUXcixkDJ9WiHZGIhC-YU01FesHiifqaNrnDPu4FEdejHJIQgoVXAoyJF0B8VZVREXlLp0SfstI8EkBVbd43j8FYl64y4_cpNIIEo0c5DmKmScnXycW0CtNLn7ytydwiqht_LbFVHEM_8nSm3SkW-hKIUNMM-MH1MKN0TfPYMQ272bKdpluo8HC-iqiH2R1-o4t4AUn0AT-TtcCW1S72q9mc-5A4irCuVajuJQ3JO2C9gE3eA5vjICrCNTulkbOmc62SHc07FcQ3vpfRt8u4PvqhTKS5jbCEv0CMIEzEZvvEiC5kLSSJNp2dUcmaET8RuNJLMLeEePMrKDpGmBO93JoLp3LLCuK8ZUu57rFsCf9NRAUzCeB-MzWPzTJQRUONlzSkWhopyn3T18dUMMDAJ-FsNZ5TwtaiWk0d8tRkPbzUQEoXyMO1XGFTXiV3zkMrrpbYn6pKe8FRPsObRrvcEopeLx4IlKMAmJCtyrgQMQyDv4nHKasG64aLIA61Sn_vVBFrTSDdndTRYkQmsae3HQ6mUzGNF3LB4BCkO40MBXoyVgna5Rx7z_1gMtM2mhRSCbGp3zheLiFwhTfoMhDbLlTwNFcXHTka1J_Bl_PRWrpB6wtYU2sLyIoOfSlbEP25iKphS4wOsbaAbDnvIFcsIO_KkzGc0NfJoHBp4MimdaSShJrnwGHwCprPRa8hqkYNtVUGsAtVCYn_c6XNf4NcS8DAn8ShCARtGduIfWLuqA8Rttvz2Gp3uOZx2zKsZsdxzZbVLAJvNj_MY8P8OOVOM8O1jm277bQs02lajl2O29Pa2KHc29rQlLK_0VEBvNv2qDi_fc6_Q1zSBNmhPdAE2eW9pwmiOYeNI4hjWEYZYXnpsRdSXuDsQPZWfVrUFXXTL8Iu4uk0Qh6YGPNbSL_n9Tp6gnfpsxeHEUl-KMQDXUBhFEV9zDGaJEfsrzik2OeW7JRGBIfeXIAuMWNPsLMLTuu38pfsrl5SxknYh3cijQittjmaaRo_O5EdYPF0s1gGIUfkTfwzxCFfndenmEVEEco3wCfi1JdeBkmHsToaz4PX7uRFSHoIAhalIISXS-JPNng8hIQgnFYLnlBCTl78QAscgdYrtQFFYja5xvPNRE7DD0ABgJnNltVwDcftNGyz4xoNp-OaVsNy2pbbcOyW02y4bdvqNEzbMc2G22qZRsOyWm27YXWcFrx320an4TabLox27KbVMNtmE2icptNqNE2z02wA1G4KyrYLHAwgaTpA4rTMVqvhGJKZSK0Np9k2QZG26digjtlyG7bdtM1Gu2mCHk1Qs9F2mx2xWKKthMNVd3NmPgVDcjDOVjd9PXNhpk9nj6Nb-fBpzvkyOj05eX19PV5iPg-m5A3q42PwuJMlgMDAR9EzZexIcD3pwp-LWbd7Ta0_p3etC3zVd3_Et39bD6O_nW_un9_tO3_oXU_GR1M6ermYjY7ad_Ovs6P-6K-rL4OVM_6rM8SPV5bjPnwnHun_sboKrNiYky_drlTsJNXsTPX3I6Vm8ksum5iHNEsdQdpdiJ_J5gHwPOpM1CohhYVOmEAZG9KnmJN0ALzw7V55AjDzefYc8fA-7yHC6vtcQ7qDWi-1dsoF8stetODKe5RXSQ9THqO8JLfg0i2z-W1MCaYL_i6DTwSEeLgPuBoTL9MfZ2OxghEEdcivyCK6WEGuvhTJZyt7JxElqMeEq2SRx6TfXyZkimMm3n-JMaMiwI3821v1GcgPwkXWfANWEOCirFccH1ZLYd_u7W0SuolURCdpOCcv5eee7lq1HmZeJJWjvsdi8Ag_qRizlMHwk1BAfL0SPaVJ_sNRjlMm6NMZaCSTM2hw44NbJLGk8FcseMLMSrnIgEiS-qZCuZHkZRfSGXgTZqob8RCHXPS40g9lxqYIMxXxotKrcO4ch5BSHiVStoYE80iVgLsKSt1-5JZqpuLmjnA8gX3i5AYCKToBG5xIenhQiQWS8WpbCCRPvEjSbjaGssEti3tB7CeelLNuYhxp4pyyRebKoluQJp6fuIv0fuXf4lEylhQ3_jLmUtXzuvA8JQF2T9hbCPbz28kWsWD23Y8XT8KU6t_1YSVPuaCR913sxuK7o0wa8jvp4PJy0Hu4-TpIIGMiz_Uoip8i9Xhe_0rJq9S6D0tAWSQMxRheRiTb6GQgJNNkgCvhJqmuafaRcj-vNUExp8EbCWFfnn2DYiCkpFCvbPyAUkqgKItEXVLETZTpxYxU96EHu7oqkQosJU8UxVx6yj33g8VgCRYqGswKJSejByzBRb6DoKFT6olSsHzJRXZUVCV28aD6w96qZL2T1kkxD3nuKWKgBovB6uxShE5GS6wqz02FVlWjxfA-8XDh3NVgMTirugNfXgXYzyWjKuF0D8ck4eQQNF3KRKOjcGUHjGQkxQw_8zkJk-K1iNMdZJ6UpDRwVL1TyCdHUWIreUorsJAYK4aqLyEFcxBjJUGTfBsoinZaHrKbZ7mC9cjTFLNSja9CGyZNtJJlSPrHBUugRktmkvbQCyaRDJcEiszB3ZeATlQjtSBktsjKkgZU2R9nI7vDH2ez3S7-OMdLOC49F653MloMf-RUFFp7uKiSSIuJCKyPcRDx9TEOom-4eDd6tF2KrLGj8iIk63DuBaejZbGfND7fzUG1Z98Nl93jd6PlBtAnUyJq4bIdIKMpCQ4e-30wBi8JDE1WUq39WWQ9u0q81F64d6aVOaroTi4_lCUARXKAEWzm1yXloh6n7AvINcFMdHoD9jGGO5c8PsJMdCvjJfYnKbvP-4r09TpoWi_gEfCUvcC-aIp-1IY-Waz2MCrW6-wkPQPKLqE4aCW9zTEPRUvgZxAs_hQXkwz32Gg6rbZrdpymep80Lkwr6VZABd6nsIyhdL9UrKD847x-1O4cW0a73exYtt1qqRtRZ_IonXRSxHPWSCnmF0dE3RP7RvASjuwCoSyljpzAJd8VGWEofVanaNQdDWrXq6UfcOqhXsxrYvmOboOIozFeLuc0JDUJltM6RS23NsITGkenaLzAjNXg5MKoB45-ioyabfxHNFRCgsHCSDkvCqaKFA1VMxAlrSTqI8WqVj6MYBoBmkE25kidwCM0gXBAVl6YWsPafcBF76ASszkGQjPPTB0XEXghEmdGJA-Gsn8Ax3R5FRmF4u_zunHs1hPDKg-UfZC1zZMmjLiWK8msTfs_3t98eRzUuuwZaAc0xLULhn-CA_egEgG5sq6sqfruFJmtmqo3kwF4Y9SSLvopGoYEGcfmsbl-1YtDcBNeU6d_mFg2YtWSths8GvlFNGtXwjiRshx4CKYRSRX5R876FCb97y-2ceQYv-atJms6ce0aHHOb1N0kVTOSBt6YUe2fxGCn5r85Dv_9xXSPbONXxAO0UBkMidpkTW5tkrvGUWsPuXISpShoQSOkUhg840iR6CxyCYV1kMI-SOEcpHD3O5tivxvg95C_BEFt8LYkIUdDFi9gypeBB8G35VGtZq0X4ik4yykSqaUGbjWlb6dIXLQv-TGOpzo_8l6XNNohk6BxbRT7JKHJPcqsMyI_ZOLZSDU7sxd93-IsuNkSZgGXjf_h0-PoViR01Yy7YsGLKCLSbwJZz7wIoMQgswJE1ryVEN1FzAivAJA1N7IrI6pPw6puqxzEPgS5IIy_YzXQ-FXcLtHGjaBUqzSVi2CyQumRvLIB3qWeWXk1905IRtzNBMi3v4Mp6P_IK2Eo92ksZX1A5DWBOpxXMmHAdyNNhbRqyYvaC08gnkXd9Y2IO0qRKs5keSQ79YE_pbOkUFI_klJJ4rM3iFPOiOjoya9Lm8lgyLBH5gGbkDDRjYiCMbldkvbuW0Y2pQLAxm2oFHYIlF29Stvu2bcCw7IOCQzX_6EnhbnFmPEcFn8s6ppo55NEsQx48w7VxMW4d8Bktl8j2k4x_SL7j0_iBgwUwxM5tZ74VATVzjQn19WY4_a6udoLNxSHqBRm66Iqy5PetW1Sx-hoLISuhhGdUfZ5KhsPYEzZPdH2E_BFcQ8VctyzT6Jo288sQ2MRKpkyf1NTx4pjL5Dnt2r-n9yrhUOjLMTy-eAAMt0eMklWu3MIs3mbt3Iy2IbZh-RlbeEs4spCrjcX3xH3GxBOzGmyVedn-eu32tnJzv_z_D_AKKYD

Before screenshot:

image
failingUnallocNodes

After screenshot:

image
workingUnallocNodes

@Peechey Peechey added the enhancement New feature, calculation, or mod label Jan 20, 2025
@LocalIdentity LocalIdentity merged commit 666a868 into PathOfBuildingCommunity:dev Jan 20, 2025
1 of 2 checks passed
@Peechey Peechey deleted the feature/merge-time-lost-jewel-mods branch January 29, 2025 15:49
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.

2 participants