Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Port carrying from DV [DO NOT MERGE] #1790

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

mqole
Copy link

@mqole mqole commented Feb 27, 2025

This is a rebuild of #1693, using code from DELTA-V. This is one of two different versions of this PR, the other using code from Frontier Station. Please see #1789 for the other version of this PR.

Both iterations of the carrying system are different. I recommend checking each version out in a localhost to test the differences yourself: if you are new to this, just paste the following code into git bash.
git checkout -b mqole-carrying-dv master
git pull https://github.com/mqole/imp-station-14.git carrying-dv

ports DeltaV-Station/Delta-v#2466.

ripped out everything related to pseudoitems because thats something they got for goblins and vulps. testing it seems to work ok, but would really love people more experienced in c# to come look because i'm not that good at this.

🆑
add: You can now pick your friends up like a ragdoll.
add: You can now throw your friends really, really far.

@kipdotnet
Copy link

Youre doing some cocomelon shit to me

@iaada
Copy link

iaada commented Feb 27, 2025

It's hard to make a comparison, but this one is using better implementation of movement speed modifier, and is also using Entity<T>. It seems better written so it gets my vote.

@mqole mqole added the PR limbo A PR that's waiting on other issues / dev stuff to get fixed before we merge label Feb 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR limbo A PR that's waiting on other issues / dev stuff to get fixed before we merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants