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

Optimize Rendering by reducing memory usage #1729

Open
gofal opened this issue Feb 24, 2024 · 0 comments
Open

Optimize Rendering by reducing memory usage #1729

gofal opened this issue Feb 24, 2024 · 0 comments

Comments

@gofal
Copy link
Contributor

gofal commented Feb 24, 2024

What is, according to you, missing from fo-dicom? Please describe.
The rendering has been reworked in PR #1591, but there are many places where memory is allocated temporary. Some of them can be avoided, MemoryPool could be used, Pixeldata has a PinnedArray, which is not necessary, etc
The render method currently always creates a new image instance. It would be better to have an additional method that allows rendering into an existing image instance (e.g. when re-rendering an image after the windowing-option was changed)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant