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.

Sign up for GitHub

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

Jump to bottom

Mitigate memory-leak in rasterize_features() #671

Closed

forman wants to merge 2 commits into master from forman-666-fix_rasterize_features_mem_leak

Member

forman commented Apr 22, 2022

Hopefully mitigated memory-leak in xcube.core.geom.rasterize_features(). (rasterize_features uses too much memory in xcube 0.10.2 #666)
Introduced new keyword argument use_background for function
xcube.core.geom.rasterize_features() that can be used to
avoid using existing feature variables as background values.

Checklist:

Add unit tests and/or doctests in docstrings
Add docstrings and API docs for any new/modified user-facing classes and functions
New/modified features documented in docs/source/*
Changes documented in CHANGES.md
AppVeyor CI passes
Test coverage remains or increases (target 100%)


          Addressing #666

b80c91e

forman requested a review from pont-us

April 22, 2022 07:59

forman self-assigned this


          Merge branch 'master' into forman-666-fix_rasterize_features_mem_leak

c7cf8af

# Conflicts:
#	CHANGES.md

Member Author

forman commented Apr 22, 2022

Closing, as this change doesn't help.

forman closed this

forman deleted the forman-666-fix_rasterize_features_mem_leak branch

April 25, 2022 11:37

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

Labels

None yet