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

v1.2 Release - Minification Tweaks + Velocity Module #8

Merged
merged 12 commits into from
Sep 23, 2021
Merged

Conversation

Tmktahu
Copy link
Owner

@Tmktahu Tmktahu commented Sep 22, 2021

This PR requires in-game testing before I can merge it.

This PR does the following:

  • Further optimizes the minification to give enough space for inverting the axes and displaying speed
  • Creates Velocity module v1
  • Adds documentation for new Velocity module
  • Adds tweaked version of IPS for displaying speed on the same screen
  • Adds a variable map text file to current release so I don't go crazy

@Tmktahu Tmktahu self-assigned this Sep 22, 2021
@Tmktahu Tmktahu added Help Wanted Extra attention is needed Needs Testing labels Sep 22, 2021
@Tmktahu
Copy link
Owner Author

Tmktahu commented Sep 23, 2021

Ran into a testing problem. The velocity calculation code compares the previous coordinate vs the current coordinate, but over the script's refresh time, which is 0.2 seconds. Since coordinates are updated every 0.6 seconds, you end up with cycles that are comparing the same coordinates to each other, which results in a speed of 0.

@Tmktahu
Copy link
Owner Author

Tmktahu commented Sep 23, 2021

It seems I need to trade update time for stability. There are small moments of lag + update time offsets that cause the speed to jump around. I'm going to aim for a 0.4 second update time in exchange for a buffer of 3 vector magnitudes that will hopefully let me stabilize things

@Tmktahu
Copy link
Owner Author

Tmktahu commented Sep 23, 2021

turns out the buffer idea was a bad one. I instead went googling around for smoothing methods and settled on using exponential smoothing

@Tmktahu Tmktahu linked an issue Sep 23, 2021 that may be closed by this pull request
@Tmktahu Tmktahu merged commit 7cc6aa1 into main Sep 23, 2021
@Tmktahu Tmktahu mentioned this pull request Sep 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Help Wanted Extra attention is needed Needs Testing
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Velocity Module
1 participant