Skip to content
New issue

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

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

Already on GitHub? # to your account

Sugestão de melhoria de código para determinar o RPM #12

Closed
plasmaXZ opened this issue Feb 20, 2022 · 1 comment
Closed

Sugestão de melhoria de código para determinar o RPM #12

plasmaXZ opened this issue Feb 20, 2022 · 1 comment
Labels
enhancement New feature or request

Comments

@plasmaXZ
Copy link

plasmaXZ commented Feb 20, 2022

No arquivo "GTAFmod.cpp" tem uma linha pra determinar o RPM do veículo que está assim:

float currentRatio = (velocity - m_pTransmission->m_aGears[nGear].m_fChangeDownVelocity)
                / (*(float*)&m_pTransmission->m_aGears[nGear].m_fMaxVelocity
                    - m_pTransmission->m_aGears[nGear].m_fChangeDownVelocity);

Pelo que testei, se essa linha for assim:

float currentRatio = velocity / m_pTransmission->m_aGears[nGear].m_fMaxVelocity;

E mais embaixo, para determinar o targetRpm for assim:

float targetRpm = currentAudio->m_Ini->m_fMaxRPM * params->m_fVelocityChangingPercentage;
targetRpm = max(currentAudio->m_Ini->m_fMinRPM, targetRpm);

Podemos ter melhores resultados.
O único problema até agora que encontrei, é que o RPM máximo padrão do .bank do AE86 tem que ser em torno de 7200 RPM para não cortar giro o tempo todo.
Mas claro, não sei se há uma razão particular pro calculo ser mais complexo que o sugerido, se houver, por favor me avise.

float currentRatio = (velocity - m_pTransmission->m_aGears[nGear].m_fChangeDownVelocity)

Edit: Aqui está uma demonstração de como ficou
https://www.youtube.com/watch?v=CYiMDqyAb3Y

@chrystianfarias chrystianfarias added the enhancement New feature or request label Feb 22, 2022
@JuniorDjjr
Copy link
Contributor

#16 implementado, mas eu deixei uma opção no .ini para o targetRpm antigo pois ele combina mais para dirigir na rua.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants