Skyblocker icon indicating copy to clipboard operation
Skyblocker copied to clipboard

Add chocolate factory helper

Open Emirlol opened this issue 9 months ago • 4 comments

Highlights the best pick based on the price and CPS gain.

Currently, it tries to recalculate every second (when the screen updates due to chocolate gain). I tried to make it update only on click inside of the mixin but doesn't seem to work and I didn't bother to debug as I couldn't really understand the solver implementation. The performance loss is very negligible, and you wouldn't really care about performance in the inventory, but it still bugs me. So lmk if you have a solution.

Also sorry about the optionals, I recently started learning rust and they feel more natural than returning -1. As for the sorted map, I first tried to add the CPS increase factor to the item tooltip but that was a whole another thing I had to read and figure out, so I just left it as is.

Emirlol avatar May 04 '24 12:05 Emirlol

Make sure to rebase this, it won't work on 1.20.6 sadly

AzureAaron avatar May 04 '24 23:05 AzureAaron

Decided to finalize this draft and make the egg helper separately later on, as it's just a game of waiting until next spring and then finding all egg locations.

Emirlol avatar May 06 '24 13:05 Emirlol

Looks like it works pretty well, you just need to do your todos!

viciscat avatar May 07 '24 21:05 viciscat

Looks like it works pretty well, you just need to do your todos!

I forgot about them, ehe Will do

Emirlol avatar May 08 '24 06:05 Emirlol

Idk why but there's a useless merge commit (the third last commit). Here's how you can rebase it away (assuming you're using intellij):

  1. Select the last commit before the rebase: Revert backwards compatible changes in ItemUtils.getHeadTexture to av….
  2. Right Click -> Interactively Rebase from Here.
  3. Select all commits between Revert backwards compatible changes in ItemUtils.getHeadTexture to av… and Fix time tower calculations being wrong due to incorrect string matching (see second screenshot).
  4. Drop (at the top, a bit left of center)
  5. Start Rebasing, should have no conflicts
  6. git push --force-with-lease
Step 1 Step 2

kevinthegreat1 avatar May 23 '24 01:05 kevinthegreat1