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

Add soak test for memory usage with external tilesets #1415

Open
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

azrogers
Copy link
Contributor

@azrogers azrogers commented May 6, 2024

To test out CesiumGS/cesium-native#876, I've created a long-running soak test to evaluate the memory usage of cesium-native over a long period. The test consists of loading the Google P3DT and flying to random locations around the globe over and over again for twelve straight hours. There's a few issues with this test as it stands:

  1. Because of Google-compatible token refresh cesium-native#749, the test can only actually run for three hours total, since after that the Google P3DT token will need to be refreshed and we don't currently handle this possibility. I can't think of a better dataset than P3DT to run this test with, so do we fix this bug instead?
  2. After each flight it chooses a new position in the opposite quadrant of the earth to fly to (if it's in the northern and western hemispheres, it will choose a point in the southern and eastern hemispheres, for example). This means most of the points it chooses are in the middle of nowhere (fields, ocean) which aren't exactly a great test. It would be great to choose random locations that have lots of buildings and therefore content to load instead, but I can't think of a way to do this other than working from a set list of locations.

@csciguy8
Copy link
Contributor

csciguy8 commented May 6, 2024

There's a few issues with this test as it stands... the test can only actually run for three hours total... I can't think of a better dataset than P3DT to run this test with, so do we fix this bug instead?

Hitting the token refresh interval doesn't sound like an issue with this PR. If the soak test does fail because of token issues, sounds like it's doing the job perfectly (finding any issue when running a long time).

@kring
Copy link
Member

kring commented Jun 4, 2024

(deleted this comment because I added it to the wrong issue)

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.

3 participants