Conversation
|
Ready for review, but needs textures and the pack.png before merging |
|
Also probably want to update the README to not be shapeless portals copy 😅 |
runcows
left a comment
There was a problem hiding this comment.
Partial Review. I didn't get through a lot of the functions, but I figured something is better than nothing :D
gm4_horsemanship/data/gm4_horsemanship/advancement/feed_horse.json
Outdated
Show resolved
Hide resolved
...rsemanship/data/gm4_horsemanship/function/horse_processing/food/type/chorus_fruit.mcfunction
Outdated
Show resolved
Hide resolved
...ship/data/gm4_horsemanship/function/horse_processing/food/feed_added_food_untamed.mcfunction
Show resolved
Hide resolved
gm4_horsemanship/data/gm4_horsemanship/function/horse_processing/start_riding_horse.mcfunction
Show resolved
Hide resolved
...semanship/data/gm4_horsemanship/function/horse_processing/food/feed_original_food.mcfunction
Outdated
Show resolved
Hide resolved
gm4_horsemanship/data/gm4_horsemanship/function/horse_processing/food/add_age.mcfunction
Outdated
Show resolved
Hide resolved
gm4_guidebook/triggers.json
Outdated
| "heart_canisters": 15, | ||
| "helious_shamir": 69, | ||
| "holographic_tags": 94, | ||
| "horsemanship": 122, |
There was a problem hiding this comment.
122 is taken by lavish livestock right now.
On main branch currently, "__next__": 124
misode
left a comment
There was a problem hiding this comment.
I've just had a go testing this module in-game. I don't think this is mergeable right now.
- There doesn't seem to be any documentation of its features (wiki, readme, guidebook) meaning you have to read the code in order to know what to feed your horse.
- From some quick survival testing, it doesn't seem very fun? The initial speed of your horse is reduced a ton meaning you have to spend a lot of time just slowly riding your horse before it becomes useful.
- The level and stamina systems are not very visible, I had to cheat and look at the scoreboards to make any sense of it.
runhorse if you will
- removed Stamina mechanics - updated Guidebook entries - level 0 horses have higher stats (level 15 horses remain the same)
205ed14 to
856ada2
Compare
Bloo-dev
left a comment
There was a problem hiding this comment.
Full review. I mostly found some minor format issues.
gm4_horsemanship/data/gm4_horsemanship/function/horse_processing/glider/wing/init.mcfunction
Outdated
Show resolved
Hide resolved
.../data/gm4_horsemanship/function/horse_processing/glider/wing/calculate_wing_pitch.mcfunction
Show resolved
Hide resolved
gm4_horsemanship/data/gm4_horsemanship/function/horse_processing/glider/wing/update.mcfunction
Show resolved
Hide resolved
| # schedule from horse_processing/graze/start | ||
| # schedule from here | ||
|
|
||
| execute as @e[type=#gm4_horsemanship:trainable,tag=gm4_horse.grazing,nbt={EatingHaystack:0b}] at @s run function gm4_horsemanship:horse_processing/graze/complete |
There was a problem hiding this comment.
So as I understand this you start a 1t clock the moment you detect a grazing horse until it stops grazing. I guess that is the only way to detect that, but the nbt check on a 20Hz is making me uncomfortable.
|
This PR currently does not merge due to an issue with Guidebook: The clean way to fix this is to make PR which fixes the Guidebook logic. The dirty way is to just show a single, specific, horse armor in the horse wing recipe for now. |
|
With #1276 merged this no longer errors. (once rebased) |
| "shearing_sound": "minecraft:item.horse_armor.unequip", | ||
| "slot": "body" | ||
| }, | ||
| "minecraft:custom_model_data": "item/horse_wings", |
There was a problem hiding this comment.
Since this PR has been retargeted at 26.1, this is no longer valid.
| "minecraft:custom_model_data": "item/horse_wings", | |
| "minecraft:custom_model_data": {"strings":["gm4_horsemanship:item/horse_wings"]}, |
| "icon": { | ||
| "id": "minecraft:elytra", | ||
| "components": { | ||
| "minecraft:custom_model_data": "item/horse_wings" |
Level up your horses by using them to improve their health, speed and jump strength
Full module description will be added soon once the design is more finalized