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

[SUGGESTION] Use Advanced Ballsockets for wheel braking #235

Open
TwistedTail opened this issue Jan 14, 2022 · 1 comment
Open

[SUGGESTION] Use Advanced Ballsockets for wheel braking #235

TwistedTail opened this issue Jan 14, 2022 · 1 comment
Assignees
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed high priority Stuff that needs to be done as quick as possible serverside

Comments

@TwistedTail
Copy link
Member

Short Description

In their current state, brakes have a lot to desire. Despite there has been attempts to fix braking while the gearbox has no engine power, there's still a lot of problems present on gearboxes.

One of the ways to improve braking could be by using an advanced ballsocket constraint on the wheels connected to the gearbox. That way, all that needs to be done is control the rotational friction on the constraint to apply a brake. This is currently done by Wire Clutch entities, which from what I've heard works flawlessly. I also tried some testbed contraptions with ballsockets applied on the wheels and the braking seems to scale just fine as the friction goes down.

However, there's only two concerns I have at the moment of writing this suggestion which root on my complete lack of knowledge on this topic:

  1. According to one of the comments on the Wire Clutch entity file, the constraint needs to be recreated every time the friction changes. This comment is 7 years old, so there may be a chance we could just update the friction key values on the go.
  2. I don't know how the braking will behave when the wheel is being powered, so this might need to be tested out.

There's also the cons of this adding one constraint per every single wheel connected to a gearbox, which is my opinion is an acceptable drawback for the improvement this could provide.

I've heard there's another methods to accomplish better braking that people does already as an alternative to ACF brakes, I'd gladly hear about them if that were the case.

@TwistedTail TwistedTail added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers high priority Stuff that needs to be done as quick as possible serverside labels Jan 14, 2022
@Stooberton
Copy link
Collaborator

Regarding the first concern, I don't think it will be an issue to recreate the constraint at a lower interval. I don't think users will be able to discern the difference between an update rate at tickrate or at 100ms

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed high priority Stuff that needs to be done as quick as possible serverside
Projects
Status: In progress
Development

No branches or pull requests

2 participants