Skip to content

šŸ› Fix interpolationPeriod logic and update to 0.03f for accurate metr…#34

Open
cagricibuk wants to merge 1 commit into
yagiziskirik:masterfrom
cagricibuk:fix-interpolation-logic
Open

šŸ› Fix interpolationPeriod logic and update to 0.03f for accurate metr…#34
cagricibuk wants to merge 1 commit into
yagiziskirik:masterfrom
cagricibuk:fix-interpolation-logic

Conversation

@cagricibuk
Copy link
Copy Markdown

What does this PR do?

Why is this necessary?

  • The previous interpolationPeriod value (5.0f) was too high and incompatible with the typical range of time, as described in issue Charts and no animation for orbitĀ #27 by @tkdavis. Also same solution for issue Charts Not LoadingĀ #30
  • The condition if (time >= interpolationPeriod) was never true, making it redundant.
  • The new logic uses Mathf.Floor to reliably determine when to process metrics, ensuring better performance and accuracy.

Linked Issue:

Closes #27,#30

Checklist:

  • Tested for functionality and performance.
  • Verified with the updated time logic.

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.

Charts and no animation for orbit

1 participant