clientcommands icon indicating copy to clipboard operation
clientcommands copied to clipboard

Villager RNG Manipulation

Open RealRTTV opened this issue 1 year ago • 20 comments
trafficstars

Add Villager RNG Manipulation to brute-force optimal trades

RealRTTV avatar May 31 '24 01:05 RealRTTV

Holy fuck nice!

xpple avatar May 31 '24 09:05 xpple

yooooo

erxson avatar May 31 '24 09:05 erxson

wait this is huge if this works

vanvac avatar Nov 24 '24 15:11 vanvac

it does, but im waiting on earth to do some final touches

RealRTTV avatar Nov 24 '24 17:11 RealRTTV

that will be sooo useful this is probably the best mod ive used

vanvac avatar Nov 24 '24 18:11 vanvac

XD i am seeing the build in progress live

vanvac avatar Nov 26 '24 21:11 vanvac

👍

vanvac avatar Nov 30 '24 16:11 vanvac

can i test out build artifacts? if so how?

vanvac avatar Nov 30 '24 22:11 vanvac

can i test out build artifacts? if so how?

Yeah you just unzip it, get the mod jar out of it and then install it like a regular mod. It's in a working state but I won't be providing support for using it. It needs to be made more user friendly before it can be merged.

Earthcomputer avatar Nov 30 '24 22:11 Earthcomputer

image the only zip file i could find in the artifacts was for im guessing what are the unit tests (cheked the zip files and all there is are the html pages)

vanvac avatar Nov 30 '24 22:11 vanvac

Ah sorry, I forgot artifacts are not built for PRs. You will have to clone the PR and build it yourself. But again, this channel is for development, it's best to join the discord and ask in support if you get stuck.

Earthcomputer avatar Nov 30 '24 22:11 Earthcomputer

any updates/problems or just waiting for soemthing to do with mc versions

vanvac avatar Dec 21 '24 19:12 vanvac

currently we need to fix levelUp changes, but other than that it should be good

RealRTTV avatar Dec 22 '24 15:12 RealRTTV

hope this comes soon bc making villiager trading halls with this will be so much easier

vanvac avatar Dec 22 '24 16:12 vanvac

Idk if this is a knows issue or not but if a viliager loses its profession while its running the rng manipulation after running /cvilliager start the villiager cracking just dies and prevents the command from being run again, it is fixed by reloging and prob many more

vanvac avatar Dec 25 '24 20:12 vanvac

does /cvillager reset fix it?

RealRTTV avatar Jan 16 '25 23:01 RealRTTV

tried to recreat it and turns out my description isnt the best also /cvilliager reset isn't an available command

reproduce problem:

  1. setup villiager cracking setup with a cracked villiager and a goal that would take a decent amount of time to find
  2. break the profession block when the bar at the bottom is flling and the cracking is doing its thing
  3. once the cracking completes it trys to open the villiager gui but cant as it is unemployed
  4. then giving the villiager a proffession and trying to crack it would fail

fixes:

  • relog
  • interact with the villiager and reset the proffesion so you can crack trades again

half fix: these things allow the villiager rng to be cracked again but do not fix the soft lock

  • anything that resets the villiager manipulation atumatically (eg. holding item in hand, villiager hurt, bad setup, etc)

ps. are there any plans to make it where you can specify like a below certian amount of emrald requirement eg. sharp 5 book for <=24 emralds

vanvac avatar Jan 18 '25 09:01 vanvac

Hello! I'm sorry for non-Development related Question, but I really want to ask this so much: Is this "RNG Manipulation" will allow to manipulate Armorer's/Smiths' last (Master-Expert) Trades like Diamond Stuff?

https://github.com/Earthcomputer/clientcommands/pull/642#issuecomment-2558501481 yes it will but im pretty sure that the feature to manupilate the traded for upgraded villiagers is broken as of now

vanvac avatar Feb 21 '25 10:02 vanvac

I'm very interested in this so when will this be merged

GodGun968 avatar Oct 02 '25 10:10 GodGun968

I was working on it a while ago, the last thing I was doing iirc was making nextLevel work correctly, I can maybe boot 'er up and see if I can patch it, I know someone made a diff to update to 1.21.9 or smth too.

RealRTTV avatar Oct 02 '25 11:10 RealRTTV

Alright, keep it up. I'll provide the Chinese translation once you've finished and merged it.

GodGun968 avatar Oct 02 '25 11:10 GodGun968