-
Notifications
You must be signed in to change notification settings - Fork 387
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 ability to crop raster source extent #1030
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the tests! It's a little confusing that the coordinate system is flipped for top vs. bottom and left vs. right. Eg. top=0.2 and bottom=0.8 refer to the same point. Being a little more explicit in the docs would be helpful.
I have updated it to use a NamedTuple and elaborated the usage a bit more in the description. |
blank comment |
Overview
Allows different portions of the same scene image to be used in different dataset splits.
extent_crop
field toRasterSourceConfig
that is passed on to the RasterSources.raster-vision/rastervision_core/rastervision/core/data/raster_source/raster_source_config.py
Lines 23 to 33 in 1d278f8
RaterioSource
andMultiRasterSource
to make use ofextent_crop
.Box()
.Checklist
docs/changelog.rst
needs-backport
label if PR is bug fix that applies to previous minor releaseTesting Instructions