1.21.5 SCUBA Gear rework #1137
1.21.5 SCUBA Gear rework #1137
Conversation
misode
added
feature-update
- updated helmet texture
|
@EpyonProjects Test needs to be fixed |
BPR02
left a comment
BPR02
left a comment
There was a problem hiding this comment.
Most stuff looks good. I left a comment about using scores instead of tags to track the buoyancy type since nbt checks are necessary to check tags in a predicate/enchantment, but it's probably not that big of a performance difference.
There's also an inapplicable test left which needs to be removed.
gm4_scuba_gear/data/gm4_scuba_gear/function/check_recipes.mcfunction
Outdated
Show resolved
Hide resolved
gm4_scuba_gear/data/gm4_scuba_gear/function/check_recipes.mcfunction
Outdated
Show resolved
Hide resolved
gm4_scuba_gear/data/gm4_scuba_gear/test/use_helmet_and_tank.mcfunction_
Outdated
Show resolved
Hide resolved
- I should be asleep - Addressed all of the concerns Beeps had - Should have checked again at literally any point because this really was not much
- Before putting it onto public server
EpyonProjects
added
the
server-testing
|
EpyonProjects#6 is a PR that adds custom assets to the armor models (i.e. shown on the player model). It however, requires a resource pack to work. This is to be discussed during next meeting. |
|
I did more in-game testing and realized the flippers were changed at some point from the last staff group play test. It seems they are now essentially Depth Strider III boots that make you slow on land. I believe initially the +1 Water Movement Efficiency (aka depth strider III) was to combine with an increase of actual movement speed while swimming (i.e. swap negative movement speed while on land to positive movement speed while swimming). |
BPR02
left a comment
BPR02
left a comment
There was a problem hiding this comment.
As noted in my last comment, the flippers do not give a speed boost while swimming.
|
Additionally, the +20% vertical speed from wetsuit is not working either. From testing, holding either sneak or space, the gravity attribute on the wetsuit item gets set to -1.2d. Tested with this command in a repeating command block: /tellraw @p {nbt:'equipment.legs.components."minecraft:attribute_modifiers"[{"type":"minecraft:gravity"}].amount',entity:"@p"} |
This will make all items appear as missing textures. It's possible to make them look like leather armor without the RP, but making the item sprites missing textures should "force" the RP to be required. This is important because without the RP the armor is invisible, so we should highly discourage that by making the item textures missing without the RP. I.e. make it basically unplayable without the RP so people don't get invisible armor
Add custom armor assets (RP required)
4 participants
Main discussion and info source located on Discord in the forum post
tl;dr:
The wetsuit allows you to float in water, or assist with swimming up- or downwards. This enchant has a tick clock, making this rework the first introduction of GM4 enchants and the first case of an enchant clock. The enchantment is displayed using lore because it uses multiple lines and displays the attributes manually.