Skip to content

feat(blocks): Implement cobweb#1189

Closed
RestartFU wants to merge 5 commits intodf-mc:masterfrom
RestartFU:master
Closed

feat(blocks): Implement cobweb#1189
RestartFU wants to merge 5 commits intodf-mc:masterfrom
RestartFU:master

Conversation

@RestartFU
Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread server/block/cobweb.go
Comment thread server/block/cobweb.go
Comment thread server/block/register.go Outdated
Comment thread server/block/register.go Outdated
Comment thread server/block/cobweb.go
Comment on lines +28 to +30
vel[0] *= 0.25
vel[1] *= 0.05
vel[2] *= 0.25
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

From reading the wiki, this logic only applies to players. We should probably have some sort of way to handle this for all entity types too

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Also worth noting https://minecraft.wiki/w/Weaving can reduce the movement speed, so something will need to be done to make that possible to implement, even if you don't actually implement that here

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

yep, I plan on implementing weaving in another PR

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Does it not make sense to put the base in this PR so you're not re-doing all the work you've already done? Technically this physics behaviour isn't correct as it stands. It works for normal players, but it's different for other entities and even more different for item entities. Have you tested to see if this works correctly with jumping behaviour too?

the player can move at a speed of about 25% of the normal walking speed, and their jumping height is severely reduced

From the wiki, but could just mean the same 25% reduction

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Does it not make sense to put the base in this PR so you're not re-doing all the work you've already done? Technically this physics behaviour isn't correct as it stands. It works for normal players, but it's different for other entities and even more different for item entities. Have you tested to see if this works correctly with jumping behaviour too?

the player can move at a speed of about 25% of the normal walking speed, and their jumping height is severely reduced

From the wiki, but could just mean the same 25% reduction

I don't see the point of implementing the base of weaving in this PR, if that's what you mean by base?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I mean the base of having different multipliers for velocity, not specifically for weaving

Comment thread server/block/cobweb.go
@RestartFU RestartFU closed this by deleting the head repository Mar 27, 2026
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.

3 participants