Fix integer Glorious Vanity notables, load LUTs from compressed files #4536
Conversation
This file contains 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
move stuff to helper function and add support for "g" format mods
…dd_Timeless-Jewel-Support
defer table creation of specific nodes until that node is read
…s and create it from .zip if it doesn't exist
LocalIdentity
added a commit
that referenced
this pull request
Jul 11, 2022
* WIP: start of Timeless Jewel implementation * QoL minor edit * Feat: add auto-addition of timeless stats to Notables for 'add' case * Feat: added Militant Faith; made additions to Notables on separate line * Feat: Add Elegant Hubris, add support for 'replace' * Fix: remove a print; add a print * Fix: added change recommendations from Lothrik * Fix: fix things that broke in last commit * fix correct file (#11) * Fix Militant Faith * Remove all remaining editedNode logic and add timeless jewel search UI (#12) * Remove all remaining editedNode logic * Initial pass on timeless jewel search UI * Update TreeTab.lua * move handling of result out of read function * swap to using numerical IDs and move seed range check * fix issues * Glorious Vanity * WIP: GV stuff * WIP: more GV processing fixes * WIP: more fixes * Fix: reading the GV file fully * Fix: Glorious Vanity timeless implemenation - still need to set roll values * Second pass on Timeless Jewel search UI (#15) * add handling of small and might_legacy of the vaal * add stats to might and legacy of the vaal * add bias to better determine node type * add stat ranges for might and legacy of the vaal * Implement timeless jewel search function (#16) * Implement timeless jewel search function * remove no-op as its unused now * Update to use 1 NodeIndex file * fix additions conflict * jewel data conflict * Move Glorious Vanity to zip file * Fix Elegant Hubris seeds in jewel search tool (#4528) * Fix integer Glorious Vanity notables, load LUTs from compressed files (#4536) * Inflate/Deflate LUTs * Fixed integer Glorious Vanity notables * move stuff to helper function and add support for "g" format mods * Fixed loading of compressed GV timeless jewel * Add generated .zip files * defer table creation of specific nodes until that node is read * cleanup * Code cleanup * Implement logic to load from binary file by default, and to decompress and create it from .zip if it doesn't exist Co-authored-by: Regisle <Regisle.godform@gmail.com> * Further timeless jewel search improvements (#4529) * Inflate/Deflate LUTs * Further timeless jewel search improvements .. including a passive tree view for socket locations and visual feedback when timeless jewels are added to the build. * Hide socketViewer if hovering over jewelType dropdown * Update TreeTab.lua * Add conqueror selection dropdown menu * Update TreeTab.lua * Add node weights to timeless jewel search * Fixed integer Glorious Vanity notables * Add "required" node weight option, improve formatting * Add small mod searching (Glorious Vanity) * move stuff to helper function and add support for "g" format mods * Swap weight slider and search dropdown positions * Fixed loading of compressed GV timeless jewel * Add generated .zip files * Fix Glorious Vanity small node processing * Update TreeTab.lua * Improve search result output formatting * Automatically determine nearby socket keystones * Exit loop earlier to avoid wasting time * Update TreeTab.lua * Update TreeTab.lua * defer table creation of specific nodes until that node is read * cleanup * Add detailed node breakdown to tooltips * Code cleanup * Update node breakdown to display actual stats * Make most timeless search UI settings persistent * Fix bug where node tooltip results were culled * Erase desired node input on jewel type change * Remove unused Might/Legacy of the Vaal code * Make "required" nodes actually required * Update TreeTab.lua * optimise search function and show total weight in first column * Code clean up * Remove unused variables * Update TreeTab.lua * Fix Glorious Vanity node filtering * Standardize search result number formatting .. and hide search result tooltips when hovering over node selection dropdown. * Add secondary node weight slider for GV * Update TreeTab.lua * Default to 0 for missing nodeWeights instead of 0.1 * Revert last commit + enable second nodeSlider for non-GV jewels * Update TimelessJewelListControl.lua * Fix broken node weights with a fairly big rewrite * Swap nodeWeight/nodeWeight2 back... * Fix timeless jewel node filtering typo * Implement logic to load from binary file by default, and to decompress and create it from .zip if it doesn't exist Co-authored-by: Wires77 <Wires77@users.noreply.github.com> Co-authored-by: Regisle <Regisle.godform@gmail.com> * Further Timeless Jewel search improvements (#4546) * Fix broken might/legacy of the vaal + misc cleanup * Update to new noTooltip draw behavior * Update edit box when slider values change .. and prevent duplicate nodes from being added to edit box. * Remove search result total weight column * Fix dynamic slider update bug * Sanitize user input and fix a few errors * Wipe all search input on reset * Fix another missed variable sync * Rebuild searchListTbl from timelessData.searchList * Save timeless jewel search config to build XML * Update TreeTab.lua * Fixed swapped weight values, minor refactors * Fix search list result updating * Fix statMod2 nil error on small nodes * Properly handle replace and add types, finally * Minor consistency edit * Remove dynamic slider width code * Set modFlag to prompt build saving on several .. .. different events, including: - timelessData.searchList change (node weight sliders) - updateSearchList called - conqueror change - jewel socket change - desired node change (direct and via dropdown list) * Remove lingering duplicate variable from old implementation * Save socketFilter state to build XML * Change tooltip colour for tree node names Co-authored-by: Wires77 <Wires77@users.noreply.github.com> Co-authored-by: LocalIdentity <localidentity2@gmail.com> Co-authored-by: Nostrademous <nostrademous@hotmail.com> Co-authored-by: Regisle <49933620+Regisle@users.noreply.github.com> Co-authored-by: LocalIdentity <localidentity2@gmail.com> Co-authored-by: Lothrik (MaXiMiUS) <maximius@gmail.com> Co-authored-by: Regisle <Regisle.godform@gmail.com> Co-authored-by: Wires77 <Wires77@users.noreply.github.com>
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.
Absorbs #4535 into one PR