Use balanced pet/skin allocation in More Animals

Entoarox / StardewMods

My updated mods repository

36 stars 37 forks source link

Use balanced pet/skin allocation in More Animals #60

Closed Pathoschild closed 5 years ago

Pathoschild commented 5 years ago

This PR changes More Animals to allocate pet types/skins in a balanced way. For example:

If you have two cats and one dog, the next pet will be a dog.
If you have one pet with skin A and none with skins B/C/D, the next pet will randomly use one of B/C/D.

~~Let me know if you want to make it configurable, or prefer to keep it entirely random. This doesn't currently apply to non-pet animals, but I can do the same there if you want.~~ Now configurable, and applies to farm animals too.

(Based on a user request sent via Nexus DMs.)

Entoarox commented 5 years ago

I would prefer a config option to switch between the default random behaviour and this.

The pet adoption logic has intentionally been kept mostly random specifically to be more realistic, but I wont deny people the ability to configure things to be less random.

Pathoschild commented 5 years ago

Balanced distribution is now configurable (disabled by default), and applies to farm animals too.

Test build for the player who suggested it: EntoaroxFramework 2.4.3-alpha.20190204.zip.

Entoarox commented 5 years ago

Alternative implementation used instead.

Pathoschild commented 5 years ago

I'm deleting my fork since the PR is done, so here are backup diffs for...