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

[BUG] ACF_CalcMassRatio giving incorrect total contraption engine power value #245

Open
TwistedTail opened this issue Jan 30, 2022 · 3 comments
Labels
bug Something isn't working 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

Might be worth giving engine code a look after the torque curve update. So far, engines seem to be vastly under-performing now, probably related to the fact the peak torque cannot be achieved and maintained as easily as before, so probably some stat tuning could be done to good effect. It's also possible that engines could have have faulty code in between all the mess that compromises them.

Branch and Version

Master, latest version at the moment (46c474b)

Screenshots

Pictured: Old torque curve

Old torque curve

@TwistedTail TwistedTail added bug Something isn't working 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 30, 2022
@Alexandre425
Copy link
Contributor

A possible, permanent solution for this: model and compensate for Havok's wrong inertia calculations.

As we know, if you apply K * mass force to a prop in gmod, a heavier prop will gain more velocity than a lighter one, even if you remove drag of any kind.

Not knowing exactly what Havok does that causes this, we could make an E2 or Starfall script that takes a prop with no gravity and drag, sets its weight and applies K * mass force for 1 second, and registers its final speed. Do this 100 times, plot the points, and it might provide some insight into what happens and how to compensate for it.

@TwistedTail TwistedTail changed the title [BUG] Noticeable engine power decrease after torque curves update [BUG] ACF_CalcMassRatio giving incorrect total contraption engine power value Jan 30, 2022
@TwistedTail
Copy link
Member Author

Updated the title of the issue, it seems to be related with ACF_CalcMassRatio giving wrong values.

@CheezusChrust
Copy link
Contributor

CheezusChrust commented Jan 31, 2022

Posting for further context - tested an engine between ACF2 and 3 with 100% physical and 50% physical masses, here's the results.

_100% PHYSICAL MASS_
ACF3 (old torque curves): 2749.7 ft-lbs peak,    780.1 HP @ 1520 RPM
ACF2:                     2712.5 ft-lbs peak,    769.5 HP @ 1520 RPM

_50% PHYSICAL MASS_
ACF3 (old torque curves): 1446.7 ft-lbs peak,    410.4 HP @ 1520 RPM
ACF2:                     2170.1 ft-lbs peak,    615.5 HP @ 1520 RPM

The new calculation seems to be providing ~65% of the power that it'd deliver in ACF2.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working 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
Development

No branches or pull requests

3 participants