Skip to content

Conversation

@kiwi2kiwi
Copy link

heeeelllllloooooooo yannick from discord here

Hey this is my first git forking and also my first modding experience so please bear in mind that this may be unbalanced. I added lots of commit messages so you know what i did. i had my fun teaking it. don't worry i won't be mad if my changes wont make it into the mod, this is after all my first time.
i tested it for a short time and it didn't crash

kiwi2kiwi and others added 11 commits September 27, 2022 12:00
…thers as a small gain, so it can be leveled as you craft things and gain more experience. need to check the latter for balancing, in my game with fast lerner and lvl 3 maintenance, the maintenance gain from the HCMaintenance_OnGiveXP turned out as 4.88
…eased the ore drops to simulate finding veins. these veins don't exist in the top layers so the normal mine doesn't have multiple drops. Added clay to every sifting recipe because sifted dirt is actually clay (not for every dirt type but in general).
…e dark for all mines that require light. These dark mining runs take way longer.
…e dark for all mines that require light. These dark mining runs take way longer.
recipe Pack Pallet of Steel Pipes
{
HCSteelpipe=12,
HCSteelpipe=48,
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure about this. ofc packing and unpacking should have the same amount, so it doesnt just delete stuff, but i assume that is just a mistake. but also getting lucky and finding a pallet in the world giving you 48 pipes is just soooo much. that is alot of steel.

secondChance = ZombRand(2);
if secondChance == 0 then player:getInventory():AddItem("Hydrocraft.HCSandstone");
elseif secondChance == 1 then player:getInventory():AddItem("Hydrocraft.HCTrashlarge");
player:getInventory():AddItems("Hydrocraft.HCRedclay",1);
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

not sure about the changes to add clay, you are meant to get the clay from the quarry


recipe Dig Quarry
{
PickAxe,
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think this would be ok, but would need to change the dig quarry to keep the auger, and the upgrade recipe from quarry to mine to consume an auger

OnTest:HCNearHerbatable,
keep HCSieve/HCFloursieve,
Popcorn/Cereal/HCHerbs,
Popcorn/Cereal/HCHerbs/HCDirt,
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

are you intending to add a recipe to the HCFindSeeds function to find something from dirt? would give nothing atm with this

Copy link
Owner

@robotex140 robotex140 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hey, left a couple of comments for your consideration

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants