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

Experiments with performance of setting wellbores in module #35

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

HavardNJ
Copy link
Contributor

@HavardNJ HavardNJ commented Dec 29, 2021

Some experiments with performance. For Troll it takes 2-3 seconds before all wellbores are rendered using batching. Running a single batch runs in ~1 sec, adding all children in one go seems to shave off ~200ms. Tried using Pixi.BitmapText instead of Pixi.Text as Pixi.Text is drawn to a texture using the canvas API into a texture per label, in theory BitmapText should be more performant as it uses a single texture. Switching to Troll from other field sometimes takes as little as 600ms for adding wellbores using this, however this is not stable as it crashes after changing pages. Not sure if a 600ms freeze would be acceptable (much faster for all other fields. f.ex. snorre takes <100ms), IMHO showing the wellbores faster provides a better user experience than having an animation which renders the same wellbores in1-2 seconds although the animation looks nice.

@HavardNJ HavardNJ marked this pull request as draft December 29, 2021 14:17
@HavardNJ HavardNJ force-pushed the feature/Performance_experiments branch from e365b7c to 3299fcb Compare March 16, 2022 19:04
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.

None yet

1 participant