cosmere icon indicating copy to clipboard operation
cosmere copied to clipboard

Refactor Metalmind Selection Code

Open leafreynolds opened this issue 1 year ago • 2 comments

The code that chooses what Metalmind to tap/store is one of the oldest pieces of code in the project, and likely needs a refactor to improve performance and be more intelligent about what order the metalminds will show in.

  • [ ] Improve performance
  • [ ] Improve Metalmind selection order (Hotbar, Curios, armor, inventory)
  • [ ] Config values for the above to change order

While we are in the area, we should:

  • [ ] Fix bug that leaves only a few points left in a metalmind
  • [ ] Allow grabbing points from multiple metalminds at once as part of one request (which may fix the above)

leafreynolds avatar Mar 21 '24 19:03 leafreynolds

Ooh, I never thought to say - I think I found a solution to the bug where a few points were left - all you need to do is tap or store at level one, depending on whether you're filling or storing.

I'm still open to a proper solution, though, especially as this workaround doesn't work for copper.

R0b0r0b avatar Mar 21 '24 19:03 R0b0r0b

May be worth taking a look at Hexcasting and media works for reference, Hexcasting has to deal with finding media (amethyst and related) in the inventory, and media works implements a better algorithm for the same

MoonlitJolteon avatar Mar 22 '24 14:03 MoonlitJolteon