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

Optimized complexity of the MemoryStream write operation. #116

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

Conversation

marko1597
Copy link

Optimized complexity of the MemoryStream write operation. Before complexity of the MemoryStream write operation was O(n) now it is
O(log(n)). Complexity gets amortized to O(1) in case of large number of write
operations.

This change solves issue that occurs when the user has 100MB environment map,
which causes Mitsuba renderer to serialize the environment in 20 minutes. Slowdown
occurs when user tries to render the scene with the large environment map using
distributed rendering with multiple machines in the network.

complexity of the MemoryStream write operation was O(n) now it is
O(log(n)). Complexity gets amortized to O(1) in case of large number of write
operations.

This change solves issue that occurs when the user has 100MB environment map,
which causes Mitsuba renderer to serialize the environment in 20 minutes. Slowdown
occurs when user tries to render the scene with the large environment map using
distributed rendering with multiple machines in the network.
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.

1 participant