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

Fix func_conveyor "SetSpeed" not properly reading float input #353

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

meatspace
Copy link

@meatspace meatspace commented Jan 11, 2025

code credit goes to Verttox from the momentum mod repo

https://togithub.com/momentum-mod/game/pull/346/commits/c4cdba18480de6e6665ef46f3bb535a1c6e15b43 (togithub.com redirect cause crosslink bumps are cringe)

Anyway heres a video of it working in my custom edit of mapbase: https://github.com/meatspace/storage/raw/refs/heads/main/setspeed.mp4

Im not a programmer so ymmv. I don't THINK it breaks anything. I specifically did not include the other half of the commit that changes m_flSpeed to 0 which I believe would break existing maps. (idk how to make that code optional, perhaps you can do that)


Does this PR close any issues?

no

PR Checklist

  • My PR follows all guidelines in the CONTRIBUTING.md file
  • My PR targets a develop branch OR targets another branch with a specific goal in mind

code credit goes to Verttox from the momentum mod repo (but really alien swarm)

https://togithub.com/momentum-mod/game/pull/346/commits/c4cdba18480de6e6665ef46f3bb535a1c6e15b43
(togithub.com redirect cause crosslink bumps are cringe)

Anyway heres a video of it working in my custom edit of mapbase:
https://github.com/meatspace/storage/raw/refs/heads/main/setspeed.mp4

Im not a coder so ymmv or whatever. just thought id suggest it.
@TeamSpen210
Copy link

If speed setting works, conveyors need to keep track of their current offset, otherwise you'll get a jump whenever they change speed. Could be a follow up though.

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